Merge pull request #311 from QAComet/qacomet/orders-tests

Tests for orders related pages

Author: VariableVic

e2e/fixtures/account/order-page.ts

@@ -26,7 +26,7 @@ export class OrderPage extends AccountPage {
 
   constructor(page: Page) {
     super(page)
-    this.container = page.getByTestId("order-complete-container")
+    this.container = page.getByTestId("order-details-container")
     this.backToOverviewButton = page.getByTestId("back-to-overview-button")
     this.orderEmail = this.container.getByTestId("order-email")
     this.orderDate = this.container.getByTestId("order-date")
@@ -63,17 +63,16 @@ export class OrderPage extends AccountPage {
   async getProduct(title: string, variant: string) {
     const productRow = this.productRow
       .filter({
-        has: this.productTitle.filter({ hasText: title }),
+        hasText: title,
       })
       .filter({
-        has: this.productVariant.filter({ hasText: variant }),
+        hasText: `Variant: ${variant}`,
       })
     return {
       productRow,
-      title: productRow.getByTestId("product-title"),
+      name: productRow.getByTestId("product-name"),
       variant: productRow.getByTestId("product-variant"),
-      deleteButton: productRow.getByTestId("delete-button"),
-      quantitySelect: productRow.getByTestId("quantity-select"),
+      quantity: productRow.getByTestId("product-quantity"),
       price: productRow.getByTestId("product-unit-price"),
       total: productRow.getByTestId("product-price"),
     }

e2e/fixtures/account/orders-page.ts

@@ -20,8 +20,14 @@ export class OrdersPage extends AccountPage {
     this.orderDisplayId = page.getByTestId("order-display-id")
   }
 
-  async getOrder(text: string) {
-    const card = this.orderCard.filter({ hasText: "text" }).first()
+  async getOrderById(orderId: string) {
+    const orderIdLocator = this.page
+      .getByTestId("order-display-id")
+      .filter({
+        hasText: orderId,
+      })
+      .first()
+    const card = this.orderCard.filter({ has: orderIdLocator }).first()
     const items = (await card.getByTestId("order-item").all()).map(
       (orderItem) => {
         return {
@@ -35,8 +41,10 @@ export class OrdersPage extends AccountPage {
       card,
       displayId: card.getByTestId("order-display-id"),
       createdAt: card.getByTestId("order-created-at"),
+      orderId: card.getByTestId("order-display-id"),
       amount: card.getByTestId("order-amount"),
       detailsLink: card.getByTestId("order-details-link"),
+      itemsLocator: card.getByTestId("order-item"),
       items,
     }
   }

e2e/fixtures/checkout-page.ts

@@ -42,6 +42,7 @@ export class CheckoutPage extends BasePage {
   submitAddressButton: Locator
   addressErrorMessage: Locator
 
+  deliveryOptionsContainer: Locator
   deliveryOptionRadio: Locator
   deliveryOptionErrorMessage: Locator
   submitDeliveryOptionButton: Locator
@@ -166,6 +167,9 @@ export class CheckoutPage extends BasePage {
       "address-error-message"
     )
 
+    this.deliveryOptionsContainer = this.container.getByTestId(
+      "delivery-options-container"
+    )
     this.deliveryOptionRadio = this.container.getByTestId(
       "delivery-option-radio"
     )