Open vikram7 opened 8 years ago
I'd gladly merge a PR for more tests! I'm keenly aware of how lacking tests for this library are, and would love to remedy that, sadly I've been cramped on time as of late. Any help is greatly appreciated!
FYI: I've added a PR testing Timex.Ecto.DateTimeWithTimezone.cast/1
-> #30
The current test suite contains a single test for
Timex.Ecto.Time.cast/1
when the input is a string. Other tests are likely needed here:Timex.Ecto.Time.cast/1
when input is binaryTimex.Ecto.Time.cast/1
when input is a timestampTimex.Ecto.Time.cast/1
when input is a map like%{"calendar" => . . . }
Timex.Ecto.Time.cast/1
when input is timeTimex.Ecto.Time.load/1
Timex.Ecto.Time.dump/1
Timex.Ecto.DateTime.cast/1
functions hereTimex.Ecto.DateTimeWithTimezone
.cast/1
,.dump/1
,.load/1
functions hereTimex.Ecto.Date
functions hereI'm happy to put up a PR for these and others that you might think are necessary!