search

Polymer / tattoo

Test All The Things Over and Over
BSD 3-Clause "New" or "Revised" License
12 stars 3 forks source link

Need messaging when web-component-tester isn't installed or old version #34

Closed usergenic closed 7 years ago

usergenic commented 7 years ago

Right now a big json blob is emitted when wct invocation has problems like:

Tests failed: {"state":"session not created","sessionId":null,"hCode":502657352,"value":{"additionalInformation":"\nDriver info: org.openqa.selenium.chrome.ChromeDriver","localizedMessage":"session not created exception\nfrom unknown error: Runtime.executionContextCreated has invalid 'context': {\"auxData\":{\"frameId\":\"89581.1\",\"isDefault\":true},\"id\":1,\"name\":\"\",\"origin\":\"://\"}\n  (Session info: chrome=54.0.2840.98)\n  (Driver info: chromedriver=2.21.371459 (36d3d07f660ff2bc1bf28a75d1cdabed0983e7c4),platform=Mac OS X 10.12.1 x86_64) (WARNING: The server did not provide any stacktrace information)\nCommand duration or timeout: 820 milliseconds\nBuild info: version: '2.52.0', revision: '4c2593c', time: '2016-02-11 19:06:42'\nSystem info: host: 'noms-macbookpro2.roam.corp.google.com', ip: '172.31.146.149', os.name: 'Mac OS X', os.arch: 'x86_6

(and it goes on and on)

We need message that's more like:

Need to install web-component-tester:
npm install web-component-tester -g

ALTERNATELY, we make web-component-tester a dependency of tattoo. I think it isn't right now because it is big. Perhaps that choice needs to be revisited because the other pain hurts more.

usergenic commented 7 years ago

wct is now installed as dependency of tattoo so no mo probs.