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

stash#59

Open
pmerwin wants to merge 1 commit intowebdriverio-boneyard:masterfrom
pmerwin:port-bail-fix
Open

stash#59
pmerwin wants to merge 1 commit intowebdriverio-boneyard:masterfrom
pmerwin:port-bail-fix

Conversation

@pmerwin
Copy link
Copy Markdown

@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
Copy Markdown

jsf-clabot commented Dec 21, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
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
Copy Markdown
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.

2 participants