theintern / intern

A next-generation code testing stack for JavaScript.
https://theintern.io/
Other
4.36k stars 309 forks source link

Report /error topics in console reporter #213

Closed vladikoff closed 10 years ago

vladikoff commented 10 years ago

STR

Set a reporter : reporters: ['console'], in the config.

Expected

Expected to see a credentials error from Sauce Labs

~/mozilla/fxa-content-server on intern2.0
$ node_modules/.bin/intern-runner config=tests/intern_sauce
Listening on 0.0.0.0:9090
Starting tunnel...
Using no proxy for connecting to Sauce Labs REST API.
**********************************************************
A newer version of Sauce Connect (build 1284) is available!
Download it here:
https://saucelabs.com/downloads/sc-4.3-osx.zip
**********************************************************
Started scproxy on port 52417.
Starting secure remote tunnel VM...
Error: Not authorized
Error: Not authorized
  at reject  <node_modules/intern-geezer/node_modules/digdug/SauceLabsTunnel.js:353:17>
  at readStartupMessage  <node_modules/intern-geezer/node_modules/digdug/SauceLabsTunnel.js:364:14>
  at <node_modules/intern-geezer/node_modules/digdug/SauceLabsTunnel.js:434:12>
  at Array.some  <native>
  at Socket.<anonymous>  <node_modules/intern-geezer/node_modules/digdug/SauceLabsTunnel.js:428:21>
  at Socket.EventEmitter.emit  <events.js:117:20>
  at Socket.<anonymous>  <_stream_readable.js:745:14>
  at Socket.EventEmitter.emit  <events.js:92:17>
  at emitReadable_  <_stream_readable.js:407:10>
  at emitReadable  <_stream_readable.js:403:5>

Actual

Process just exits

~/mozilla/fxa-content-server on intern2.0*
$ node_modules/.bin/intern-runner config=tests/intern_sauce 
csnover commented 10 years ago

Per the reporters documentation, the console reporter is not intended to work in conjunction with the test runner, so the behaviour in this case is undefined. However, currently, the console reporter only outputs test failures; it would probably be in our best interest to have it output fatal errors as well, so I will convert this to an enhancement request.

vladikoff commented 10 years ago

Thank you very much, this is really useful!