added new tests

.github/workflows/e2e_api_tests.yml

@@ -79,7 +79,7 @@ jobs:
             - name: Composer install and build (Dokan-lite)
               if: success()
               run: |
-                  composer install --no-dev -o
+                composer i --no-dev -o || composer update --no-dev -o
 
             - name: Npm install and build (Dokan-lite)
               if: success()
@@ -185,7 +185,7 @@ jobs:
             # Run API tests
             - name: 🧪 Running the api tests
               id: api-test
-              if: steps.db-port.outcome == 'success' && ( github.event_name != 'workflow_dispatch'  || ( github.event_name == 'workflow_dispatch' && (github.event.inputs.testsuite == 'API' || github.event.inputs.testsuite == 'All')))
+              if: always() && steps.db-port.outcome == 'success' && ( github.event_name != 'workflow_dispatch'  || ( github.event_name == 'workflow_dispatch' && (github.event.inputs.testsuite == 'API' || github.event.inputs.testsuite == 'All')))
               timeout-minutes: 5
               run: |
                   cd tests/pw 
@@ -225,7 +225,7 @@ jobs:
 
             # Backup Database
             - name: Backup Database
-              if: steps.db-port.outcome == 'success'
+              if: always() && steps.db-port.outcome == 'success'
               run: |
                   cd tests/pw
                   npm run wp-env run tests-cli wp db export wp-data/db.sql

tests/pw/README.MD

@@ -124,7 +124,7 @@ Create .env file according to .env.example file
 ADMIN=John_Doe                          [Admin username]
 ADMIN_PASSWORD=AdminPass123             [Password for the admin account]
 VENDOR=David_Johnson                    [Vendor username]
-VENDOR2=jhonathon_Smith                 [Vendor username]
+VENDOR2=Johnathon_Smith                 [Vendor username]
 CUSTOMER=Michael_Williams               [Customer username]
 USER_PASSWORD=Passw0rd123               [Password for all other users]
 GMAP=ABCD1234EFGH5678                   [Google Maps API key]

tests/pw/api.config.ts

@@ -27,7 +27,7 @@ export default defineConfig({
     preserveOutput: 'always',
     /* Run tests in files in parallel */
     // fullyParallel  : true,
-    /* Fail the build on CI if you accidentally left testonly in the source code. */
+    /* Fail the build on CI if you accidentally left test.only in the source code. */
     // forbidOnly     : !!CI,
     /* The number of times to repeat each test, useful for debugging flaky tests. */
     repeatEach: CI ? 1 : 0,

tests/pw/e2e.config.ts

@@ -45,7 +45,7 @@ export default defineConfig({
     /* Configure reporters */
     reporter: CI
         ? [
-              ['github'],
+              //   ['github'],
               ['html', { open: 'never', outputFolder: 'playwright-report/e2e/html/html-report-e2e' }],
               //   ['junit', { outputFile: 'playwright-report/e2e/junit-report/e2e-results.xml' }],
               ['list', { printSteps: true }],

tests/pw/package.json

@@ -51,22 +51,22 @@
     "author": "",
     "license": "ISC",
     "devDependencies": {
-        "@typescript-eslint/eslint-plugin": "^7.4.0",
-        "@typescript-eslint/parser": "^7.4.0",
-        "eslint": "^8.57.0",
+        "@typescript-eslint/eslint-plugin": "^7.7.0",
+        "@typescript-eslint/parser": "^7.7.0",
+        "eslint": "8.56",
         "eslint-config-prettier": "^9.1.0",
-        "eslint-plugin-playwright": "^1.5.4",
+        "eslint-plugin-playwright": "^1.6.0",
         "npm-check-updates": "^16.14.18",
         "prettier": "^3.2.5",
-        "typescript": "^5.4.3"
+        "typescript": "^5.4.5"
     },
     "dependencies": {
         "@faker-js/faker": "^8.4.1",
-        "@playwright/test": "1.42",
-        "@wordpress/env": "^9.6.0",
+        "@playwright/test": "1.43",
+        "@wordpress/env": "^9.8.0",
         "dotenv": "^16.4.5",
         "mysqlconnector": "^2.0.6",
         "php-serialize": "^4.1.1",
-        "zod": "^3.22.4"
+        "zod": "^3.22.5"
     }
 }

tests/pw/pages/basePage.ts

@@ -1360,7 +1360,7 @@ export class BasePage {
         await expect(this.page.locator(selector)).toHaveValue(value);
     }
 
-    // assert element to have attribute along with attirbute value
+    // assert element to have attribute along with attribute value
     async toHaveAttribute(selector: string, attribute: string, value: string) {
         await expect(this.page.locator(selector)).toHaveAttribute(attribute, value);
     }

tests/pw/pages/customerPage.ts

@@ -270,12 +270,12 @@ export class CustomerPage extends BasePage {
         const couponIsApplied = await this.isVisible(selector.customer.cCart.removeCoupon(couponCode));
         if (couponIsApplied) {
             await this.clickAndWaitForResponse(data.subUrls.api.wc.store, selector.customer.cCart.removeCoupon(couponCode), 207);
-            await this.toContainText(selector.customer.cWooSelector.wooCommerceNoriceBannerContent, `Coupon code "${couponCode}" has been removed from your cart.`);
+            await this.toContainText(selector.customer.cWooSelector.wooCommerceNoticeBannerContent, `Coupon code "${couponCode}" has been removed from your cart.`);
         }
         await this.click(selector.customer.cCart.addCoupon);
         await this.clearAndType(selector.customer.cCart.couponCode, couponCode);
         await this.clickAndWaitForResponse(data.subUrls.api.wc.store, selector.customer.cCart.applyCoupon, 207);
-        await this.toContainText(selector.customer.cWooSelector.wooCommerceNoriceBannerContent, `Coupon code "${couponCode}" has been applied to your cart.`);
+        await this.toContainText(selector.customer.cWooSelector.wooCommerceNoticeBannerContent, `Coupon code "${couponCode}" has been applied to your cart.`);
     }
 
     // add billing address in checkout

tests/pw/pages/licensePage.ts

@@ -54,7 +54,7 @@ export class LicensePage extends AdminPage {
     }
 
     // refresh license
-    async refresehLicense() {
+    async refreshLicense() {
         await this.goIfNotThere(data.subUrls.backend.dokan.license);
         await this.clickAndWaitForResponse(data.subUrls.backend.dokan.license, licenseAdmin.refreshLicense);
         await this.toContainText(licenseAdmin.successNotice, 'License refreshed successfully.');

tests/pw/pages/loginPage.ts

@@ -79,7 +79,9 @@ export class LoginPage extends BasePage {
     async logoutBackend(): Promise<void> {
         await this.goIfNotThere(data.subUrls.backend.adminLogin);
         await this.hover(selector.backend.userMenu);
-        await this.clickAndWaitForResponseAndLoadState(data.subUrls.backend.adminLogout, selector.backend.logout, 302);
+        await this.page.hover(selector.backend.logout);
+        await this.clickAndWaitForResponseAndLoadState(data.subUrls.backend.adminLogout, selector.backend.logout);
+        await this.toContainText(selector.backend.logoutSuccessMessage, 'You are now logged out.');
         const loggedInUser = await this.getCurrentUser();
         expect(loggedInUser).toBeUndefined();
     }

tests/pw/pages/menuManagerPage.ts

@@ -11,7 +11,7 @@ export class MenuManagerPage extends BasePage {
         super(page);
     }
 
-    // seave settings
+    // save settings
     async saveSettings() {
         await this.toContainText(settingsAdmin.dokanUpdateSuccessMessage, 'Setting has been saved successfully.');
     }
@@ -23,9 +23,9 @@ export class MenuManagerPage extends BasePage {
 
         switch (action) {
             case 'activate':
-                await this.enableSwitcher(settingsAdmin.menuManager.menuSwithcher(menu));
+                await this.enableSwitcher(settingsAdmin.menuManager.menuSwitcher(menu));
                 await this.clickAndWaitForResponseAndLoadState(data.subUrls.ajax, settingsAdmin.menuManager.menuManagerSaveChanges);
-                await this.toHaveBackgroundColor(settingsAdmin.menuManager.menuSwithcher(menu) + '//span', 'rgb(0, 144, 255)');
+                await this.toHaveBackgroundColor(settingsAdmin.menuManager.menuSwitcher(menu) + '//span', 'rgb(0, 144, 255)');
                 //assertion
                 await this.goto(data.subUrls.frontend.vDashboard.dashboard);
                 await this.toBeVisible((selector.vendor.vDashboard.menus as any)[menuLink]);
@@ -34,9 +34,9 @@ export class MenuManagerPage extends BasePage {
                 break;
 
             case 'deactivate':
-                await this.disableSwitcher(settingsAdmin.menuManager.menuSwithcher(menu));
+                await this.disableSwitcher(settingsAdmin.menuManager.menuSwitcher(menu));
                 await this.clickAndWaitForResponseAndLoadState(data.subUrls.ajax, settingsAdmin.menuManager.menuManagerSaveChanges);
-                await this.toHaveBackgroundColor(settingsAdmin.menuManager.menuSwithcher(menu) + '//span', 'rgb(215, 218, 221)');
+                await this.toHaveBackgroundColor(settingsAdmin.menuManager.menuSwitcher(menu) + '//span', 'rgb(215, 218, 221)');
                 //assertion
                 await this.goto(data.subUrls.frontend.vDashboard.dashboard);
                 await this.notToBeVisible((selector.vendor.vDashboard.menus as any)[menuLink]);
@@ -82,18 +82,18 @@ export class MenuManagerPage extends BasePage {
     async cantRenameMenu(menu: string) {
         await this.goIfNotThere(data.subUrls.backend.dokan.settings);
         await this.click(settingsAdmin.menus.menuManager);
-        await this.disableSwitcher(settingsAdmin.menuManager.menuSwithcher(menu));
+        await this.disableSwitcher(settingsAdmin.menuManager.menuSwitcher(menu));
         await this.notToBeVisible(settingsAdmin.menuManager.menuEdit(menu));
     }
 
-    // dashboard cant be alterd
+    // dashboard cant be altered
     async cantAlterMenu(menu: string, isSubmenu?: boolean) {
         await this.goIfNotThere(data.subUrls.backend.dokan.settings);
         await this.click(settingsAdmin.menus.menuManager);
         isSubmenu && (await this.click(settingsAdmin.menuManager.settingsSubMenu));
 
         await this.hasClass(settingsAdmin.menuManager.menuGrabber(menu), 'not-sortable');
-        await this.notToBeVisible(settingsAdmin.menuManager.menuSwithcher(menu));
+        await this.notToBeVisible(settingsAdmin.menuManager.menuSwitcher(menu));
     }
 
     // reorderMenu
@@ -130,6 +130,6 @@ export class MenuManagerPage extends BasePage {
         await this.click(settingsAdmin.menuManager.confirmReset);
         await this.clickAndWaitForResponseAndLoadState(data.subUrls.ajax, settingsAdmin.menuManager.menuManagerSaveChanges);
 
-        await this.toHaveBackgroundColor(settingsAdmin.menuManager.menuSwithcher(menu) + '//span', 'rgb(0, 144, 255)');
+        await this.toHaveBackgroundColor(settingsAdmin.menuManager.menuSwitcher(menu) + '//span', 'rgb(0, 144, 255)');
     }
 }

tests/pw/pages/productAddonsPage.ts

@@ -32,7 +32,7 @@ export class ProductAddonsPage extends VendorPage {
         await this.multipleElementVisible(addonsVendor.table);
 
         await this.clickAndWaitForLoadState(addonsVendor.createNewAddon);
-        await this.click(addonsVendor.addon.addField);
+        await this.clickAndWaitForResponse(data.subUrls.ajax, addonsVendor.addon.addField);
         await this.check(addonsVendor.addon.enableDescription);
 
         // product addon fields elements are visible
@@ -53,7 +53,7 @@ export class ProductAddonsPage extends VendorPage {
         await this.toContainText(addonsVendor.addon.result, addon.category);
         await this.press(data.key.enter);
 
-        add ? await this.click(addonsVendor.addon.addField) : await this.click(addonsVendor.addon.addonFieldsRow('Add-on Title'));
+        add ? await this.clickAndWaitForResponse(data.subUrls.ajax, addonsVendor.addon.addField) : await this.click(addonsVendor.addon.addonFieldsRow('Add-on Title'));
 
         await this.selectByValue(addonsVendor.addon.type, addon.type);
         await this.selectByValue(addonsVendor.addon.displayAs, addon.displayAs);

tests/pw/pages/productQAPage.ts

@@ -1,6 +1,4 @@
 import { Page, expect } from '@playwright/test';
-import { AdminPage } from '@pages/adminPage';
-// import { VendorPage } from '@pages/vendorPage';
 import { selector } from '@pages/selectors';
 import { data } from '@utils/testData';
 import { helpers } from '@utils/helpers';
@@ -38,7 +36,7 @@ export class ProductQAPage extends BasePage {
         await this.multipleElementVisible(bulkActions);
 
         // filter elements are visible
-        const { filterInput, result, resetFilterByVensors, ...filters } = productQAAdmin.filters;
+        const { filterInput, result, resetFilterByVendors, ...filters } = productQAAdmin.filters;
         await this.multipleElementVisible(filters);
 
         // product question & answers table elements are visible
@@ -150,7 +148,6 @@ export class ProductQAPage extends BasePage {
         await this.click(productQAAdmin.questionDetails.answer.deleteAnswer);
         await this.clickAndWaitForResponse(data.subUrls.api.dokan.productAnswers, productQAAdmin.questionDetails.confirmAction, 204);
         await this.toBeVisible(productQAAdmin.questionDetails.answerDeleteSuccessMessage);
-
     }
 
     // delete question

tests/pw/pages/productsPage.ts

@@ -324,7 +324,7 @@ export class ProductsPage extends AdminPage {
         }
     }
 
-    // vendor add downloadble product
+    // vendor add downloadable product
     async vendorAddDownloadableProduct(product: product['downloadable'], productPopup = false): Promise<void> {
         const productName = product.productName();
         const productPrice = product.regularPrice();
@@ -350,7 +350,7 @@ export class ProductsPage extends AdminPage {
         await this.toHaveValue(productsVendor.downloadableOptions.downloadExpiry, product.downloadableOptions.downloadExpiry);
     }
 
-    // vendor add vitual product
+    // vendor add virtual product
     async vendorAddVirtualProduct(product: product['simple'], productPopup = false): Promise<void> {
         const productName = product.productName();
         const productPrice = product.regularPrice();
@@ -408,7 +408,7 @@ export class ProductsPage extends AdminPage {
         await this.clickAndWaitForResponseAndLoadState(data.subUrls.frontend.vDashboard.products, productsVendor.saveProduct, 302);
         await this.toContainText(productsVendor.updatedSuccessMessage, product.saveSuccessMessage);
         await this.toHaveValue(productsVendor.edit.title, productName);
-        //todo: add more assettions
+        //todo: add more assertions
     }
 
     // vendor add variable subscription product
@@ -434,7 +434,7 @@ export class ProductsPage extends AdminPage {
         await this.clickAndWaitForResponseAndLoadState(data.subUrls.frontend.vDashboard.products, productsVendor.saveProduct, 302);
         await this.toContainText(productsVendor.updatedSuccessMessage, product.saveSuccessMessage);
         await this.toHaveValue(productsVendor.edit.title, productName);
-        //todo: add more assettions
+        //todo: add more assertions
     }
 
     // vendor add external product
@@ -449,7 +449,7 @@ export class ProductsPage extends AdminPage {
         await this.clickAndWaitForResponseAndLoadState(data.subUrls.frontend.vDashboard.products, productsVendor.saveProduct, 302);
         await this.toContainText(productsVendor.updatedSuccessMessage, product.saveSuccessMessage);
         await this.toHaveValue(productsVendor.edit.title, productName);
-        //todo: add more assettions
+        //todo: add more assertions
     }
 
     // go to product edit