IanEsling/fyodor — pull request #2 by KarlWalsh (2014-08-19 21:52:55+00:00)
Title: Added org.joda.time.LocalDate generator

Now with less java 8

I replaced the static java.util.Random instance in RDG with a RandomValues instance (DefaultRandomValues). I think this should be the only way for generators to get hold of basic random values.

As an added bonus If we ever change the way we generate random values in the future (e.g using commons-math3) then the change will be in only one place, and very straightforward.

I added joda-time 2.4 but i'm starting to think this is a bad idea. Perhaps there should be two libraries, a core set of generators and some basic functionality and a joda time specific library, maybe also a third joda currency module:

fyodor-core
fyodor-joda-time
fyodor-joda-money

This will make the transition to java 8 easier, we just stop supporting fyodor-joda-time and those upgrading to java 8 simply stop importing fyodor-joda-time.

I'm not sure how we would tie these three libraries together yet, but something to think about at-least.

Finally, we have a dependency on guava only for the range classes. It may be worth coming up with our own set of range classes so we can avoid pulling in the entire guava library for such a small subset of functionality.

let me know what you think.