Skip to content

Change missed test tag #5395

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.

Sign up for GitHub

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

Merged
szczecha merged 8 commits into from
Feb 17, 2025
Merged

Change missed test tag #5395

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
2ccb268
replace missing tag
szczecha Feb 5, 2025
6d246ef
Merge branch 'main' into replace-tag
szczecha Feb 11, 2025
6530ebd
Merge branch 'main' into replace-tag
szczecha Feb 12, 2025
fc34c58
Merge branch 'main' into replace-tag
szczecha Feb 12, 2025
e8017d1
Merge branch 'main' into replace-tag
szczecha Feb 12, 2025
7f04cb9
Merge branch 'main' into replace-tag
szczecha Feb 13, 2025
6adf69e
Merge branch 'main' into replace-tag
szczecha Feb 17, 2025
8e9272f
Merge branch 'main' into replace-tag
szczecha Feb 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/cyan-flowers-mate.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"saleor-dashboard": patch
---

Test reports no longer mention users due to test tags matching GitHub usernames. Replace overlooked tag
6 changes: 3 additions & 3 deletions playwright/tests/orders.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,7 @@ test("TC: SALEOR_84 Create draft order #e2e #draft", async () => {
await draftOrdersPage.expectSuccessBanner({ message: "finalized" });
});

test("TC: SALEOR_191 Refund products from the fully paid order #e2e @refunds", async () => {
test("TC: SALEOR_191 Refund products from the fully paid order #e2e #refunds", async () => {
// All steps of this test pass (including after hooks), but Playwright
// marks it as failed because of exceeding 30s timeout
test.slow();
Expand Down Expand Up @@ -302,7 +302,7 @@ test("TC: SALEOR_191 Refund products from the fully paid order #e2e @refunds", a
await refundPage.expectSuccessBanner({ message: "Refund has been sent" });
});

test("TC: SALEOR_192 Should create a manual refund with a custom amount #e2e @refunds", async () => {
test("TC: SALEOR_192 Should create a manual refund with a custom amount #e2e #refunds", async () => {
const order = ORDERS.fullyPaidOrderWithSeveralTransactions;

await ordersPage.goToExistingOrderPage(order.id);
Expand All @@ -328,7 +328,7 @@ test("TC: SALEOR_192 Should create a manual refund with a custom amount #e2e @re
const orderRefunds = ORDERS.orderWithRefundsInStatusOtherThanSuccess.refunds;

for (const refund of orderRefunds) {
test(`TC: SALEOR_193 Update order with non-manual refund in ${refund.status} status #e2e @refunds`, async () => {
test(`TC: SALEOR_193 Update order with non-manual refund in ${refund.status} status #e2e #refunds`, async () => {
await ordersPage.goToExistingOrderPage(ORDERS.orderWithRefundsInStatusOtherThanSuccess.id);
await ordersPage.orderRefundList.scrollIntoViewIfNeeded();

Expand Down
Loading