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.

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

Jump to bottom

Revert Don't run export tests on PRs (#5206) #5208

Merged
merged 1 commit into from
Oct 11, 2024
+19 −30
Merged

Revert Don't run export tests on PRs (#5206) #5208

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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: 0 additions & 5 deletions .changeset/gold-penguins-check.md
View file
Open in desktop

This file was deleted.

2 changes: 1 addition & 1 deletion .github/workflows/run-test-cron.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ jobs:
PW_WORKERS: ${{ vars.PW_WORKERS }}
PW_RETRIES: ${{ vars.PW_RETRIES }}
URL_TO_RUN: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
PROJECT: "e2e apps-e2e release"
PROJECT: "e2e apps-e2e"

- name: submit-results-to-testmo
if: always()
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/run-test-manual.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ jobs:
PW_WORKERS: ${{ vars.PW_WORKERS }}
PW_RETRIES: ${{ vars.PW_RETRIES }}
URL_TO_RUN: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
PROJECT: "e2e apps-e2e release"
PROJECT: "e2e apps-e2e"

- name: submit-results-to-testmo
if: always()
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/run-tests-on-release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ jobs:
URL_TO_RUN: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
PW_WORKERS: ${{ vars.PW_WORKERS }}
PW_RETRIES: ${{ vars.PW_RETRIES }}
PROJECT: "e2e apps-e2e release"
PROJECT: "e2e apps-e2e"

- name: submit-results-to-testmo
if: always()
Expand Down
6 changes: 0 additions & 6 deletions playwright.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,11 +62,5 @@ export default defineConfig({
use: { ...devices["Desktop Chrome"] },
testMatch: "playwright/tests/apps.spec.ts",
},
{
name: "release",
dependencies: ["setup"],
use: { ...devices["Desktop Chrome"] },
testIgnore: "playwright/tests/apps.spec.ts",
},
],
});
4 changes: 2 additions & 2 deletions playwright/tests/giftCards.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ test("TC: SALEOR_181 Set gift card balance @e2e @gift", async () => {
await giftCardsPage.setGiftCardsBalanceDialog.setBalance("34");
await giftCardsPage.expectSuccessBanner();
});
test("TC: SALEOR_182 Export gift card codes in XLSX file @gift @release", async () => {
test("TC: SALEOR_182 Export gift card codes in XLSX file @e2e @gift", async () => {
await giftCardsPage.clickShowMoreMenu();
await giftCardsPage.clickExportGiftCards();
await giftCardsPage.exportGiftCardsDialog.exportGiftCardCodes("XLSX");
Expand All @@ -130,7 +130,7 @@ test("TC: SALEOR_182 Export gift card codes in XLSX file @gift @release", async
"Your exported gift cards data is ready",
);
});
test("TC: SALEOR_183 Export gift card codes in CSV file @gift @release", async () => {
test("TC: SALEOR_183 Export gift card codes in CSV file @e2e @gift", async () => {
await giftCardsPage.clickShowMoreMenu();
await giftCardsPage.clickExportGiftCards();
await giftCardsPage.exportGiftCardsDialog.exportGiftCardCodes("CSV");
Expand Down
28 changes: 14 additions & 14 deletions playwright/tests/product.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ test.beforeEach(({ page, request }) => {
variantsPage = new VariantsPage(page);
mailpitService = new MailpitService(request);
});
test("TC: SALEOR_3 Create basic product with variants @e2e", async () => {
test("TC: SALEOR_3 Create basic product with variants @e2e @product", async () => {
await productPage.gotoProductListPage();
await productPage.clickCreateProductButton();
await productCreateDialog.selectProductTypeWithVariants();
Expand All @@ -33,7 +33,7 @@ test("TC: SALEOR_3 Create basic product with variants @e2e", async () => {
await productPage.clickSaveButton();
await productPage.expectSuccessBanner();
});
test("TC: SALEOR_5 Create basic - single product type - product without variants @e2e", async () => {
test("TC: SALEOR_5 Create basic - single product type - product without variants @e2e @product", async () => {
await productPage.gotoCreateProductPage(PRODUCTS.singleProductType.id);
await productPage.rightSideDetailsPage.selectOneChannelAsAvailableWhenMoreSelected("Channel-PLN");
await productPage.typeNameDescAndRating();
Expand All @@ -46,7 +46,7 @@ test("TC: SALEOR_5 Create basic - single product type - product without variants
await productPage.clickSaveButton();
await productPage.expectSuccessBanner();
});
test("TC: SALEOR_26 Create basic info variant - via edit variant page @e2e", async () => {
test("TC: SALEOR_26 Create basic info variant - via edit variant page @e2e @product", async () => {
const variantName = `TC: SALEOR_26 - variant name - ${new Date().toISOString()}`;

await productPage.gotoExistingProductPage(PRODUCTS.productWithOneVariant.id);
Expand All @@ -68,7 +68,7 @@ test("TC: SALEOR_26 Create basic info variant - via edit variant page @e2e", asy
`New variant name: ${variantName} should be visible on the list`,
).toBeVisible();
});
test("TC: SALEOR_27 Create full info variant - via edit variant page @e2e", async () => {
test("TC: SALEOR_27 Create full info variant - via edit variant page @e2e @product", async () => {
const variantName = `TC: SALEOR_27 - variant name - ${new Date().toISOString()}`;

await productPage.gotoExistingProductPage(PRODUCTS.productWithOneVariant.id);
Expand Down Expand Up @@ -98,7 +98,7 @@ test("TC: SALEOR_27 Create full info variant - via edit variant page @e2e", asyn
await variantsPage.clickSaveVariantButton();
await variantsPage.expectSuccessBanner();
});
test("TC: SALEOR_44 As an admin I should be able to delete a several products @e2e", async () => {
test("TC: SALEOR_44 As an admin I should be able to delete a several products @basic-regression @product @e2e", async () => {
await productPage.gotoProductListPage();

await productPage.searchAndFindRowIndexes("a product to be deleted via bulk");
Expand All @@ -115,7 +115,7 @@ test("TC: SALEOR_44 As an admin I should be able to delete a several products @e
`Given products: ${PRODUCTS.productsToBeBulkDeleted.names} should be deleted from the list`,
).toEqual([]);
});
test("TC: SALEOR_45 As an admin I should be able to delete a single products @e2e", async () => {
test("TC: SALEOR_45 As an admin I should be able to delete a single products @basic-regression @product @e2e", async () => {
await productPage.gotoExistingProductPage(
PRODUCTS.productWithOneVariantToBeDeletedFromDetails.id,
);
Expand All @@ -130,7 +130,7 @@ test("TC: SALEOR_45 As an admin I should be able to delete a single products @e2
}),
).not.toBeVisible();
});
test("TC: SALEOR_46 As an admin, I should be able to update a product by uploading media, assigning channels, assigning tax, and adding a new variant @e2e", async () => {
test("TC: SALEOR_46 As an admin, I should be able to update a product by uploading media, assigning channels, assigning tax, and adding a new variant @basic-regression @product @e2e", async () => {
const newVariantName = "variant 2";

await productPage.gotoExistingProductPage(PRODUCTS.singleProductTypeToBeUpdated.id);
Expand Down Expand Up @@ -167,7 +167,7 @@ test("TC: SALEOR_46 As an admin, I should be able to update a product by uploadi
"Newly added single image should be present",
).toEqual(1);
});
test("TC: SALEOR_56 As an admin, I should be able to export products from single channel as CSV file @release", async () => {
test("TC: SALEOR_56 As an admin, I should be able to export products from single channel as CSV file @basic-regression @product @e2e", async () => {
await productPage.gotoProductListPage();
await productPage.clickCogShowMoreButtonButton();
await productPage.clickExportButton();
Expand All @@ -182,7 +182,7 @@ test("TC: SALEOR_56 As an admin, I should be able to export products from single
"Your exported products data is ready",
);
});
test("TC: SALEOR_57 As an admin, I should be able to search products on list view @e2e", async () => {
test("TC: SALEOR_57 As an admin, I should be able to search products on list view @basic-regression @product @e2e", async () => {
await productPage.gotoProductListPage();
await productPage.searchAndFindRowIndexes(PRODUCTS.productToAddVariants.name);
await productPage.checkListRowsBasedOnContainingText([PRODUCTS.productToAddVariants.name]);
Expand All @@ -191,7 +191,7 @@ test("TC: SALEOR_57 As an admin, I should be able to search products on list vie
"There should be only one product visible on list",
).toEqual(1);
});
test("TC: SALEOR_58 As an admin I should be able use pagination on product list view @e2e", async () => {
test("TC: SALEOR_58 As an admin I should be able use pagination on product list view @basic-regression @product @e2e", async () => {
await productPage.gotoProductListPage();

const firstPageProductName = await productPage.getGridCellText(0, 0);
Expand All @@ -216,7 +216,7 @@ test("TC: SALEOR_58 As an admin I should be able use pagination on product list
`Product from first page: ${firstPageProductName} should be visible again`,
).toContainText(firstPageProductName);
});
test("TC: SALEOR_59 As an admin I should be able to filter products by channel on product list view @e2e", async () => {
test("TC: SALEOR_59 As an admin I should be able to filter products by channel on product list view @basic-regression @product @e2e", async () => {
await productPage.gotoProductListPage();
await productPage.searchAndFindRowIndexes(PRODUCTS.productAvailableOnlyInUsdChannel.name);
expect(
Expand All @@ -236,7 +236,7 @@ test("TC: SALEOR_59 As an admin I should be able to filter products by channel o
`Product: ${PRODUCTS.productAvailableOnlyInPlnChannel.name} should be visible on grid table`,
).toContainText(PRODUCTS.productAvailableOnlyInPlnChannel.name);
});
test("TC: SALEOR_60 As an admin I should be able update existing variant @e2e", async () => {
test("TC: SALEOR_60 As an admin I should be able update existing variant @basic-regression @product @e2e", async () => {
const variantName = `TC: SALEOR_60 - variant name - ${new Date().toISOString()}`;
const sku = `SALEOR_60-sku-${new Date().toISOString()}`;

Expand Down Expand Up @@ -269,7 +269,7 @@ test("TC: SALEOR_60 As an admin I should be able update existing variant @e2e",
).toBeVisible();
await productPage.productImage.waitFor({ state: "visible" });
});
test("TC: SALEOR_61 As an admin I should be able to delete existing variant @e2e", async () => {
test("TC: SALEOR_61 As an admin I should be able to delete existing variant @basic-regression @product @e2e", async () => {
await productPage.waitForNetworkIdleAfterAction(() =>
variantsPage.gotoExistingVariantPage(
PRODUCTS.singleVariantDeleteProduct.productId,
Expand All @@ -288,7 +288,7 @@ test("TC: SALEOR_61 As an admin I should be able to delete existing variant @e2e
"Deleting last variant from variant details page should redirect to product page",
).toContain(PRODUCTS.singleVariantDeleteProduct.productId);
});
test("TC: SALEOR_62 As an admin I should be able to bulk delete existing variants @e2e", async () => {
test("TC: SALEOR_62 As an admin I should be able to bulk delete existing variants @basic-regression @product @e2e", async () => {
await productPage.waitForNetworkIdleAfterAction(() =>
productPage.gotoExistingProductPage(PRODUCTS.multipleVariantsBulkDeleteProduct.productId),
);
Expand Down
Loading