Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Migrating two tests from WebDriver to Cypress #335

Merged
merged 12 commits into from
Aug 25, 2020

Conversation

Salayna
Copy link
Contributor

@Salayna Salayna commented Aug 25, 2020

Closes #199

What does this PR do / solve?

Migrating two tests from WebDriver to Cypress

@Salayna Salayna changed the title Fix Email Test: 2 Passed fix: Email Test: 2 Passed Aug 25, 2020
@Salayna Salayna changed the title fix: Email Test: 2 Passed fix: Migrating two tests from WebDriver to Cypress Aug 25, 2020
adrienjoly
adrienjoly previously approved these changes Aug 25, 2020
Copy link
Member

@adrienjoly adrienjoly left a comment

Choose a reason for hiding this comment

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

LGTM

Good job, Salayna! 💪

@adrienjoly adrienjoly added this to 📥 Inbox / ideas in Development via automation Aug 25, 2020
@adrienjoly adrienjoly moved this from 📥 Inbox / ideas to ⚙ In progress in Development Aug 25, 2020
@adrienjoly
Copy link
Member

image

source: https://app.codacy.com/gh/openwhyd/openwhyd/file/47694698324/issues/source?bid=17325385&fileBranchId=19877779#l14

I think that Codacy does not take our Eslint/prettier configuration into account, because both npm run lint:fix and my VSCode editor formats the file the same way you did!

=> I'm going to disable that rule on the Codacy side.

Copy link
Member

@adrienjoly adrienjoly left a comment

Choose a reason for hiding this comment

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

LGTM

@adrienjoly
Copy link
Member

adrienjoly commented Aug 25, 2020

FYI, I'm trying to fix ESlint's config, for codacy: #339 and #340.

@adrienjoly adrienjoly merged commit 36451d9 into openwhyd:master Aug 25, 2020
Development automation moved this from ⚙ In progress to ✔️ Done / pending QA Aug 25, 2020
adrienjoly pushed a commit that referenced this pull request Aug 25, 2020
## [1.34.9](v1.34.8...v1.34.9) (2020-08-25)

### Bug Fixes

* **tests:** Migrate 2 tests from Webdriver to Cypress ([#335](#335)) ([36451d9](36451d9))
adrienjoly pushed a commit that referenced this pull request Aug 27, 2020
## [1.34.11](v1.34.10...v1.34.11) (2020-08-27)

### Bug Fixes

* **deps:** Update ESLint and Prettier + plugins ([#341](#341)) ([b8c7728](b8c7728)), closes [/github.com//pull/335#issuecomment-680291284](https://github.com//github.com/openwhyd/openwhyd/pull/335/issues/issuecomment-680291284)
@adrienjoly adrienjoly moved this from ✔️ Done / pending QA to 🌲 In production in Development Aug 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development
  
🌲 In production
Development

Successfully merging this pull request may close these issues.

Migrate e2e tests from Webdriver to Cypress
2 participants