Use # in tests tags 320

.changeset/lucky-windows-sin.md

@@ -0,0 +1,5 @@
+---
+"saleor-dashboard": patch
+---
+
+Test reports no longer mention users due to test tags matching GitHub usernames.

.github/actions/run-pw-tests/action.yml

@@ -89,7 +89,7 @@ runs:
           PROJECT_PARAMS+="--project=${PROJECT} "
         done
         echo "reportName=all-blob-reports-${SHARD_NUMBER%%/*}" >> $GITHUB_ENV
-        npx playwright test --grep @e2e $PROJECT_PARAMS --shard "$SHARD_NUMBER"
+        npx playwright test --grep "#e2e" $PROJECT_PARAMS --shard "$SHARD_NUMBER"
 
     - name: Upload blob report to GitHub Actions Artifacts
       uses: actions/upload-artifact@v4

package.json

@@ -280,10 +280,10 @@
     "qa:move-videos": "find cypress/videos -type f -name \"*.js.mp4\" -exec mv {} cypress/reports/mochareports/videos \\;",
     "qa:artifact-move-screenshots": "find cypress/reports/*/mochareports -type d -name \"*.js\" -exec mv {} cypress/reports/mochareports \\;",
     "qa:artifact-move-videos": "find cypress/reports/*/mochareports/videos -type f -name \"*.js.mp4\" -exec mv {} cypress/reports/mochareports/videos \\;",
-    "qa:pw-e2e": "npx playwright test --grep @e2e",
+    "qa:pw-e2e": "npx playwright test --grep \"#e2e\"",
     "qa:pw-ui": "npx playwright test --ui",
     "knip": "knip --reporter markdown",
     "knip:fix": "knip --fix"
   },
   "description": "![Saleor Dashboard](https://user-images.githubusercontent.com/44495184/185379472-2a204c0b-9b7a-4a3e-93c0-2cb85205ed5e.png)"
-}
+}

playwright/tests/apps.spec.ts

@@ -21,7 +21,7 @@ const PRE_INSTALLATION_TIMEOUT = 20 * 1000;
 const INSTALLATION_PENDING_TIMEOUT = 50 * 1000;
 const APP_EXPECT_UI_TIMEOUT = 15 * 1000;
 
-test("TC: SALEOR_119 User should be able to install and configure app from manifest @e2e", async ({
+test("TC: SALEOR_119 User should be able to install and configure app from manifest #e2e", async ({
   page,
 }) => {
   await appsPage.gotoAppsList();
@@ -56,7 +56,7 @@ test("TC: SALEOR_119 User should be able to install and configure app from manif
   await iframeLocator.getByText("Save").click();
   await appsPage.expectSuccessBanner({ timeout: INSTALLATION_PENDING_TIMEOUT });
 });
-test("TC: SALEOR_120 User should be able to delete thirdparty app @e2e", async () => {
+test("TC: SALEOR_120 User should be able to delete thirdparty app #e2e", async () => {
   await appPage.waitForNetworkIdleAfterAction(() =>
     appPage.goToExistingAppPage(APPS.appToBeDeleted.id),
   );

playwright/tests/attributes.spec.ts

@@ -22,7 +22,7 @@ for (const attr of attributeClasses) {
   for (const type of ATTRIBUTES.attributeTypesWithAbilityToAddValues.names) {
     const uniqueSlug = `${attr}-${type}-${SALEOR_124_uuid}`.replace(/\s+/g, "-");
 
-    test(`TC: SALEOR_124 User should be able to create ${attr} ${type} attribute with ability to add values, required, public @e2e @attributes`, async ({
+    test(`TC: SALEOR_124 User should be able to create ${attr} ${type} attribute with ability to add values, required, public #e2e #attributes`, async ({
       page,
     }) => {
       await page.context().storageState({ path: "./playwright/.auth/admin.json" });
@@ -56,7 +56,7 @@ for (const attr of attributeClasses) {
   for (const type of ATTRIBUTES.attributeTypesWithoutAbilityToAddValues.names) {
     const uniqueSlug = `${attr}-${type}-${SALEOR_125_uuid}`.replace(/\s+/g, "-");
 
-    test(`TC: SALEOR_125 User should be able to create ${attr} ${type} attribute without ability to add values, NOT required, private @e2e @attributes`, async ({
+    test(`TC: SALEOR_125 User should be able to create ${attr} ${type} attribute without ability to add values, NOT required, private #e2e #attributes`, async ({
       page,
     }) => {
       await page.context().storageState({ path: "./playwright/.auth/admin.json" });
@@ -91,7 +91,7 @@ for (const attr of attributeClasses) {
   for (const entity of ATTRIBUTES.attributeReferencesEntities.names) {
     const uniqueSlug = `${attr}-${entity}-${SALEOR_126_uuid}`.replace(/\s+/g, "-");
 
-    test(`TC: SALEOR_126 User should be able to create ${attr} References attribute for ${entity}, NOT required, public @e2e @attributes`, async ({
+    test(`TC: SALEOR_126 User should be able to create ${attr} References attribute for ${entity}, NOT required, public #e2e #attributes`, async ({
       page,
     }) => {
       await page.context().storageState({ path: "./playwright/.auth/admin.json" });
@@ -133,7 +133,7 @@ const contentAttrWithValues = {
 const attributesWithValuesToBeUpdated = [productAttrWithValues, contentAttrWithValues];
 
 for (const attribute of attributesWithValuesToBeUpdated) {
-  test(`TC: SALEOR_127 User should be able to update attribute values in existing ${attribute.name} attribute @e2e @attributes`, async () => {
+  test(`TC: SALEOR_127 User should be able to update attribute values in existing ${attribute.name} attribute #e2e #attributes`, async () => {
     await attributesPage.gotoExistingAttributePage(attribute.id, attribute.name);
     await attributesPage.clickDeleteAttrValueButton(attribute.valueToBeDeleted);
     await expect(attributesPage.dialog).toBeVisible();
@@ -160,7 +160,7 @@ for (const attribute of attributesWithValuesToBeUpdated) {
 }
 
 for (const attr of ATTRIBUTES.attributesToBeUpdated) {
-  test(`TC: SALEOR_128 User should be able to edit existing ${attr.name} attribute @e2e @attributes`, async () => {
+  test(`TC: SALEOR_128 User should be able to edit existing ${attr.name} attribute #e2e #attributes`, async () => {
     await attributesPage.gotoExistingAttributePage(attr.id, attr.name);
 
     await attributesPage.attributeDefaultLabelInput.fill(`updated ${attr.name}`);
@@ -198,7 +198,7 @@ const contentAttribute = {
 const attributesToBeDeleted = [productAttribute, contentAttribute];
 
 for (const attribute of attributesToBeDeleted) {
-  test(`TC: SALEOR_129 Delete a single ${attribute.name} @e2e @attributes`, async () => {
+  test(`TC: SALEOR_129 Delete a single ${attribute.name} #e2e #attributes`, async () => {
     await attributesPage.gotoExistingAttributePage(attribute.id, attribute.name);
     await attributesPage.clickDeleteButton();
     await attributesPage.dialog.waitFor({
@@ -212,7 +212,7 @@ for (const attribute of attributesToBeDeleted) {
   });
 }
 
-test("TC: SALEOR_130 Bulk delete attributes @e2e @attributes", async () => {
+test("TC: SALEOR_130 Bulk delete attributes #e2e #attributes", async () => {
   await attributesPage.gotoListView();
   await attributesPage.searchAndFindRowIndexes("e2e attribute to be bulk deleted");
   await attributesPage.clickGridCell(0, 0);

playwright/tests/categories.spec.ts

@@ -10,7 +10,7 @@ let categoriesPage: CategoriesPage;
 test.beforeEach(({ page }) => {
   categoriesPage = new CategoriesPage(page);
 });
-test("TC: SALEOR_102 Create basic category @e2e @category", async () => {
+test("TC: SALEOR_102 Create basic category #e2e #category", async () => {
   await categoriesPage.gotoCategoryListView();
   await categoriesPage.waitForDatagridLoaderToDisappear();
   await categoriesPage.clickCreateNewCategoryButton();
@@ -21,7 +21,7 @@ test("TC: SALEOR_102 Create basic category @e2e @category", async () => {
   await categoriesPage.clickSaveButton();
   await categoriesPage.expectSuccessBanner();
 });
-test("TC: SALEOR_103 Edit category @e2e @category", async () => {
+test("TC: SALEOR_103 Edit category #e2e #category", async () => {
   await categoriesPage.gotoExistingCategoriesPage(CATEGORIES.categoryToBeUpdated.id);
   await categoriesPage.typeCategoryName("Updated category");
   await categoriesPage.typeCategoryDescription("Utils description updated");
@@ -30,7 +30,7 @@ test("TC: SALEOR_103 Edit category @e2e @category", async () => {
   await categoriesPage.expectSuccessBanner();
   await expect(categoriesPage.productsGridList).toContainText("beer to be updated");
 });
-test("TC: SALEOR_104 Bulk delete categories @e2e @category", async () => {
+test("TC: SALEOR_104 Bulk delete categories #e2e #category", async () => {
   await categoriesPage.gotoCategoryListView();
   await categoriesPage.waitForDOMToFullyLoad();
   await categoriesPage.checkListRowsBasedOnContainingText(

playwright/tests/channels.spec.ts

@@ -13,7 +13,7 @@ test.beforeEach(({ page }) => {
   configurationPage = new ConfigurationPage(page);
   channelPage = new ChannelPage(page);
 });
-test("TC: SALEOR_97 Create basic channel @e2e @channels", async () => {
+test("TC: SALEOR_97 Create basic channel #e2e #channels", async () => {
   const slugName = new Date().toISOString();
 
   await configurationPage.goToConfigurationView();
@@ -27,7 +27,7 @@ test("TC: SALEOR_97 Create basic channel @e2e @channels", async () => {
   await channelPage.expectSuccessBanner();
 });
 
-test("TC: SALEOR_208 Create channel with all settings @e2e @channels", async () => {
+test("TC: SALEOR_208 Create channel with all settings #e2e #channels", async () => {
   const slugName = new Date().toISOString();
 
   await configurationPage.goToConfigurationView();
@@ -54,7 +54,7 @@ test("TC: SALEOR_208 Create channel with all settings @e2e @channels", async ()
   await expect(channelPage.allowUnpaidOrdersCheckbox).toBeChecked();
 });
 
-test("TC: SALEOR_98 Edit channel - transaction flow, allow unpaid, authorize, prio high stock @e2e @channels", async () => {
+test("TC: SALEOR_98 Edit channel - transaction flow, allow unpaid, authorize, prio high stock #e2e #channels", async () => {
   await channelPage.gotoChannelDetails(CHANNELS.channelToBeEditedSettings.id);
   await channelPage.clickTransactionFlowCheckbox();
   await channelPage.clickAllowUnpaidOrdersCheckbox();
@@ -66,7 +66,7 @@ test("TC: SALEOR_98 Edit channel - transaction flow, allow unpaid, authorize, pr
   await channelPage.clickSaveButton();
   await channelPage.expectSuccessBanner();
 });
-test("TC: SALEOR_99 Delete channel @e2e @channels", async () => {
+test("TC: SALEOR_99 Delete channel #e2e #channels", async () => {
   await channelPage.gotoChannelList();
   await channelPage.clickDeleteButtonOnRowContainingChannelName(CHANNELS.channelToBeDeleted.name);
   await channelPage.deleteChannelDialog.clickDeleteButton();

playwright/tests/collections.spec.ts

@@ -10,7 +10,7 @@ let collectionsPage: CollectionsPage;
 test.beforeEach(({ page }) => {
   collectionsPage = new CollectionsPage(page);
 });
-test("TC: SALEOR_112 Create collection @collections @e2e", async () => {
+test("TC: SALEOR_112 Create collection #collections  #e2e", async () => {
   await collectionsPage.gotoCollectionsListView();
   await collectionsPage.waitForDOMToFullyLoad();
   await collectionsPage.clickCreateCollectionButton();
@@ -27,7 +27,7 @@ test("TC: SALEOR_112 Create collection @collections @e2e", async () => {
   await collectionsPage.clickSaveButton();
   await collectionsPage.expectSuccessBanner();
 });
-test("TC: SALEOR_113 Edit collection: assign product @collections @e2e", async () => {
+test("TC: SALEOR_113 Edit collection: assign product #collections  #e2e", async () => {
   const productToBeAssigned = "Bean Juice";
 
   await collectionsPage.gotoExistingCollectionView(COLLECTIONS.collectionToBeUpdated.id);
@@ -45,7 +45,7 @@ test("TC: SALEOR_113 Edit collection: assign product @collections @e2e", async (
     `Only 1 category should be visible in table`,
   ).toEqual(1);
 });
-test("TC: SALEOR_114 Bulk delete collections @collections @e2e", async () => {
+test("TC: SALEOR_114 Bulk delete collections #collections  #e2e", async () => {
   await collectionsPage.gotoCollectionsListView();
   await collectionsPage.waitForDOMToFullyLoad();
   await collectionsPage.checkListRowsBasedOnContainingText(

playwright/tests/customers.spec.ts

@@ -28,7 +28,7 @@ test.beforeEach(({ page }) => {
   deleteAddressDialog = new DeleteAddressDialog(page);
 });
 
-test("TC: SALEOR_199 Create customer @e2e @customer", async () => {
+test("TC: SALEOR_199 Create customer #e2e #customer", async () => {
   const firstName = faker.name.firstName();
   const lastName = faker.name.lastName();
   const note = faker.lorem.sentence();
@@ -56,7 +56,7 @@ test("TC: SALEOR_199 Create customer @e2e @customer", async () => {
   await expect(customersPage.customerEmailInput).toHaveValue(email.toLowerCase());
 });
 
-test("TC: SALEOR_200 As an admin I should not be able to create customer with duplicated email @e2e @customer", async () => {
+test("TC: SALEOR_200 As an admin I should not be able to create customer with duplicated email #e2e #customer", async () => {
   const firstName = faker.name.firstName();
   const lastName = faker.name.lastName();
   const note = faker.lorem.sentence();
@@ -78,7 +78,7 @@ test("TC: SALEOR_200 As an admin I should not be able to create customer with du
   await customersPage.expectErrorBannerMessage("User with this Email already exists.");
 });
 
-test("TC: SALEOR_201 Update customer account info @e2e @customer", async () => {
+test("TC: SALEOR_201 Update customer account info #e2e #customer", async () => {
   const firstName = faker.name.firstName();
   const lastName = faker.name.lastName();
   const email = faker.internet.email();
@@ -97,23 +97,23 @@ test("TC: SALEOR_201 Update customer account info @e2e @customer", async () => {
   await expect(customersPage.customerEmailInput).toHaveValue(email.toLowerCase());
 });
 
-test("TC: SALEOR_202 Deactivate a customer @e2e @customer", async () => {
+test("TC: SALEOR_202 Deactivate a customer #e2e #customer", async () => {
   await customersPage.gotoCustomerDetailsPage(CUSTOMERS.customerToBeDeactivated.id);
   await customersPage.customerActiveCheckbox.click();
   await customersPage.saveCustomer();
   await customersPage.expectSuccessBanner();
   await expect(customersPage.customerActiveCheckbox).not.toBeChecked();
 });
 
-test("TC: SALEOR_203 Activate a customer @e2e @customer", async () => {
+test("TC: SALEOR_203 Activate a customer #e2e #customer", async () => {
   await customersPage.gotoCustomerDetailsPage(CUSTOMERS.customerToBeActivated.id);
   await customersPage.customerActiveCheckbox.click();
   await customersPage.saveCustomer();
   await customersPage.expectSuccessBanner();
   await expect(customersPage.customerActiveCheckbox).toBeChecked();
 });
 
-test("TC: SALEOR_204 Delete customer from the details page @e2e @customer", async () => {
+test("TC: SALEOR_204 Delete customer from the details page #e2e #customer", async () => {
   await customersPage.gotoCustomerDetailsPage(CUSTOMERS.deleteCustomer.id);
   await customersPage.deleteCustomer();
   await customersPage.deleteDialog.clickDeleteButton();
@@ -123,7 +123,7 @@ test("TC: SALEOR_204 Delete customer from the details page @e2e @customer", asyn
   await expect(customersPage.emptyDataGridListView).toBeVisible();
 });
 
-test("TC: SALEOR_205 Bulk delete customers @e2e @customer", async () => {
+test("TC: SALEOR_205 Bulk delete customers #e2e #customer", async () => {
   const customersToBeBulkDeleted = CUSTOMERS.customersToBeBulkDeleted.names;
 
   await customersPage.goToCustomersListView();
@@ -139,7 +139,7 @@ test("TC: SALEOR_205 Bulk delete customers @e2e @customer", async () => {
   await expect(customersPage.emptyDataGridListView).toBeVisible();
 });
 
-test("TC: SALEOR_206 As an admin I want to add address to the customer and set it as default shipping @e2e @customer", async () => {
+test("TC: SALEOR_206 As an admin I want to add address to the customer and set it as default shipping #e2e #customer", async () => {
   await customersPage.gotoCustomerDetailsPage(CUSTOMERS.editCustomer.id);
   await addressesListPage.clickManageAddresses();
   await addressesListPage.clickAddAddressButton();
@@ -167,7 +167,7 @@ test("TC: SALEOR_206 As an admin I want to add address to the customer and set i
   );
 });
 
-test("TC: SALEOR_209 As an admin I want to update customer's address and set it as default billing @e2e @customer", async () => {
+test("TC: SALEOR_209 As an admin I want to update customer's address and set it as default billing #e2e #customer", async () => {
   await customersPage.gotoCustomerDetailsPage(CUSTOMERS.editCustomer.id);
   await addressesListPage.clickManageAddresses();
   await addressesListPage.clickShowMoreMenu(CUSTOMERS.editCustomer.initialShippingAddress.lastName);
@@ -198,7 +198,7 @@ test("TC: SALEOR_209 As an admin I want to update customer's address and set it
   );
 });
 
-test("TC: SALEOR_210 Delete customer's address @e2e @customer", async () => {
+test("TC: SALEOR_210 Delete customer's address #e2e #customer", async () => {
   await customersPage.gotoCustomerDetailsPage(CUSTOMERS.editCustomer.id);
   await addressesListPage.clickManageAddresses();
   await addressesListPage.clickShowMoreMenu(CUSTOMERS.editCustomer.initialBillingAddress.lastName);
@@ -211,7 +211,7 @@ test("TC: SALEOR_210 Delete customer's address @e2e @customer", async () => {
   ).not.toBeVisible();
 });
 
-test("TC: SALEOR_207 Issue a new gift card for the customer @e2e @customer", async () => {
+test("TC: SALEOR_207 Issue a new gift card for the customer #e2e #customer", async () => {
   const amount = faker.datatype.number(1000).toPrecision(2).toString();
 
   await customersPage.gotoCustomerDetailsPage(CUSTOMERS.editCustomer.id);