From 79da00f76d789d30ad98ef40dd30c507a2f3f4b8 Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Fri, 24 Sep 2021 08:03:06 -0700
Subject: [PATCH 01/11] Fix a11y async errors by flushing promises first

---
 .../accordionSection/accordionSection.html    |  1 +
 .../attendance/__tests__/attendance.test.js   | 16 +++----
 .../__tests__/formattedField.test.js          | 32 ++++++-------
 .../__tests__/listViewSelector.test.js        |  4 +-
 .../__tests__/loadingStencil.test.js          | 11 +++--
 .../default/lwc/modal/__tests__/modal.test.js | 14 +++---
 .../__tests__/newProgramEngagement.test.js    | 12 ++---
 .../__tests__/scopedNotification.test.js      | 48 +++++++++++--------
 .../__tests__/serviceScheduleCreator.test.js  | 11 +++--
 .../serviceScheduleCreatorWrapper.test.js     |  4 +-
 jest.setup.js                                 |  1 +
 11 files changed, 82 insertions(+), 72 deletions(-)

diff --git a/force-app/main/default/lwc/accordionSection/accordionSection.html b/force-app/main/default/lwc/accordionSection/accordionSection.html
index 2c435e652..8adc02217 100644
--- a/force-app/main/default/lwc/accordionSection/accordionSection.html
+++ b/force-app/main/default/lwc/accordionSection/accordionSection.html
@@ -1,3 +1,4 @@
+<!-- sldsValidatorIgnore -->
 <!--
   - /*
   -  * Copyright (c) 2020, salesforce.com, inc.
diff --git a/force-app/main/default/lwc/attendance/__tests__/attendance.test.js b/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
index 301f3fb7d..d835cf8f9 100644
--- a/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
+++ b/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
@@ -48,7 +48,7 @@ describe("c-attendance", () => {
         element.recordId = "a06630000074mVnAAI";
     });
 
-    it("shows multiple returned rows with perms", async () => {
+    it("shows multiple returned rows with perms", () => {
         document.body.appendChild(element);
         wiredSessionAdapter.emit(mockWiredSession);
         fieldSetAdapter.emit(mockGetFieldSet);
@@ -56,7 +56,7 @@ describe("c-attendance", () => {
         wiredPermissionsAdapter.emit(mockWiredPermissions);
         wiredCurrentPageReference.emit(mockCurrentPageReference);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
                 "c-attendance-row"
             );
@@ -64,7 +64,7 @@ describe("c-attendance", () => {
             global.isAccessible(element);
         });
     });
-    it("shows empty state and no buttons when zero returned rows with no perms", async () => {
+    it("shows empty state and no buttons when zero returned rows with no perms", () => {
         document.body.appendChild(element);
         wiredSessionAdapter.emit(mockWiredSession);
         fieldSetAdapter.emit(mockGetFieldSet);
@@ -72,7 +72,7 @@ describe("c-attendance", () => {
         wiredPermissionsAdapter.emit(mockWiredNoPermissions);
         wiredCurrentPageReference.emit(mockCurrentPageReference);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
                 "c-attendance-row"
             );
@@ -92,7 +92,7 @@ describe("c-attendance", () => {
             global.isAccessible(element);
         });
     });
-    it("shows empty state and no buttons when zero returned rows with perms", async () => {
+    it("shows empty state and no buttons when zero returned rows with perms", () => {
         document.body.appendChild(element);
         wiredSessionAdapter.emit(mockWiredSession);
         fieldSetAdapter.emit(mockGetFieldSet);
@@ -100,7 +100,7 @@ describe("c-attendance", () => {
         wiredPermissionsAdapter.emit(mockWiredPermissions);
         wiredCurrentPageReference.emit(mockCurrentPageReference);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
                 "c-attendance-row"
             );
@@ -120,7 +120,7 @@ describe("c-attendance", () => {
             global.isAccessible(element);
         });
     });
-    it("shows perms error when rows are returned with no perms", async () => {
+    it("shows perms error when rows are returned with no perms", () => {
         document.body.appendChild(element);
         wiredSessionAdapter.emit(mockWiredSession);
         fieldSetAdapter.emit(mockGetFieldSet);
@@ -128,7 +128,7 @@ describe("c-attendance", () => {
         wiredPermissionsAdapter.emit(mockWiredNoPermissions);
         wiredCurrentPageReference.emit(mockCurrentPageReference);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
                 "c-attendance-row"
             );
diff --git a/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js b/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js
index 2f6b6ccbf..fad18c4a6 100644
--- a/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js
+++ b/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js
@@ -36,12 +36,12 @@ describe("c-formattedField", () => {
         };
     });
 
-    it("displays a lightning formatted text field when a value is present", async () => {
+    it("displays a lightning formatted text field when a value is present", () => {
         element.field = mockField;
         element.record = mockRecord;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedText = element.shadowRoot.querySelector(
                 "lightning-formatted-text"
             );
@@ -49,14 +49,14 @@ describe("c-formattedField", () => {
             global.isAccessible(element);
         });
     });
-    it("displays a lightning formatted text field when a related value is present", async () => {
+    it("displays a lightning formatted text field when a related value is present", () => {
         mockField.path = "Account.Name";
         mockRecord.Account = { Name: "Hola" };
         element.field = mockField;
         element.record = mockRecord;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedText = element.shadowRoot.querySelector(
                 "lightning-formatted-text"
             );
@@ -64,13 +64,13 @@ describe("c-formattedField", () => {
             global.isAccessible(element);
         });
     });
-    it("displays a lightning formatted text field when a related value is not present", async () => {
+    it("displays a lightning formatted text field when a related value is not present", () => {
         mockField.path = "Account.Name";
         element.field = mockField;
         element.record = mockRecord;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedText = element.shadowRoot.querySelector(
                 "lightning-formatted-text"
             );
@@ -78,14 +78,14 @@ describe("c-formattedField", () => {
             global.isAccessible(element);
         });
     });
-    it("displays a lightning formatted text field when a value is missing", async () => {
+    it("displays a lightning formatted text field when a value is missing", () => {
         mockField.apiName = "Title";
 
         element.field = mockField;
         element.record = mockRecord;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedText = element.shadowRoot.querySelector(
                 "lightning-formatted-text"
             );
@@ -94,11 +94,11 @@ describe("c-formattedField", () => {
         });
     });
 
-    it("displays a lightning formatted text field when the record is missing", async () => {
+    it("displays a lightning formatted text field when the record is missing", () => {
         element.field = mockField;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedText = element.shadowRoot.querySelector(
                 "lightning-formatted-text"
             );
@@ -107,7 +107,7 @@ describe("c-formattedField", () => {
         });
     });
 
-    it("displays a lightning formatted date time field when type is date", async () => {
+    it("displays a lightning formatted date time field when type is date", () => {
         let today = new Date();
         mockField.type = "DATE";
         mockField.apiName = "Today";
@@ -117,7 +117,7 @@ describe("c-formattedField", () => {
         element.record = mockRecord;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedDateTime = element.shadowRoot.querySelector(
                 "lightning-formatted-date-time"
             );
@@ -126,7 +126,7 @@ describe("c-formattedField", () => {
         });
     });
 
-    it("displays a lightning formatted date time field when type is date-time", async () => {
+    it("displays a lightning formatted date time field when type is date-time", () => {
         let today = new Date();
         mockField.type = "DATETIME";
         mockField.apiName = "Today";
@@ -136,7 +136,7 @@ describe("c-formattedField", () => {
         element.record = mockRecord;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedDateTime = element.shadowRoot.querySelector(
                 "lightning-formatted-date-time"
             );
@@ -145,7 +145,7 @@ describe("c-formattedField", () => {
         });
     });
 
-    it("displays a lightning formatted time field when type is time", async () => {
+    it("displays a lightning formatted time field when type is time", () => {
         let time = 1800000;
         mockField.type = "TIME";
         mockField.apiName = "Time";
@@ -155,7 +155,7 @@ describe("c-formattedField", () => {
         element.record = mockRecord;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningFormattedTime = element.shadowRoot.querySelector(
                 "lightning-formatted-time"
             );
diff --git a/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js b/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js
index cbc739028..c1debc5cc 100644
--- a/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js
+++ b/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js
@@ -29,10 +29,10 @@ describe("c-listViewSelector", () => {
         getListUiAdapter.emit(mockGetListViews);
     });
 
-    it("renders first list view returned as checked", async () => {
+    it("renders first list view returned as checked", () => {
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const lightningMenuItem = element.shadowRoot.querySelector(
                 "lightning-menu-item"
             );
diff --git a/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js b/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js
index 9df5265fd..a8d2d5ed6 100644
--- a/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js
+++ b/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js
@@ -12,13 +12,16 @@ describe("c-loading-stencil", () => {
         });
     });
 
-    it("displays the stencils with default levels", async () => {
+    it("displays the stencils with default levels", () => {
         document.body.appendChild(element);
 
         let stencils = element.shadowRoot.querySelectorAll(".stencil");
-        expect(stencils.length).toBe(element.levels.length);
-        expect(element.levels.length).toBeGreaterThan(0);
-        global.isAccessible(element);
+
+        return global.flushPromises().then(() => {
+            expect(stencils.length).toBe(element.levels.length);
+            expect(element.levels.length).toBeGreaterThan(0);
+            global.isAccessible(element);
+        });
     });
 
     it("displays the stencils with provided levels", () => {
diff --git a/force-app/main/default/lwc/modal/__tests__/modal.test.js b/force-app/main/default/lwc/modal/__tests__/modal.test.js
index eef84d549..a8af31f07 100644
--- a/force-app/main/default/lwc/modal/__tests__/modal.test.js
+++ b/force-app/main/default/lwc/modal/__tests__/modal.test.js
@@ -38,7 +38,7 @@ describe("c-modal", () => {
         const HEADER_STRING = "Modal Header";
         element.header = HEADER_STRING;
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             //verify that the header is displayed
             const headerContainer = element.shadowRoot.querySelector(".header-string");
             expect(headerContainer).not.toBeNull();
@@ -78,7 +78,7 @@ describe("c-modal", () => {
         element.addEventListener("dialogclose", handler);
         buttonContainer.click();
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const headerContainer = element.shadowRoot.querySelector(".header-string");
             expect(headerContainer.textContent).toBe(HEADER_STRING);
             expect(handler).toHaveBeenCalled();
@@ -91,7 +91,7 @@ describe("c-modal", () => {
         const HEADER_STRING = "Modal Header";
         element.header = HEADER_STRING;
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const headerContainer = element.shadowRoot.querySelector(".header-string");
             expect(headerContainer.textContent).toBe(HEADER_STRING);
 
@@ -108,12 +108,14 @@ describe("c-modal", () => {
     });
 
     // This jest test below validates that the component is accessible
-    it("is accessible", async () => {
+    it("is accessible", () => {
         const h2Container = element.shadowRoot.querySelector("h2");
         let slot = document.createElement("span");
         slot.textContent = "Test Accessibility";
         h2Container.attachShadow({ mode: "open" }).appendChild(slot.cloneNode(true));
-        // assert that DOM is accessible (using extended preset-rule)
-        global.isAccessible(element);
+
+        return global.flushPromises().then(() => {
+            global.isAccessible(element);
+        });
     });
 });
diff --git a/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js b/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
index 9777c63f1..c207c3f6e 100644
--- a/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
+++ b/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
@@ -63,7 +63,7 @@ describe("c-new-program-engagement with a known contact", () => {
 
         element.showModal();
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const inputFields = element.shadowRoot.querySelectorAll(
                 "lightning-input-field"
             );
@@ -105,7 +105,7 @@ describe("c-new-program-engagement with a known contact", () => {
                     new CustomEvent("success", { detail: recordId })
                 );
             })
-            .then(async () => {
+            .then(() => {
                 expect(handler).toHaveBeenCalled();
                 global.isAccessible(element);
             });
@@ -129,15 +129,9 @@ describe("c-new-program-engagement with a known contact", () => {
                 expect(cancelButton.label).toBe(cancel);
                 cancelButton.click();
             })
-            .then(async () => {
+            .then(() => {
                 expect(handler).toHaveBeenCalled();
                 global.isAccessible(element);
             });
     });
-
-    it("is accessible", async () => {
-        document.body.appendChild(element);
-        // assert that DOM is accessible (using extended preset-rule)
-        global.isAccessible(element);
-    });
 });
diff --git a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
index 61506581f..194a0e618 100644
--- a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
+++ b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
@@ -23,41 +23,48 @@ describe("c-scoped-notification", () => {
         });
     });
 
-    it("displays the info theme by default", async () => {
+    it("displays the info theme by default", () => {
         document.body.appendChild(element);
-        const icon = element.shadowRoot.querySelector("lightning-icon");
-        const themeDiv = element.shadowRoot.querySelector(".slds-theme_info");
-        const titlePara = element.shadowRoot.querySelector("p");
 
-        expect(themeDiv).toBeDefined();
-        expect(icon.iconName).toBe("utility:info");
-        expect(titlePara.textContent).toEqual("");
-        global.isAccessible(element);
+        return global.flushPromises().then(() => {
+            const icon = element.shadowRoot.querySelector("lightning-icon");
+            const themeDiv = element.shadowRoot.querySelector(".slds-theme_info");
+            const titlePara = element.shadowRoot.querySelector("p");
+
+            expect(themeDiv).toBeDefined();
+            expect(icon.iconName).toBe("utility:info");
+            expect(titlePara.textContent).toEqual("");
+            global.isAccessible(element);
+        });
     });
 
-    it("displays the light info theme", async () => {
+    it("displays the light info theme", () => {
         element.title = TITLE;
         element.theme = "light";
         document.body.appendChild(element);
-        const icon = element.shadowRoot.querySelector("lightning-icon");
-        const themeDiv = element.shadowRoot.querySelector(
-            "slds-scoped-notification_light"
-        );
-        const titlePara = element.shadowRoot.querySelector("p");
 
-        expect(themeDiv).toBeDefined();
-        expect(icon.iconName).toBe("utility:info");
-        expect(titlePara.textContent).toEqual(TITLE);
-        global.isAccessible(element);
+        return global.flushPromises().then(() => {
+            const icon = element.shadowRoot.querySelector("lightning-icon");
+            const themeDiv = element.shadowRoot.querySelector(
+                "slds-scoped-notification_light"
+            );
+            const titlePara = element.shadowRoot.querySelector("p");
+
+            expect(themeDiv).toBeDefined();
+            expect(icon.iconName).toBe("utility:info");
+            expect(titlePara.textContent).toEqual(TITLE);
+            global.isAccessible(element);
+        });
     });
 
     it("displays the theme provided", () => {
         const themes = ["warning", "info", "success", "error"];
         element.title = TITLE;
 
-        themes.forEach(async theme => {
+        themes.forEach(theme => {
             element.theme = theme;
             document.body.appendChild(element);
+
             const icon = element.shadowRoot.querySelector("lightning-icon");
             const themeDiv = element.shadowRoot.querySelector(`.slds-theme_${theme}`);
             const titlePara = element.shadowRoot.querySelector("p");
@@ -65,7 +72,8 @@ describe("c-scoped-notification", () => {
             expect(themeDiv).toBeDefined();
             expect(icon.iconName).toBe(`utility:${theme}`);
             expect(titlePara.textContent).toEqual(TITLE);
-            global.isAccessible(element);
+            // TODO: appears to be an issue running this in a 4 loop
+            // global.isAccessible(element);
         });
     });
 });
diff --git a/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js b/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
index 48007260b..a0c3cb6ee 100644
--- a/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
+++ b/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
@@ -24,13 +24,14 @@ describe("c-service-schedule-creator", () => {
     it("element is accessible", () => {
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const spinner = element.shadowRoot.querySelector("lightning-spinner");
 
-            expect(spinner).not.toBeNull();
-
-            // TODO: Validate accessibility when each step is loads.
-            global.isAccessible(element);
+            return global.flushPromises().then(() => {
+                expect(spinner).not.toBeNull();
+                // TODO: Validate accessibility when each step is loads.
+                global.isAccessible(element);
+            });
         });
     });
 });
diff --git a/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js b/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js
index 6d5ea3df7..ecaf5f74f 100644
--- a/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js
+++ b/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js
@@ -25,7 +25,7 @@ describe("c-service-schedule-creator", () => {
         element.isCommunity = false;
         document.body.appendChild(element);
 
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const modal = element.shadowRoot.querySelector("c-modal");
 
             // Modal will only display with a spinner loaded
@@ -40,7 +40,7 @@ describe("c-service-schedule-creator", () => {
     it("modal does not appear in experience-cloud context and element is accessible", () => {
         element.isCommunity = true;
         document.body.appendChild(element);
-        return global.flushPromises().then(async () => {
+        return global.flushPromises().then(() => {
             const modal = element.shadowRoot.querySelector("c-modal");
             expect(modal).toBeNull();
 
diff --git a/jest.setup.js b/jest.setup.js
index f96ada462..21105f8a3 100644
--- a/jest.setup.js
+++ b/jest.setup.js
@@ -5,6 +5,7 @@ registerSa11yMatcher();
 
 global.flushPromises = () => new Promise(resolve => setImmediate(resolve));
 
+// assert that DOM is accessible (using extended preset-rule)
 global.isAccessible = async element => {
     try {
         await expect(element).toBeAccessible();

From 1dcd5fad9ffdbd5e2c02f62158fb6b279c2e81ff Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Sat, 25 Sep 2021 11:17:20 -0700
Subject: [PATCH 02/11] Add the index to the service delivery records.

---
 force-app/main/default/lwc/attendance/attendance.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/force-app/main/default/lwc/attendance/attendance.js b/force-app/main/default/lwc/attendance/attendance.js
index 523dec951..500371a53 100644
--- a/force-app/main/default/lwc/attendance/attendance.js
+++ b/force-app/main/default/lwc/attendance/attendance.js
@@ -157,7 +157,10 @@ export default class Attendance extends NavigationMixin(LightningElement) {
         }
 
         if (result.data) {
-            this.serviceDeliveries = [...result.data];
+            this.serviceDeliveries = result.data.map((item, index) => ({
+                index,
+                ...item,
+            }));
             this.serviceDeliveries.sort((a, b) => {
                 return getChildObjectByName(a, "Contact__r").Name >
                     getChildObjectByName(b, "Contact__r").Name

From 282c63e9fc2fcaa3c90427d2f261bb88b9975dca Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Mon, 27 Sep 2021 08:49:15 -0700
Subject: [PATCH 03/11] Add jest tests for accordion section and move aria text
 from icon to the button

---
 .../__tests__/accordionSection.test.js        | 113 ++++++++++++++++++
 .../accordionSection/accordionSection.html    |   2 +-
 2 files changed, 114 insertions(+), 1 deletion(-)
 create mode 100644 force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js

diff --git a/force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js b/force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js
new file mode 100644
index 000000000..ef19fe6a1
--- /dev/null
+++ b/force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js
@@ -0,0 +1,113 @@
+/*
+ *
+ *  * Copyright (c) 2020, salesforce.com, inc.
+ *  * All rights reserved.
+ *  * SPDX-License-Identifier: BSD-3-Clause
+ *  * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/BSD-3-Clause
+ *
+ */
+
+import { createElement } from "lwc";
+import AccordionSection from "c/accordionSection";
+
+describe("c-accordionSection", () => {
+    let element;
+
+    afterEach(global.clearDOM);
+
+    beforeEach(() => {
+        element = createElement("c-accordionSection", {
+            is: AccordionSection,
+        });
+    });
+
+    it("is closed by default", () => {
+        document.body.appendChild(element);
+
+        return global.flushPromises().then(() => {
+            const button = element.shadowRoot.querySelector("button");
+            expect(button.ariaExpanded).toBe("false");
+            global.isAccessible(element);
+        });
+    });
+
+    it("is open when open current section is true", () => {
+        element.openCurrentSection = true;
+        document.body.appendChild(element);
+
+        return global.flushPromises().then(() => {
+            const button = element.shadowRoot.querySelector("button");
+            expect(button.ariaExpanded).toBe("true");
+            global.isAccessible(element);
+        });
+    });
+
+    it("is accessible with background shaded", () => {
+        element.shadeOnOpen = "#EFEFEF";
+        element.openCurrentSection = true;
+        document.body.appendChild(element);
+
+        return global.flushPromises().then(() => {
+            // TODO: see if we can verify background color
+            global.isAccessible(element);
+        });
+    });
+
+    it("opens", () => {
+        document.body.appendChild(element);
+        element.open();
+
+        return global.flushPromises().then(() => {
+            const button = element.shadowRoot.querySelector("button");
+            expect(button.ariaExpanded).toBe("true");
+            global.isAccessible(element);
+        });
+    });
+
+    it("closes", () => {
+        element.openCurrentSection = true;
+        document.body.appendChild(element);
+        element.close();
+
+        return global.flushPromises().then(() => {
+            const button = element.shadowRoot.querySelector("button");
+            expect(button.ariaExpanded).toBe("false");
+            global.isAccessible(element);
+        });
+    });
+
+    it("does not open when prevent toggle is set", () => {
+        element.preventToggle = true;
+        document.body.appendChild(element);
+        element.open();
+
+        return global.flushPromises().then(() => {
+            const button = element.shadowRoot.querySelector("button");
+            expect(button.ariaExpanded).toBe("false");
+            global.isAccessible(element);
+        });
+    });
+
+    it("override open overrides prevent toggle and opens", () => {
+        element.preventToggle = true;
+        document.body.appendChild(element);
+        element.overrideOpen();
+
+        return global.flushPromises().then(() => {
+            const button = element.shadowRoot.querySelector("button");
+            expect(button.ariaExpanded).toBe("true");
+            global.isAccessible(element);
+        });
+    });
+
+    it("opens on button click", () => {
+        document.body.appendChild(element);
+        const button = element.shadowRoot.querySelector("button");
+        button.click();
+
+        return global.flushPromises().then(() => {
+            expect(button.ariaExpanded).toBe("true");
+            global.isAccessible(element);
+        });
+    });
+});
diff --git a/force-app/main/default/lwc/accordionSection/accordionSection.html b/force-app/main/default/lwc/accordionSection/accordionSection.html
index 8adc02217..9706a17d3 100644
--- a/force-app/main/default/lwc/accordionSection/accordionSection.html
+++ b/force-app/main/default/lwc/accordionSection/accordionSection.html
@@ -22,11 +22,11 @@
                         onclick={handleToggle}
                         aria-controls="accordion-details"
                         aria-expanded={isOpen}
+                        aria-label={toggleInstructions}
                         class="slds-align-middle slds-text-link_reset slds-button"
                     >
                         <lightning-icon
                             icon-name={toggleIcon}
-                            alternative-text={toggleInstructions}
                             size="x-small"
                         ></lightning-icon>
                     </button>

From e2fdb28b54a0984c4c4512ebceb1fba6854b540b Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Mon, 27 Sep 2021 10:02:06 -0700
Subject: [PATCH 04/11] Return the a11y check to ensure we wait for it to
 finish.

---
 .../__tests__/accordionSection.test.js           | 16 ++++++++--------
 .../lwc/attendance/__tests__/attendance.test.js  |  8 ++++----
 .../__tests__/formattedField.test.js             | 16 ++++++++--------
 .../__tests__/listViewSelector.test.js           |  2 +-
 .../__tests__/loadingStencil.test.js             |  2 +-
 .../default/lwc/modal/__tests__/modal.test.js    | 10 +++-------
 .../__tests__/newProgramEngagement.test.js       | 14 +++++++-------
 .../__tests__/scopedNotification.test.js         |  6 +++---
 .../__tests__/serviceScheduleCreator.test.js     |  2 +-
 .../serviceScheduleCreatorWrapper.test.js        |  4 ++--
 10 files changed, 38 insertions(+), 42 deletions(-)

diff --git a/force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js b/force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js
index ef19fe6a1..4f66a2b93 100644
--- a/force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js
+++ b/force-app/main/default/lwc/accordionSection/__tests__/accordionSection.test.js
@@ -27,7 +27,7 @@ describe("c-accordionSection", () => {
         return global.flushPromises().then(() => {
             const button = element.shadowRoot.querySelector("button");
             expect(button.ariaExpanded).toBe("false");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -38,7 +38,7 @@ describe("c-accordionSection", () => {
         return global.flushPromises().then(() => {
             const button = element.shadowRoot.querySelector("button");
             expect(button.ariaExpanded).toBe("true");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -49,7 +49,7 @@ describe("c-accordionSection", () => {
 
         return global.flushPromises().then(() => {
             // TODO: see if we can verify background color
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -60,7 +60,7 @@ describe("c-accordionSection", () => {
         return global.flushPromises().then(() => {
             const button = element.shadowRoot.querySelector("button");
             expect(button.ariaExpanded).toBe("true");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -72,7 +72,7 @@ describe("c-accordionSection", () => {
         return global.flushPromises().then(() => {
             const button = element.shadowRoot.querySelector("button");
             expect(button.ariaExpanded).toBe("false");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -84,7 +84,7 @@ describe("c-accordionSection", () => {
         return global.flushPromises().then(() => {
             const button = element.shadowRoot.querySelector("button");
             expect(button.ariaExpanded).toBe("false");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -96,7 +96,7 @@ describe("c-accordionSection", () => {
         return global.flushPromises().then(() => {
             const button = element.shadowRoot.querySelector("button");
             expect(button.ariaExpanded).toBe("true");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -107,7 +107,7 @@ describe("c-accordionSection", () => {
 
         return global.flushPromises().then(() => {
             expect(button.ariaExpanded).toBe("true");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 });
diff --git a/force-app/main/default/lwc/attendance/__tests__/attendance.test.js b/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
index d835cf8f9..7d2284b02 100644
--- a/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
+++ b/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
@@ -61,7 +61,7 @@ describe("c-attendance", () => {
                 "c-attendance-row"
             );
             expect(attendanceRows).toHaveLength(mockGenerateRoster.length);
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
     it("shows empty state and no buttons when zero returned rows with no perms", () => {
@@ -89,7 +89,7 @@ describe("c-attendance", () => {
             const buttons = element.shadowRoot.querySelectorAll("lightning-button");
             expect(buttons).toHaveLength(0);
 
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
     it("shows empty state and no buttons when zero returned rows with perms", () => {
@@ -117,7 +117,7 @@ describe("c-attendance", () => {
             const buttons = element.shadowRoot.querySelectorAll("lightning-button");
             expect(buttons).toHaveLength(0);
 
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
     it("shows perms error when rows are returned with no perms", () => {
@@ -142,7 +142,7 @@ describe("c-attendance", () => {
             );
             expect(scopedNotification).toHaveLength(1);
 
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 });
diff --git a/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js b/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js
index fad18c4a6..e6568efb9 100644
--- a/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js
+++ b/force-app/main/default/lwc/formattedField/__tests__/formattedField.test.js
@@ -46,7 +46,7 @@ describe("c-formattedField", () => {
                 "lightning-formatted-text"
             );
             expect(lightningFormattedText.value).toBe(mockRecord.Name);
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
     it("displays a lightning formatted text field when a related value is present", () => {
@@ -61,7 +61,7 @@ describe("c-formattedField", () => {
                 "lightning-formatted-text"
             );
             expect(lightningFormattedText.value).toBe(mockRecord.Account.Name);
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
     it("displays a lightning formatted text field when a related value is not present", () => {
@@ -75,7 +75,7 @@ describe("c-formattedField", () => {
                 "lightning-formatted-text"
             );
             expect(lightningFormattedText.value).toBeUndefined();
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
     it("displays a lightning formatted text field when a value is missing", () => {
@@ -90,7 +90,7 @@ describe("c-formattedField", () => {
                 "lightning-formatted-text"
             );
             expect(lightningFormattedText.value).toBeUndefined();
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -103,7 +103,7 @@ describe("c-formattedField", () => {
                 "lightning-formatted-text"
             );
             expect(lightningFormattedText.value).toBeUndefined();
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -122,7 +122,7 @@ describe("c-formattedField", () => {
                 "lightning-formatted-date-time"
             );
             expect(lightningFormattedDateTime.value).toBe(today);
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -141,7 +141,7 @@ describe("c-formattedField", () => {
                 "lightning-formatted-date-time"
             );
             expect(lightningFormattedDateTime.value).toBe(today);
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -162,7 +162,7 @@ describe("c-formattedField", () => {
             // Javascript will convert the number into time format, relying on TimeZones
             // validating the semi colons vs the actual time.
             expect(lightningFormattedTime.value.includes(":")).toBeTruthy();
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 });
diff --git a/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js b/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js
index c1debc5cc..7efaae629 100644
--- a/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js
+++ b/force-app/main/default/lwc/listViewSelector/__tests__/listViewSelector.test.js
@@ -38,7 +38,7 @@ describe("c-listViewSelector", () => {
             );
             expect(lightningMenuItem.label).toBe("All");
             expect(lightningMenuItem.checked).toBeTruthy();
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 });
diff --git a/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js b/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js
index a8d2d5ed6..8627742e4 100644
--- a/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js
+++ b/force-app/main/default/lwc/loadingStencil/__tests__/loadingStencil.test.js
@@ -20,7 +20,7 @@ describe("c-loading-stencil", () => {
         return global.flushPromises().then(() => {
             expect(stencils.length).toBe(element.levels.length);
             expect(element.levels.length).toBeGreaterThan(0);
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
diff --git a/force-app/main/default/lwc/modal/__tests__/modal.test.js b/force-app/main/default/lwc/modal/__tests__/modal.test.js
index a8af31f07..88e5aec4f 100644
--- a/force-app/main/default/lwc/modal/__tests__/modal.test.js
+++ b/force-app/main/default/lwc/modal/__tests__/modal.test.js
@@ -47,7 +47,7 @@ describe("c-modal", () => {
             //verify that the header slot is not displayed because the header is present
             const slotContainer = element.shadowRoot.querySelector(".header-slot");
             expect(slotContainer).toBeNull();
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -82,7 +82,7 @@ describe("c-modal", () => {
             const headerContainer = element.shadowRoot.querySelector(".header-string");
             expect(headerContainer.textContent).toBe(HEADER_STRING);
             expect(handler).toHaveBeenCalled();
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -99,8 +99,6 @@ describe("c-modal", () => {
             element.show();
             //verify that there is no CSS class present
             expect(modalContainer.classList.value).toBe("");
-            global.isAccessible(element);
-
             element.hide();
             //verify that the CSS class exists
             expect(modalContainer.classList.value).toBe(MODAL_HIDDEN);
@@ -114,8 +112,6 @@ describe("c-modal", () => {
         slot.textContent = "Test Accessibility";
         h2Container.attachShadow({ mode: "open" }).appendChild(slot.cloneNode(true));
 
-        return global.flushPromises().then(() => {
-            global.isAccessible(element);
-        });
+        return global.isAccessible(element);
     });
 });
diff --git a/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js b/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
index c207c3f6e..9fc5fd89d 100644
--- a/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
+++ b/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
@@ -78,7 +78,7 @@ describe("c-new-program-engagement with a known contact", () => {
                 index++;
             });
 
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -107,20 +107,21 @@ describe("c-new-program-engagement with a known contact", () => {
             })
             .then(() => {
                 expect(handler).toHaveBeenCalled();
-                global.isAccessible(element);
+                return global.isAccessible(element);
             });
     });
 
     it("closes the modal when the close button is clicked", () => {
-        document.body.appendChild(element);
-        element.newContactMode = false;
-        element.showModal();
-
         const handler = jest.fn();
         element.addEventListener("cancel", handler);
+        element.newContactMode = false;
+        document.body.appendChild(element);
 
         return global
             .flushPromises()
+            .then(() => {
+                element.showModal();
+            })
             .then(() => {
                 let cancelButton = element.shadowRoot.querySelectorAll(
                     "lightning-button"
@@ -131,7 +132,6 @@ describe("c-new-program-engagement with a known contact", () => {
             })
             .then(() => {
                 expect(handler).toHaveBeenCalled();
-                global.isAccessible(element);
             });
     });
 });
diff --git a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
index 194a0e618..5f4b0ab9b 100644
--- a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
+++ b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
@@ -34,7 +34,7 @@ describe("c-scoped-notification", () => {
             expect(themeDiv).toBeDefined();
             expect(icon.iconName).toBe("utility:info");
             expect(titlePara.textContent).toEqual("");
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -53,7 +53,7 @@ describe("c-scoped-notification", () => {
             expect(themeDiv).toBeDefined();
             expect(icon.iconName).toBe("utility:info");
             expect(titlePara.textContent).toEqual(TITLE);
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -73,7 +73,7 @@ describe("c-scoped-notification", () => {
             expect(icon.iconName).toBe(`utility:${theme}`);
             expect(titlePara.textContent).toEqual(TITLE);
             // TODO: appears to be an issue running this in a 4 loop
-            // global.isAccessible(element);
+            // return global.isAccessible(element);
         });
     });
 });
diff --git a/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js b/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
index a0c3cb6ee..a5a34300a 100644
--- a/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
+++ b/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
@@ -30,7 +30,7 @@ describe("c-service-schedule-creator", () => {
             return global.flushPromises().then(() => {
                 expect(spinner).not.toBeNull();
                 // TODO: Validate accessibility when each step is loads.
-                global.isAccessible(element);
+                return global.isAccessible(element);
             });
         });
     });
diff --git a/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js b/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js
index ecaf5f74f..b4fba5c59 100644
--- a/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js
+++ b/force-app/main/default/lwc/serviceScheduleCreatorWrapper/__tests__/serviceScheduleCreatorWrapper.test.js
@@ -33,7 +33,7 @@ describe("c-service-schedule-creator", () => {
             modal.dispatchEvent(new CustomEvent("dialogclose"));
 
             // TODO: Validate accessibility when each step is loads.
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 
@@ -54,7 +54,7 @@ describe("c-service-schedule-creator", () => {
             slot.textContent = "Test Accessibility";
             h2Container.attachShadow({ mode: "open" }).appendChild(slot.cloneNode(true));
             // assert that DOM is accessible (using extended preset-rule)
-            global.isAccessible(element);
+            return global.isAccessible(element);
         });
     });
 });

From ed7b902f7376d1f8a8933a173cdadbf25652d7f7 Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Mon, 27 Sep 2021 10:13:43 -0700
Subject: [PATCH 05/11] Verify each theme and a11y for each.

---
 .../__tests__/scopedNotification.test.js      | 42 ++++++++++++-------
 1 file changed, 28 insertions(+), 14 deletions(-)

diff --git a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
index 5f4b0ab9b..f2fedb389 100644
--- a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
+++ b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
@@ -57,23 +57,37 @@ describe("c-scoped-notification", () => {
         });
     });
 
-    it("displays the theme provided", () => {
-        const themes = ["warning", "info", "success", "error"];
+    it("displays the success theme", () => {
+        const theme = "success";
         element.title = TITLE;
 
-        themes.forEach(theme => {
-            element.theme = theme;
-            document.body.appendChild(element);
+        element.theme = theme;
+        document.body.appendChild(element);
 
-            const icon = element.shadowRoot.querySelector("lightning-icon");
-            const themeDiv = element.shadowRoot.querySelector(`.slds-theme_${theme}`);
-            const titlePara = element.shadowRoot.querySelector("p");
+        const icon = element.shadowRoot.querySelector("lightning-icon");
+        const themeDiv = element.shadowRoot.querySelector(`.slds-theme_${theme}`);
+        const titlePara = element.shadowRoot.querySelector("p");
 
-            expect(themeDiv).toBeDefined();
-            expect(icon.iconName).toBe(`utility:${theme}`);
-            expect(titlePara.textContent).toEqual(TITLE);
-            // TODO: appears to be an issue running this in a 4 loop
-            // return global.isAccessible(element);
-        });
+        expect(themeDiv).toBeDefined();
+        expect(icon.iconName).toBe(`utility:${theme}`);
+        expect(titlePara.textContent).toEqual(TITLE);
+        return global.isAccessible();
+    });
+
+    it("displays the error theme", () => {
+        const theme = "error";
+        element.title = TITLE;
+
+        element.theme = theme;
+        document.body.appendChild(element);
+
+        const icon = element.shadowRoot.querySelector("lightning-icon");
+        const themeDiv = element.shadowRoot.querySelector(`.slds-theme_${theme}`);
+        const titlePara = element.shadowRoot.querySelector("p");
+
+        expect(themeDiv).toBeDefined();
+        expect(icon.iconName).toBe(`utility:${theme}`);
+        expect(titlePara.textContent).toEqual(TITLE);
+        return global.isAccessible();
     });
 });

From 54e0395f530e8cb7531e1dd2d32b845af8c910a6 Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Mon, 27 Sep 2021 10:15:54 -0700
Subject: [PATCH 06/11] Remove inner flush, added by mistake.

---
 .../__tests__/serviceScheduleCreator.test.js              | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js b/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
index a5a34300a..1918e0b90 100644
--- a/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
+++ b/force-app/main/default/lwc/serviceScheduleCreator/__tests__/serviceScheduleCreator.test.js
@@ -26,12 +26,8 @@ describe("c-service-schedule-creator", () => {
 
         return global.flushPromises().then(() => {
             const spinner = element.shadowRoot.querySelector("lightning-spinner");
-
-            return global.flushPromises().then(() => {
-                expect(spinner).not.toBeNull();
-                // TODO: Validate accessibility when each step is loads.
-                return global.isAccessible(element);
-            });
+            expect(spinner).not.toBeNull();
+            return global.isAccessible(element);
         });
     });
 });

From 09a2c23548fa6ea95486017c613a62e228dcfe0a Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Mon, 27 Sep 2021 10:24:45 -0700
Subject: [PATCH 07/11] Remove a11y when modal is closed, the component lives
 within the modal.

---
 .../newProgramEngagement/__tests__/newProgramEngagement.test.js  | 1 -
 1 file changed, 1 deletion(-)

diff --git a/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js b/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
index 9fc5fd89d..c718f3c44 100644
--- a/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
+++ b/force-app/main/default/lwc/newProgramEngagement/__tests__/newProgramEngagement.test.js
@@ -107,7 +107,6 @@ describe("c-new-program-engagement with a known contact", () => {
             })
             .then(() => {
                 expect(handler).toHaveBeenCalled();
-                return global.isAccessible(element);
             });
     });
 

From 4dd4ce37570e321d6d32aa1901e77935d154d49a Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Mon, 27 Sep 2021 11:37:24 -0700
Subject: [PATCH 08/11] Fix new tests.

---
 .../__tests__/scopedNotification.test.js      | 40 +++++++++----------
 jest.setup.js                                 |  6 +--
 2 files changed, 21 insertions(+), 25 deletions(-)

diff --git a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
index f2fedb389..b8deacecf 100644
--- a/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
+++ b/force-app/main/default/lwc/scopedNotification/__tests__/scopedNotification.test.js
@@ -58,36 +58,36 @@ describe("c-scoped-notification", () => {
     });
 
     it("displays the success theme", () => {
-        const theme = "success";
         element.title = TITLE;
-
-        element.theme = theme;
+        element.theme = "success";
         document.body.appendChild(element);
 
-        const icon = element.shadowRoot.querySelector("lightning-icon");
-        const themeDiv = element.shadowRoot.querySelector(`.slds-theme_${theme}`);
-        const titlePara = element.shadowRoot.querySelector("p");
+        return global.flushPromises().then(() => {
+            const icon = element.shadowRoot.querySelector("lightning-icon");
+            const themeDiv = element.shadowRoot.querySelector(".slds-theme_success");
+            const titlePara = element.shadowRoot.querySelector("p");
 
-        expect(themeDiv).toBeDefined();
-        expect(icon.iconName).toBe(`utility:${theme}`);
-        expect(titlePara.textContent).toEqual(TITLE);
-        return global.isAccessible();
+            expect(themeDiv).toBeDefined();
+            expect(icon.iconName).toBe("utility:success");
+            expect(titlePara.textContent).toEqual(TITLE);
+            return global.isAccessible(element);
+        });
     });
 
     it("displays the error theme", () => {
-        const theme = "error";
         element.title = TITLE;
-
-        element.theme = theme;
+        element.theme = "error";
         document.body.appendChild(element);
 
-        const icon = element.shadowRoot.querySelector("lightning-icon");
-        const themeDiv = element.shadowRoot.querySelector(`.slds-theme_${theme}`);
-        const titlePara = element.shadowRoot.querySelector("p");
+        return global.flushPromises().then(() => {
+            const icon = element.shadowRoot.querySelector("lightning-icon");
+            const themeDiv = element.shadowRoot.querySelector(".slds-theme_error");
+            const titlePara = element.shadowRoot.querySelector("p");
 
-        expect(themeDiv).toBeDefined();
-        expect(icon.iconName).toBe(`utility:${theme}`);
-        expect(titlePara.textContent).toEqual(TITLE);
-        return global.isAccessible();
+            expect(themeDiv).toBeDefined();
+            expect(icon.iconName).toBe("utility:error");
+            expect(titlePara.textContent).toEqual(TITLE);
+            return global.isAccessible(element);
+        });
     });
 });
diff --git a/jest.setup.js b/jest.setup.js
index 21105f8a3..375bce172 100644
--- a/jest.setup.js
+++ b/jest.setup.js
@@ -7,11 +7,7 @@ global.flushPromises = () => new Promise(resolve => setImmediate(resolve));
 
 // assert that DOM is accessible (using extended preset-rule)
 global.isAccessible = async element => {
-    try {
-        await expect(element).toBeAccessible();
-    } catch (e) {
-        console.log(JSON.stringify(e.message));
-    }
+    await expect(element).toBeAccessible();
 };
 
 global.clearDOM = () => {

From 3d47e9410902ad3f81422671207779d0a41e5f32 Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Thu, 30 Sep 2021 19:17:42 -0700
Subject: [PATCH 09/11] Update jest tests on attendance.

---
 .../default/lwc/__tests__/data/apexError.json |   6 +
 .../lwc/__tests__/lightning/uiRecordApi.js    |  36 ++
 .../attendance/__tests__/attendance.test.js   | 382 +++++++++++++++++-
 .../data/attendanceTabPageReference.json      |   5 +
 .../__tests__/data/currentPageReference.json  |   7 -
 .../__tests__/data/getFieldSet.json           |  12 +-
 .../data/serviceSessionPageReference.json     |   9 +
 .../main/default/lwc/attendance/attendance.js |  18 +-
 .../attendanceRow/__mocks__/attendanceRow.js  |   8 +
 jest.config.js                                |   7 +
 10 files changed, 463 insertions(+), 27 deletions(-)
 create mode 100644 force-app/main/default/lwc/__tests__/data/apexError.json
 create mode 100644 force-app/main/default/lwc/__tests__/lightning/uiRecordApi.js
 create mode 100644 force-app/main/default/lwc/attendance/__tests__/data/attendanceTabPageReference.json
 delete mode 100644 force-app/main/default/lwc/attendance/__tests__/data/currentPageReference.json
 create mode 100644 force-app/main/default/lwc/attendance/__tests__/data/serviceSessionPageReference.json

diff --git a/force-app/main/default/lwc/__tests__/data/apexError.json b/force-app/main/default/lwc/__tests__/data/apexError.json
new file mode 100644
index 000000000..fa3f27887
--- /dev/null
+++ b/force-app/main/default/lwc/__tests__/data/apexError.json
@@ -0,0 +1,6 @@
+{
+    "body": { "message": "An internal server error has occurred" },
+    "ok": false,
+    "status": 400,
+    "statusText": "Bad Request"
+}
diff --git a/force-app/main/default/lwc/__tests__/lightning/uiRecordApi.js b/force-app/main/default/lwc/__tests__/lightning/uiRecordApi.js
new file mode 100644
index 000000000..1db4367bd
--- /dev/null
+++ b/force-app/main/default/lwc/__tests__/lightning/uiRecordApi.js
@@ -0,0 +1,36 @@
+/*
+ * Copyright (c) 2018, salesforce.com, inc.
+ * All rights reserved.
+ * SPDX-License-Identifier: MIT
+ * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/MIT
+ */
+export const getRecord = jest.fn();
+export const getRecordCreateDefaults = jest.fn();
+export const updateRecord = jest.fn();
+export const createRecord = jest.fn().mockResolvedValue({});
+export const deleteRecord = jest.fn().mockResolvedValue();
+export const generateRecordInputForCreate = jest.fn();
+export const generateRecordInputForUpdate = jest.fn();
+export const createRecordInputFilteredByEditedFields = jest.fn();
+export const getRecordInput = jest.fn();
+export const refresh = jest.fn().mockResolvedValue();
+export const getRecordUi = jest.fn();
+export const getFieldValue = jest.fn((data, fieldReference) => {
+    if (data) {
+        const fields = fieldReference.fieldApiName.split(".");
+        if (data.result) {
+            const fieldData = fields.reduce((o, i) => o[i], data.result.fields);
+            if (fieldData && fieldData.value) {
+                return fieldData.value;
+            }
+        } else {
+            const fieldData = fields.reduce((o, i) => o[i], data.fields);
+            if (fieldData && fieldData.value) {
+                return fieldData.value;
+            }
+        }
+        return null;
+    }
+    return null;
+});
+export const getFieldDisplayValue = jest.fn();
diff --git a/force-app/main/default/lwc/attendance/__tests__/attendance.test.js b/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
index 7d2284b02..3719da2e7 100644
--- a/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
+++ b/force-app/main/default/lwc/attendance/__tests__/attendance.test.js
@@ -9,15 +9,26 @@
 
 import { createElement } from "lwc";
 import Attendance from "c/attendance";
-import { getRecord } from "lightning/uiRecordApi";
+import { getRecord, updateRecord } from "lightning/uiRecordApi";
 import { CurrentPageReference } from "lightning/navigation";
+import * as cUtil from "c/util";
 import generateRoster from "@salesforce/apex/AttendanceController.generateRoster";
 import getFieldSet from "@salesforce/apex/FieldSetController.getFieldSetForLWC";
 import checkFieldPermissions from "@salesforce/apex/AttendanceController.checkFieldPermissions";
+import upsertRows from "@salesforce/apex/AttendanceController.upsertServiceDeliveries";
 import {
     registerApexTestWireAdapter,
     registerLdsTestWireAdapter,
+    registerTestWireAdapter,
 } from "@salesforce/sfdx-lwc-jest";
+import SUBMIT_LABEL from "@salesforce/label/c.Submit";
+import UPDATE_LABEL from "@salesforce/label/c.Update";
+import PRINT_LABEL from "@salesforce/label/c.Print";
+import PRINTABLE_VIEW_LABEL from "@salesforce/label/c.Printable_View";
+import SESSION_STATUS_FIELD from "@salesforce/schema/ServiceSession__c.Status__c";
+import SAVE_LABEL from "@salesforce/label/c.Save";
+import CANCEL_LABEL from "@salesforce/label/c.Cancel";
+import QUANTITY_LABEL from "@salesforce/label/c.Quantity";
 
 const mockGenerateRoster = require("./data/generateRoster.json");
 const mockGenerateRosterEmpty = require("../../__tests__/data/emptyList.json");
@@ -25,17 +36,27 @@ const mockGetFieldSet = require("./data/getFieldSet.json");
 const mockWiredSession = require("./data/wiredPendingSession.json");
 const mockWiredPermissions = require("./data/wiredPermissions.json");
 const mockWiredNoPermissions = require("./data/wiredNoPermissions.json");
-const mockCurrentPageReference = require("./data/currentPageReference.json");
+const mockAttendanceTabPageReference = require("./data/attendanceTabPageReference.json");
+const mockServiceSessionPageReference = require("./data/serviceSessionPageReference.json");
+const mockApexError = require("../../__tests__/data/apexError.json");
 
 //Register the  wire adapters
 const generateRosterAdapter = registerApexTestWireAdapter(generateRoster);
 const fieldSetAdapter = registerApexTestWireAdapter(getFieldSet);
 const wiredSessionAdapter = registerLdsTestWireAdapter(getRecord);
 const wiredPermissionsAdapter = registerApexTestWireAdapter(checkFieldPermissions);
-const wiredCurrentPageReference = registerApexTestWireAdapter(CurrentPageReference);
+const currentPageReferenceAdapter = registerTestWireAdapter(CurrentPageReference);
 
 jest.mock("c/attendanceRow");
-
+jest.mock(
+    "@salesforce/apex/AttendanceController.upsertServiceDeliveries",
+    () => {
+        return {
+            default: jest.fn(),
+        };
+    },
+    { virtual: true }
+);
 describe("c-attendance", () => {
     let element;
 
@@ -46,6 +67,7 @@ describe("c-attendance", () => {
             is: Attendance,
         });
         element.recordId = "a06630000074mVnAAI";
+        currentPageReferenceAdapter.emit(mockAttendanceTabPageReference);
     });
 
     it("shows multiple returned rows with perms", () => {
@@ -54,7 +76,6 @@ describe("c-attendance", () => {
         fieldSetAdapter.emit(mockGetFieldSet);
         generateRosterAdapter.emit(mockGenerateRoster);
         wiredPermissionsAdapter.emit(mockWiredPermissions);
-        wiredCurrentPageReference.emit(mockCurrentPageReference);
 
         return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
@@ -70,7 +91,6 @@ describe("c-attendance", () => {
         fieldSetAdapter.emit(mockGetFieldSet);
         generateRosterAdapter.emit(mockGenerateRosterEmpty);
         wiredPermissionsAdapter.emit(mockWiredNoPermissions);
-        wiredCurrentPageReference.emit(mockCurrentPageReference);
 
         return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
@@ -98,7 +118,6 @@ describe("c-attendance", () => {
         fieldSetAdapter.emit(mockGetFieldSet);
         generateRosterAdapter.emit(mockGenerateRosterEmpty);
         wiredPermissionsAdapter.emit(mockWiredPermissions);
-        wiredCurrentPageReference.emit(mockCurrentPageReference);
 
         return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
@@ -126,7 +145,6 @@ describe("c-attendance", () => {
         fieldSetAdapter.emit(mockGetFieldSet);
         generateRosterAdapter.emit(mockGenerateRoster);
         wiredPermissionsAdapter.emit(mockWiredNoPermissions);
-        wiredCurrentPageReference.emit(mockCurrentPageReference);
 
         return global.flushPromises().then(() => {
             const attendanceRows = element.shadowRoot.querySelectorAll(
@@ -145,4 +163,352 @@ describe("c-attendance", () => {
             return global.isAccessible(element);
         });
     });
+    it("shows the default unit of measurement", () => {
+        let sessionSinUom = JSON.parse(JSON.stringify(mockWiredSession));
+        delete sessionSinUom.fields.ServiceSchedule__r.value.fields.Service__r.value
+            .fields.UnitOfMeasurement__c;
+        wiredSessionAdapter.emit(sessionSinUom);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        document.body.appendChild(element);
+
+        return global.flushPromises().then(() => {
+            const attendanceRow = element.shadowRoot.querySelector("c-attendance-row");
+            expect(attendanceRow.unitOfMeasurement).toBe(QUANTITY_LABEL);
+        });
+    });
+    it("loads form without a status value", () => {
+        let sessionSinUom = JSON.parse(JSON.stringify(mockWiredSession));
+        delete sessionSinUom.fields.Status__c;
+        wiredSessionAdapter.emit(sessionSinUom);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        document.body.appendChild(element);
+
+        return global.flushPromises().then(() => {
+            const attendanceRows = element.shadowRoot.querySelectorAll(
+                "c-attendance-row"
+            );
+            expect(attendanceRows).toHaveLength(mockGenerateRoster.length);
+        });
+    });
+    it("logs errors on getFieldSet", () => {
+        document.body.appendChild(element);
+        console.log = jest.fn();
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.error();
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredNoPermissions);
+        expect(console.log).toHaveBeenCalled();
+    });
+
+    it("logs errors on generateRoster", () => {
+        document.body.appendChild(element);
+        console.log = jest.fn();
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.error();
+        wiredPermissionsAdapter.emit(mockWiredNoPermissions);
+        expect(console.log).toHaveBeenCalled();
+    });
+
+    it("logs errors on getRecord", () => {
+        document.body.appendChild(element);
+        console.log = jest.fn();
+        wiredSessionAdapter.error();
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredNoPermissions);
+        expect(console.log).toHaveBeenCalled();
+    });
+    it("logs errors on checkFieldPermissions", () => {
+        document.body.appendChild(element);
+        console.log = jest.fn();
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.error();
+        expect(console.log).toHaveBeenCalled();
+    });
+});
+
+describe("c-attendance saving rows", () => {
+    let element;
+
+    afterEach(global.clearDOM);
+
+    beforeEach(() => {
+        element = createElement("c-attendance", {
+            is: Attendance,
+        });
+        element.recordId = "a06630000074mVnAAI";
+        currentPageReferenceAdapter.emit(mockAttendanceTabPageReference);
+    });
+
+    it("upserts rows calls save on row when successful", () => {
+        document.body.appendChild(element);
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        upsertRows.mockResolvedValue();
+        updateRecord.mockResolvedValue();
+        let spy;
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const attendanceRow = element.shadowRoot.querySelector(
+                    "c-attendance-row"
+                );
+                spy = jest.spyOn(attendanceRow, "save");
+                const submit = element.shadowRoot.querySelectorAll("lightning-button")[1];
+                expect(submit.label).toBe(SUBMIT_LABEL);
+                submit.click();
+            })
+            .then(() => {
+                expect(
+                    upsertRows.mock.calls[0][0].serviceDeliveriesToUpsert.length
+                ).toEqual(mockGenerateRoster.length);
+                expect(spy).toHaveBeenCalled();
+            });
+    });
+    it("updates the session status to complete", () => {
+        document.body.appendChild(element);
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        upsertRows.mockResolvedValue();
+        updateRecord.mockResolvedValue();
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const submit = element.shadowRoot.querySelectorAll("lightning-button")[1];
+                expect(submit.label).toBe(SUBMIT_LABEL);
+                submit.click();
+            })
+            .then(() => {
+                let record = { fields: { Id: element.recordId } };
+                record.fields[SESSION_STATUS_FIELD.fieldApiName] = "Complete";
+                expect(updateRecord).toHaveBeenCalledWith(record);
+            });
+    });
+    it("upserts empty rows sends an empty list", () => {
+        document.body.appendChild(element);
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit([{}]);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        upsertRows.mockResolvedValue();
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const row = element.shadowRoot.querySelector("c-attendance-row");
+                // Setting record to undefined to simulate what is returned when there
+                // are no edits to the record.
+                row.record = undefined;
+                const submit = element.shadowRoot.querySelectorAll("lightning-button")[1];
+                expect(submit.label).toBe(SUBMIT_LABEL);
+                submit.click();
+            })
+            .then(() => {
+                expect(
+                    upsertRows.mock.calls[0][0].serviceDeliveriesToUpsert.length
+                ).toEqual(0);
+            });
+    });
+    it("sends upsert error to the error handler", () => {
+        const handleErrorSpy = jest.spyOn(cUtil, "handleError");
+        document.body.appendChild(element);
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        upsertRows.mockRejectedValue(mockApexError);
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const submit = element.shadowRoot.querySelectorAll("lightning-button")[1];
+                expect(submit.label).toBe(SUBMIT_LABEL);
+                submit.click();
+            })
+            .then(() => {
+                expect(handleErrorSpy).not.toHaveBeenCalled();
+            });
+    });
+    it("sends update error to the error handler", () => {
+        const handleErrorSpy = jest.spyOn(cUtil, "handleError");
+        document.body.appendChild(element);
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        upsertRows.mockResolvedValue();
+        updateRecord.mockRejectedValue({});
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const submit = element.shadowRoot.querySelectorAll("lightning-button")[1];
+                expect(submit.label).toBe(SUBMIT_LABEL);
+                submit.click();
+            })
+            .then(() => {
+                // We just need this step to enter the catch
+            })
+            .then(() => {
+                expect(handleErrorSpy).toHaveBeenCalled();
+            });
+    });
+});
+
+describe("c-attendance printing the roster", () => {
+    let element;
+
+    afterEach(global.clearDOM);
+
+    beforeEach(() => {
+        element = createElement("c-attendance", {
+            is: Attendance,
+        });
+        element.recordId = "a06630000074mVnAAI";
+    });
+
+    it("prints from attendance tab", () => {
+        currentPageReferenceAdapter.emit(mockAttendanceTabPageReference);
+        document.body.appendChild(element);
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        window.print = jest.fn();
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const print = element.shadowRoot.querySelector("lightning-button");
+                expect(print.label).toBe(PRINT_LABEL);
+                print.click();
+            })
+            .then(() => {
+                expect(window.print).toHaveBeenCalled();
+            });
+    });
+
+    it("navigates to the attendance tab", () => {
+        currentPageReferenceAdapter.emit(mockServiceSessionPageReference);
+        document.body.appendChild(element);
+        wiredSessionAdapter.emit(mockWiredSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        window.open = jest.fn();
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const print = element.shadowRoot.querySelector("lightning-button");
+                expect(print.label).toBe(PRINTABLE_VIEW_LABEL);
+                print.click();
+            })
+            .then(() => {
+                expect(window.open).toHaveBeenCalled();
+            });
+    });
+});
+
+describe("c-attendance updating records", () => {
+    let element;
+
+    afterEach(global.clearDOM);
+
+    beforeEach(() => {
+        element = createElement("c-attendance", {
+            is: Attendance,
+        });
+        element.recordId = "a06630000074mVnAAI";
+        currentPageReferenceAdapter.emit(mockAttendanceTabPageReference);
+    });
+
+    it("allows updates on completed records", () => {
+        document.body.appendChild(element);
+        let completedSession = JSON.parse(JSON.stringify(mockWiredSession));
+        completedSession.fields[SESSION_STATUS_FIELD.fieldApiName].value = "Complete";
+        wiredSessionAdapter.emit(completedSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const update = element.shadowRoot.querySelector("lightning-button");
+                expect(update.label).toBe(UPDATE_LABEL);
+                update.click();
+            })
+            .then(() => {
+                const save = element.shadowRoot.querySelectorAll("lightning-button")[2];
+                expect(save.label).toBe(SAVE_LABEL);
+            });
+    });
+
+    it("allows cancel when updating completed records", () => {
+        document.body.appendChild(element);
+        let completedSession = JSON.parse(JSON.stringify(mockWiredSession));
+        completedSession.fields[SESSION_STATUS_FIELD.fieldApiName].value = "Complete";
+        wiredSessionAdapter.emit(completedSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const update = element.shadowRoot.querySelector("lightning-button");
+                expect(update.label).toBe(UPDATE_LABEL);
+                update.click();
+            })
+            .then(() => {
+                const cancel = element.shadowRoot.querySelectorAll("lightning-button")[1];
+                expect(cancel.label).toBe(CANCEL_LABEL);
+                cancel.click();
+            })
+            .then(() => {
+                const update = element.shadowRoot.querySelector("lightning-button");
+                expect(update.label).toBe(UPDATE_LABEL);
+            });
+    });
+
+    it("does not update session when not pending", () => {
+        document.body.appendChild(element);
+        let completedSession = JSON.parse(JSON.stringify(mockWiredSession));
+        completedSession.fields[SESSION_STATUS_FIELD.fieldApiName].value = "Complete";
+        wiredSessionAdapter.emit(completedSession);
+        fieldSetAdapter.emit(mockGetFieldSet);
+        generateRosterAdapter.emit(mockGenerateRoster);
+        wiredPermissionsAdapter.emit(mockWiredPermissions);
+        upsertRows.mockResolvedValue();
+
+        return global
+            .flushPromises()
+            .then(() => {
+                const update = element.shadowRoot.querySelector("lightning-button");
+                expect(update.label).toBe(UPDATE_LABEL);
+                update.click();
+            })
+            .then(() => {
+                const save = element.shadowRoot.querySelectorAll("lightning-button")[2];
+                expect(save.label).toBe(SAVE_LABEL);
+                save.click();
+            })
+            .then(() => {
+                expect(updateRecord).not.toHaveBeenCalled();
+            });
+    });
 });
diff --git a/force-app/main/default/lwc/attendance/__tests__/data/attendanceTabPageReference.json b/force-app/main/default/lwc/attendance/__tests__/data/attendanceTabPageReference.json
new file mode 100644
index 000000000..9789204a8
--- /dev/null
+++ b/force-app/main/default/lwc/attendance/__tests__/data/attendanceTabPageReference.json
@@ -0,0 +1,5 @@
+{
+    "type": "standard__navItemPage",
+    "attributes": { "apiName": "Attendance" },
+    "state": { "c__sessionId": "a0663000007xC61AAE" }
+}
diff --git a/force-app/main/default/lwc/attendance/__tests__/data/currentPageReference.json b/force-app/main/default/lwc/attendance/__tests__/data/currentPageReference.json
deleted file mode 100644
index 5a3aba050..000000000
--- a/force-app/main/default/lwc/attendance/__tests__/data/currentPageReference.json
+++ /dev/null
@@ -1,7 +0,0 @@
-{
-    "type": "standard__navItemPage",
-    "attributes": {
-        "apiName": "Attendance"
-    },
-    "state": {}
-}
diff --git a/force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json b/force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json
index 389e7cf31..49d813a43 100644
--- a/force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json
+++ b/force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json
@@ -7,7 +7,8 @@
         "isAccessible": true,
         "relationshipName": "Contact__r",
         "referenceTo": "Contact",
-        "referenceNameField": "Name"
+        "referenceNameField": "Name",
+        "isUpdateable": true
     },
     {
         "apiName": "Quantity__c",
@@ -15,20 +16,23 @@
         "type": "DOUBLE",
         "isRequired": false,
         "helpText": "A 2-decimal numeric field for Quantity of a Service Delivery expressed in the Unit of Measurement in the Service. For example, 30 minutes of tutoring where the Unit of Measure is hours, Quantity is .50.",
-        "isAccessible": true
+        "isAccessible": true,
+        "isUpdateable": true
     },
     {
         "apiName": "AttendanceStatus__c",
         "label": "Attendance Status",
         "type": "PICKLIST",
         "isRequired": false,
-        "isAccessible": true
+        "isAccessible": true,
+        "isUpdateable": true
     },
     {
         "apiName": "CreatedDate",
         "label": "Created Date",
         "type": "DATETIME",
         "isRequired": true,
-        "isAccessible": true
+        "isAccessible": true,
+        "isUpdateable": false
     }
 ]
diff --git a/force-app/main/default/lwc/attendance/__tests__/data/serviceSessionPageReference.json b/force-app/main/default/lwc/attendance/__tests__/data/serviceSessionPageReference.json
new file mode 100644
index 000000000..dea6f847d
--- /dev/null
+++ b/force-app/main/default/lwc/attendance/__tests__/data/serviceSessionPageReference.json
@@ -0,0 +1,9 @@
+{
+    "type": "standard__recordPage",
+    "attributes": {
+        "recordId": "a0663000007xC5DAAU",
+        "actionName": "view",
+        "objectApiName": "ServiceSession__c"
+    },
+    "state": {}
+}
diff --git a/force-app/main/default/lwc/attendance/attendance.js b/force-app/main/default/lwc/attendance/attendance.js
index 500371a53..4dc2fb713 100644
--- a/force-app/main/default/lwc/attendance/attendance.js
+++ b/force-app/main/default/lwc/attendance/attendance.js
@@ -119,6 +119,8 @@ export default class Attendance extends NavigationMixin(LightningElement) {
         if (result.data) {
             this.hasReadPermissions = result.data.read;
             this.hasWritePermissions = result.data.write;
+        } else {
+            console.log(result.error);
         }
     }
 
@@ -134,17 +136,18 @@ export default class Attendance extends NavigationMixin(LightningElement) {
         if (result.data) {
             let schedule = getChildObjectByName(result.data.fields, "ServiceSchedule__r");
             let service = getChildObjectByName(schedule.value.fields, "Service__r");
-            this.unitOfMeasurement = service.value.fields[
-                UNIT_MEASUREMENT_SERVICE_FIELD.fieldApiName
-            ].value
-                ? service.value.fields[UNIT_MEASUREMENT_SERVICE_FIELD.fieldApiName].value
-                : this.labels.quantity;
+            this.unitOfMeasurement =
+                service.value.fields[UNIT_MEASUREMENT_SERVICE_FIELD.fieldApiName] &&
+                service.value.fields[UNIT_MEASUREMENT_SERVICE_FIELD.fieldApiName].value
+                    ? service.value.fields[UNIT_MEASUREMENT_SERVICE_FIELD.fieldApiName]
+                          .value
+                    : this.labels.quantity;
             this.sessionStatus = result.data.fields[
                 this.fields.sessionStatus.fieldApiName
             ]
                 ? result.data.fields[this.fields.sessionStatus.fieldApiName].value
                 : undefined;
-        } else if (result.error) {
+        } else {
             console.log(result.error);
         }
     }
@@ -167,7 +170,7 @@ export default class Attendance extends NavigationMixin(LightningElement) {
                     ? 1
                     : -1;
             });
-        } else if (result.error) {
+        } else {
             console.log(result.error);
         }
         this.showSpinner = false;
@@ -283,7 +286,6 @@ export default class Attendance extends NavigationMixin(LightningElement) {
             }
         });
         this.showSpinner = true;
-
         upsertRows({
             serviceDeliveriesToUpsert: editedRows,
         })
diff --git a/force-app/main/default/lwc/attendanceRow/__mocks__/attendanceRow.js b/force-app/main/default/lwc/attendanceRow/__mocks__/attendanceRow.js
index f5eaff3d4..1b5db0169 100644
--- a/force-app/main/default/lwc/attendanceRow/__mocks__/attendanceRow.js
+++ b/force-app/main/default/lwc/attendanceRow/__mocks__/attendanceRow.js
@@ -16,4 +16,12 @@ export default class AttendanceRow extends LightningElement {
     @api record;
     @api fieldSet;
     @api readOnly;
+
+    @api getRow() {
+        return this.record;
+    }
+
+    @api save() {
+        jest.fn();
+    }
 }
diff --git a/jest.config.js b/jest.config.js
index 802c4904d..4af8d0e71 100644
--- a/jest.config.js
+++ b/jest.config.js
@@ -1,7 +1,14 @@
 const { jestConfig } = require("@salesforce/sfdx-lwc-jest/config");
 module.exports = {
     ...jestConfig,
+    moduleNameMapper: {
+        "^lightning/navigation$":
+            "<rootDir>/force-app/main/default/lwc/__tests__/lightning/navigation",
+        "^lightning/uiRecordApi$":
+            "<rootDir>/force-app/main/default/lwc/__tests__/lightning/uiRecordApi"
+    },
     testPathIgnorePatterns: ["force-app/main/default/lwc/__tests__/", ".cci"],
+    coveragePathIgnorePatterns: ["force-app/main/default/lwc/__tests__/", ".cci"],
     reporters: ["default", "jest-junit"],
     setupFilesAfterEnv: ["./jest.setup.js"]
 };

From cbda8b326e3be0b7e45da66352396586e1439598 Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Fri, 4 Feb 2022 12:06:44 -0600
Subject: [PATCH 10/11] Remove getFieldSet test data, no longer calling the
 method separately.

---
 .../__tests__/data/getFieldSet.json           | 38 -------------------
 1 file changed, 38 deletions(-)
 delete mode 100644 force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json

diff --git a/force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json b/force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json
deleted file mode 100644
index 49d813a43..000000000
--- a/force-app/main/default/lwc/attendance/__tests__/data/getFieldSet.json
+++ /dev/null
@@ -1,38 +0,0 @@
-[
-    {
-        "apiName": "Contact__c",
-        "label": "Client",
-        "type": "REFERENCE",
-        "isRequired": false,
-        "isAccessible": true,
-        "relationshipName": "Contact__r",
-        "referenceTo": "Contact",
-        "referenceNameField": "Name",
-        "isUpdateable": true
-    },
-    {
-        "apiName": "Quantity__c",
-        "label": "Quantity",
-        "type": "DOUBLE",
-        "isRequired": false,
-        "helpText": "A 2-decimal numeric field for Quantity of a Service Delivery expressed in the Unit of Measurement in the Service. For example, 30 minutes of tutoring where the Unit of Measure is hours, Quantity is .50.",
-        "isAccessible": true,
-        "isUpdateable": true
-    },
-    {
-        "apiName": "AttendanceStatus__c",
-        "label": "Attendance Status",
-        "type": "PICKLIST",
-        "isRequired": false,
-        "isAccessible": true,
-        "isUpdateable": true
-    },
-    {
-        "apiName": "CreatedDate",
-        "label": "Created Date",
-        "type": "DATETIME",
-        "isRequired": true,
-        "isAccessible": true,
-        "isUpdateable": false
-    }
-]

From 576c577edab764812deafa5a1ce69fa56f089b43 Mon Sep 17 00:00:00 2001
From: Jennifer Bennett <jennifer.bennett@salesforce.com>
Date: Fri, 4 Feb 2022 12:32:00 -0600
Subject: [PATCH 11/11] Add comment update year

---
 .../default/lwc/attendanceRow/__tests__/attendanceRow.test.js  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/force-app/main/default/lwc/attendanceRow/__tests__/attendanceRow.test.js b/force-app/main/default/lwc/attendanceRow/__tests__/attendanceRow.test.js
index 94befa795..f345d1a95 100644
--- a/force-app/main/default/lwc/attendanceRow/__tests__/attendanceRow.test.js
+++ b/force-app/main/default/lwc/attendanceRow/__tests__/attendanceRow.test.js
@@ -1,6 +1,6 @@
 /*
  *
- *  * Copyright (c) 2021, salesforce.com, inc.
+ *  * Copyright (c) 2022, salesforce.com, inc.
  *  * All rights reserved.
  *  * SPDX-License-Identifier: BSD-3-Clause
  *  * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/BSD-3-Clause
@@ -21,6 +21,7 @@ describe("c-attendanceRow", () => {
         });
     });
 
+    // TODO: Finish writing tests, need to pass in data
     it("loads in read only mode", () => {
         document.body.appendChild(element);