Fix issue graphql.Time unmarshal time check #432

[panikgaul]

Now we can use both unix timestamps:

t.UnmarshalGraphQL(time.Now().UnixNano())
t.UnmarshalGraphQL(time.Now().Unix())

[pavelnikolov]

Can you, please, add unit tests and I'd be happy to merge this PR. Feel free to create a time_test.go file in the root folder.

[panikgaul]

Can you, please, add unit tests and I'd be happy to merge this PR. Feel free to create a time_test.go file in the root folder.

Of course, I added a couple of tests, the coverage of time_test.go should be 100% now.

[pavelnikolov]

Could you, pelase, check the expected values too? So that each test has got, want pair. This way you will not only check that marshalling and unmarshalling work but also that they return what you expect.

[pavelnikolov]

Also, I wouldn't use time.Now() instead I prefer constants, please

[panikgaul]

Now I added the checks for expeced values with assertion library and remove time.Now() usage.
Can you please check if it is ok for you, Thx!

[pavelnikolov]

@panikgaul could you, please, check the results too in the tests?

[pavelnikolov]

@panikgaul I'm ready to merge this PR, but I don't want testify as part of the library's dependencies. You only use two of it's methods, so it doesn't really bring any significant value. Thank you for your understanding and I'm sorry for delaying the merge.

[pavelnikolov]

@panikgaul are your intentions to remove testify and continue working on this PR?

[pavelnikolov]

#486 solves this problem. Thank you