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

Playwright test - users spec for /pro/users #13434

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Playwright test - users spec for /pro/users #13434

merged 1 commit into from
Feb 26, 2024

Conversation

minkyngkm
Copy link
Contributor

@minkyngkm minkyngkm commented Jan 4, 2024

Done

  • This test can be running only locally.
  • Test adding and deleting users on /pro/users

QA

how to run tests locally

  • Save this credentials in .env.local
  • Go to playwright.config.ts and comment out testIgnore: '**\/pro/**',
  • Command dotrun
  • Command yarn playwright test /tests/playwright/tests/pro/users.spec.ts --ui or yarn playwright test /tests/playwright/tests/pro/users.spec.ts (yarn playwright test if you want to run all the playwright tests)
  • Check tests pass locally

Issue / Card

Fixes #https://warthogs.atlassian.net/browse/WD-8057

Copy link

codecov bot commented Jan 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.41%. Comparing base (f61e4c6) to head (fa99673).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #13434   +/-   ##
=======================================
  Coverage   74.41%   74.41%           
=======================================
  Files         107      107           
  Lines        2838     2838           
  Branches      946      946           
=======================================
  Hits         2112     2112           
  Misses        702      702           
  Partials       24       24           

@minkyngkm minkyngkm changed the title Write users spect for /pro/users Playwright test - users spec for /pro/users Jan 4, 2024
@minkyngkm minkyngkm merged commit 81eb1db into main Feb 26, 2024
27 checks passed
@minkyngkm minkyngkm deleted the users-spec branch February 26, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants