You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by GitBox <gi...@apache.org> on 2019/11/24 13:36:38 UTC
[GitHub] [cordova-coho] raphinesse opened a new pull request #259: test:
remove legacy tape tests
raphinesse opened a new pull request #259: test: remove legacy tape tests
URL: https://github.com/apache/cordova-coho/pull/259
The tests were not very good. They just called `flagutil.computeReposFromFlag` with every imagineable repo, which amounts to a _lot_ of test code that's poorly maintainable, but results in very little code coverage.
Moreover we have proper unit tests for `flagutil.computeReposFromFlag` in `flagutil.spec.js`.
Finally this allows us to get rid of all the `tape` related development dependencies.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org