search

sitespeedio / coach

Clear Eyes. Full Hearts. Can’t Lose.
MIT License
1.21k stars 64 forks source link

Merged DOM/HAR result is not working correctly #42

Closed soulgalore closed 8 years ago

soulgalore commented 8 years ago

When we implemented the coachAdvice structure in the DOM result, the merge broke.

  1. Create tests that checks output from the CLI
  2. Fix the merge issues.
soulgalore commented 8 years ago

We could add some love to the cli test case so we are even more sure the next time something like this happens.

soulgalore commented 8 years ago

added tests to take care of it.