Skip to content

Conversation

nolanlawson
Copy link
Contributor

Details

Updates WebDriverio to v8, and fixes some minor breaking changes in our tests.

Does this pull request introduce a breaking change?

  • ✅ No, it does not introduce a breaking change.

Does this pull request introduce an observable change?

  • ✅ No, it does not introduce an observable change.

@nolanlawson nolanlawson requested a review from a team as a code owner June 15, 2023 21:13
@@ -47,7 +47,7 @@ const wdSuites = suites.reduce((seed, suite) => {
exports.config = {
logLevel: 'warn',

specs: ['./src/**/*.spec.js'],
specs: ['../src/**/*.spec.js'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Now relative to the config file

@@ -17,7 +17,7 @@ module.exports = function (browser) {
browser.addCommand(
'focus',
function () {
return this.execute(function (target) {
return browser.execute(function (target) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this just happened to work – execute should be on the browser, not this (which is an element).

@nolanlawson
Copy link
Contributor Author

/nucleus test

@nolanlawson
Copy link
Contributor Author

/nucleus ignore --reason 'downstream is failing for unrelated reason'

@nolanlawson
Copy link
Contributor Author

/nucleus test

@nolanlawson nolanlawson changed the title chore: update webdrivio to v8 chore: update webdriverio to v8 Jun 16, 2023
@nolanlawson
Copy link
Contributor Author

/nucleus test

@nolanlawson
Copy link
Contributor Author

/nucleus ignore --reason 'downstream is failing due to flapper'

@nolanlawson nolanlawson merged commit 99a3115 into master Jun 16, 2023
@nolanlawson nolanlawson deleted the nolan/wdio-8 branch June 16, 2023 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants