search

Khan / aphrodite

Framework-agnostic CSS-in-JS with support for server-side rendering, browser prefixing, and minimum CSS generation
5.35k stars 188 forks source link

Fix coverage reports #370

Open kevinbarabash opened 5 years ago

kevinbarabash commented 5 years ago

I'm not sure why coveralls is report 0% coverage. It might have to do with using @babel/register. I was thinking we might look at moving to jest which also produces coverage reports doesn't require @babel/register to run ES6. I was also thinking of maybe using codecov.io for tracking coverage as it provides reports inside PRs. I'm not sure if coveralls does that. The current coverage has been broken for a while so starting fresh doesn't seem so bad. https://coveralls.io/github/Khan/aphrodite?branch=master

lencioni commented 5 years ago

Big +1 on moving to jest.

lencioni commented 5 years ago

I think the snapshots would be a great fit for a lot of the current unit tests in particular.