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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[QA] Fixing flaky test - TC: SALEOR_205 Bulk delete customers #4857

Merged
merged 2 commits into from May 15, 2024

Conversation

yellowee
Copy link
Contributor

What type of PR is this?

  • 馃拝 Refactor
  • 馃専 Feature
  • 馃敟 Bug Fix
  • 馃敥 Maintenance
  • 馃洜 Workflow CI/CD changes

Related Issues or Documents

  • closes #

Usage Instructions, Screenshots, Recordings

Have you written tests?

  • Yes!
  • No... here is why: Writing tests are mandatory, please replace this text with why test are not included in this PR

[Optional] Description

@yellowee yellowee self-assigned this May 10, 2024
Copy link

changeset-bot bot commented May 10, 2024

馃 Changeset detected

Latest commit: c5ba86c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
saleor-dashboard Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot temporarily deployed to pr-4857 May 10, 2024 10:30 Destroyed
@yellowee yellowee changed the title Fixing flaky test - TC: SALEOR_205 Bulk delete customers [QA] Fixing flaky test - TC: SALEOR_205 Bulk delete customers May 10, 2024
@yellowee yellowee force-pushed the fixing_flaky_bulk_delete_customers_test_e2e branch from fa70f38 to 4dddd70 Compare May 14, 2024 05:56
@github-actions github-actions bot temporarily deployed to pr-4857 May 14, 2024 05:58 Destroyed
@github-actions github-actions bot temporarily deployed to pr-4857 May 14, 2024 07:37 Destroyed
@github-actions github-actions bot temporarily deployed to pr-4857 May 14, 2024 08:05 Destroyed
@yellowee yellowee force-pushed the fixing_flaky_bulk_delete_customers_test_e2e branch from 899f092 to 24f3582 Compare May 14, 2024 10:17
@github-actions github-actions bot temporarily deployed to pr-4857 May 14, 2024 10:19 Destroyed
@yellowee yellowee force-pushed the fixing_flaky_bulk_delete_customers_test_e2e branch from 24f3582 to c5ba86c Compare May 15, 2024 08:48
@yellowee yellowee marked this pull request as ready for review May 15, 2024 09:02
@yellowee yellowee requested review from a team as code owners May 15, 2024 09:02
@yellowee yellowee requested review from Cloud11PL and removed request for witoszekdev and krzysztofzuraw May 15, 2024 11:29
Comment on lines +54 to +55
await this.loader.waitFor({ state: "hidden" });
await this.giftCardDialog.waitFor({ state: "visible" });
Copy link
Member

Choose a reason for hiding this comment

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

Suggestion: It seems like it would be a fit for a separate function as apart from clicking the button it also checks for modal mount

@yellowee yellowee merged commit 70b2c40 into main May 15, 2024
15 checks passed
@yellowee yellowee deleted the fixing_flaky_bulk_delete_customers_test_e2e branch May 15, 2024 11:55
karola312 pushed a commit that referenced this pull request May 23, 2024
* Fixing flaky test - TC: SALEOR_205 Bulk delete customers

* fixing flaky gift card tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

3 participants