brodybits commented on issue #643: [WIP] test: use verbose spec reporter
URL: https://github.com/apache/cordova-ios/pull/643#issuecomment-586739869
 
 
   Thanks to @erisu and others for the feedback. I still fail to see much 
benefit of reducing the verbose test output in case of running `npm test` on a 
workstation such as mine due to the amount of time I have to wait for the tests 
to finish. I am also not so happy about the idea of little green dots from 
Jasmine interspersed with the amount of console logging that comes from e2e 
tests.
   
   I did make some updates to use the verbose reporter in "e2e-tests", which 
seems to only include a single create.spec.js test. I can think of a few more 
things we should consider:
   
   - move the e2e test into its own subdirectory
   - `npm test` should do `npm run e2e-tests` before the slower `npm run 
objc-tests`, which is already done on Travis CI
   - `npm run objc-tests` seems to output a bunch of console output that is 
hard to follow while waiting for test results, would be nice to see this 
improved someday
   
   I am keeping this PR in WIP status for now, will understand if others want 
to close it.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to