From 79d1cc4ca8f782cafffb2f08284605fed12e7e58 Mon Sep 17 00:00:00 2001
From: Ankita Shrivastava <ashrivastava@mozilla.com>
Date: Wed, 10 Jul 2024 14:17:37 -0400
Subject: [PATCH] chore(functional): integrate html reporting with smoke tests

---
 .circleci/config.yml                          | 20 ++++++++++++++++++-
 .../tests/settings/changeEmail.spec.ts        |  2 +-
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index a18f296958..8c4015091e 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -371,7 +371,6 @@ commands:
           paths:
             - artifacts/blob-report
 
-
   build:
     steps:
       - run:
@@ -656,6 +655,19 @@ jobs:
       - run-playwright-tests:
           project: << parameters.project >>
       - store-artifacts
+      - persist_to_workspace:
+          root: /home/circleci/project
+          paths:
+            - packages/functional-tests/playwright-report
+
+  #create separate job for html reporting for Stage and Prod
+  create-html-report:
+    executor: default-executor
+    steps:
+      - attach_workspace:
+          at: /home/circleci/project
+      - store_artifacts:
+          path: packages/functional-tests/playwright-report
 
   # Runs functional tests using playwright. These tests support splitting
   # and parallel execution.
@@ -871,6 +883,9 @@ workflows:
               only: /.*/
             tags:
               only: /.*/
+      - create-html-report:
+          requires:
+            - Smoke Test Production - Playwright
 
   # Triggered remotely. See .circleci/README.md
   stage_smoke_tests:
@@ -884,6 +899,9 @@ workflows:
               only: /.*/
             tags:
               only: /.*/
+      - create-html-report:
+          requires:
+            - Smoke Test Stage - Playwright
 
   deploy_fxa_image:
     # This workflow can be triggered after a PR lands on main. It requires approval.
diff --git a/packages/functional-tests/tests/settings/changeEmail.spec.ts b/packages/functional-tests/tests/settings/changeEmail.spec.ts
index 8185205a66..8ac3a48a86 100644
--- a/packages/functional-tests/tests/settings/changeEmail.spec.ts
+++ b/packages/functional-tests/tests/settings/changeEmail.spec.ts
@@ -140,7 +140,7 @@ test.describe('severity-1 #smoke', () => {
       await signin.fillOutEmailFirstForm(initialEmail);
       await signin.fillOutPasswordForm(newPassword);
 
-      await expect(settings.primaryEmail.status).toHaveText(credentials.email);
+      await expect(settings.settingsHeading).toBeVisible();
 
       // Update which password to use the account cleanup
       credentials.password = newPassword;