Migrate tests to Ava

[brunocodutra]

relates to #205

[alexander-akait]

Good work!

[alexander-akait]

Only looks tests failed, need fix it 👍

[brunocodutra]

@evilebottnawi I'll add code coverage report with this PR before getting it merged

[alexander-akait]

@brunocodutra let's always use npm@5 in tests and use npm ci

[alexander-akait]

npm i -g npm@5

[brunocodutra]

@evilebottnawi should I commit package-lock.json as well then?

[alexander-akait]

@brunocodutra yes, lock file always should be in repo

[brunocodutra]

@evilebottnawi for some reason npm ci doesn't fly on travis, it chokes on prepublish because appearently gulp is not installed, any idea what's going wrong?

https://travis-ci.org/evilebottnawi/favicons/jobs/360218204#L462

[alexander-akait]

very strange, seems npm bug, let's install gulp manually after npm ci

[brunocodutra]

@evilebottnawi oh wait, that won't work, because prepublish runs just after npm ci, so installing it afterwards won't help.

I guess I'll have to fall back to npm install?

[alexander-akait]

@brunocodutra hm, let's use npm install before we migrate from gulp, we can rewrite stream tests without gulp

[brunocodutra]

@evilebottnawi CI finally passed, could you take a look at it again?

[alexander-akait]

👍