Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

feat: add Nightwatch.js as test framework option #572

Merged
merged 1 commit into from
Oct 19, 2020

Conversation

astagi
Copy link
Contributor

@astagi astagi commented Jul 16, 2020

This PR adds Nightwatch.js as test framework option 🦉

@clarkdo clarkdo requested review from atinux and pi0 July 27, 2020 18:28
@atinux
Copy link
Member

atinux commented Jul 28, 2020

Do you mind resolving the conflicts so we can merge it?

@astagi
Copy link
Contributor Author

astagi commented Jul 29, 2020

No problem @atinux I'll check it this evening or tomorrow and rebase this branch with master resolving the conflict.

@astagi astagi force-pushed the feature/nightwatchjs branch from aaafa12 to 4cf9466 Compare July 30, 2020 08:09
@astagi
Copy link
Contributor Author

astagi commented Jul 30, 2020

@atinux @clarkdo conflicts resolved 👍 (tests on macos lastest 12 timed out, try to restart the CI process)

@astagi astagi force-pushed the feature/nightwatchjs branch from 4cf9466 to 7b7f883 Compare September 24, 2020 15:42
@astagi
Copy link
Contributor Author

astagi commented Sep 24, 2020

@atinux @clarkdo @pi0 I've just rebased this branch with master and updated Nightwatch.js, unfortunately I get some errors on snapshots 😞

@astagi astagi requested review from atinux and clarkdo September 25, 2020 08:58
renovate.json Outdated
@@ -57,6 +57,10 @@
{
"packagePatterns": ["wdio", "webdriver"],
"groupName": "webdriverio packages"
},
{
"packagePatterns": ["nightwatch"],
Copy link
Member

Choose a reason for hiding this comment

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

Should chromedriver be included ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, it needs at least one driver to run tests, chromedriver is the most common one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@clarkdo sorry I replied too fast here! I've just added chromedriver in renovate.json, I forgot to add it before!

@clarkdo
Copy link
Member

clarkdo commented Oct 19, 2020

@astagi I've updated snapshot job, can you please merge latest master and try ?

@astagi astagi force-pushed the feature/nightwatchjs branch from 7b7f883 to b586810 Compare October 19, 2020 15:35
@astagi astagi force-pushed the feature/nightwatchjs branch from b586810 to 1e790b4 Compare October 19, 2020 15:47
@astagi
Copy link
Contributor Author

astagi commented Oct 19, 2020

@astagi I've updated snapshot job, can you please merge latest master and try ?

Thank you @clarkdo 👍 I've just rebased my branch!

@clarkdo clarkdo merged commit 75985e4 into nuxt:master Oct 19, 2020
@clarkdo
Copy link
Member

clarkdo commented Oct 19, 2020

Thanks! @astagi

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.

3 participants