cozy / cozy-emails

Email Client for Cozy
GNU Affero General Public License v3.0
66 stars 41 forks source link

[test] fix and clean basic tests to add complex one (such as remove message) #878

Closed misstick closed 8 years ago

misstick commented 8 years ago

wait for these merge before merge this one:

frankrousseau commented 8 years ago

Can you make sure that the linter returns no error?

m4dz commented 8 years ago

@frankrousseau linters are a complex problem IMHO: never same linters nor version, sparse configs… If I can, I suggest we let @misstick finished the objective of this PR, and I'll open another one to address specific linters issues / config :)

frankrousseau commented 8 years ago

OK to finish this PR without linting. But the linter is already configured. So, there is not much work to do about it. It just needs to be added to Travis.

misstick commented 8 years ago

@poupotte : I've not finished the 1rst scope, but:

@m4dz: you can review this PR, it is "complete" :)

aenario commented 8 years ago

If possible, avoid using new Date() in tests as it is not pure and cause intermittence in my tests.