Skip to content
This repository has been archived by the owner on Dec 18, 2019. It is now read-only.

stash #59

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

pmerwin
Copy link

@pmerwin pmerwin commented Dec 21, 2018

First attempt to port webdriverio/webdriverio@a6ff45a back to v4, not sure what to do about the logger methods as the old service was not using that? Also getting an undefined error

․TypeError: Cannot read property 'mochaOpts' of undefined
at SauceService.after (/Users/pmerwin/Projects/qa/NWDIO/lib-webdriverio-nodejs/node_modules/wdio-sauce-service/build/sauce-service.js:132:34)
at execHook (/Users/pmerwin/Projects/qa/NWDIO/lib-webdriverio-nodejs/node_modules/wdio-sync/build/index.js:162:35)

@jsf-clabot
Copy link

jsf-clabot commented Dec 21, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Author

@pmerwin pmerwin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

․TypeError: Cannot read property 'mochaOpts' of undefined
at SauceService.after (/Users/pmerwin/Projects/qa/NWDIO/lib-webdriverio-nodejs/node_modules/wdio-sauce-service/build/sauce-service.js:132:34)
at execHook (/Users/pmerwin/Projects/qa/NWDIO/lib-webdriverio-nodejs/node_modules/wdio-sync/build/index.js:162:35)

so far changes are failing

failures = 1
}

// const status = 'status: ' + (failures > 0 ? 'failing' : 'passing')
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what do about the logging methods as that was not previously done here in this service it looks like?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants