search

dalekjs / dalek-driver-sauce

[unmaintained] Sauce Labs bindings for DalekJS
MIT License
4 stars 5 forks source link

Finishing command for sauce labs #6

Open cupito opened 10 years ago

cupito commented 10 years ago

Seems that the tests do not send a term signal to sauce labs. I can see Test did not see a new command for 90 seconds. Timing out. for every test send with dalekjs. Due to the fact that the accounts are time based, it's maybe not the best way for waste money. Can a finishing signal send to saucelabs?

asciidisco commented 10 years ago

Damn. That is a huge bug, thanks for finding. We will take care of it asap.

marsch commented 10 years ago

@cupito mhh i have the same problem and I basically think that only happens if a test isn't finished with .done() - which happens when a command isn't found for me its .setValue() - which isn't implemented in the sauce-driver yet

@asciidisco is that right? any thought?

jonpreston commented 10 years ago

I'd really love to be able to use sauce for my test suite, but this is a major blocker. Any updates?

amitaibu commented 9 years ago

I have a test that used done(), but still face this issue -- it seems related to using the screenshot() command,

https://github.com/dalekjs/dalek-driver-sauce/issues/10#issuecomment-74051489