From 79118cfbe2fceb98a9ba1904537d25a74cba680d Mon Sep 17 00:00:00 2001
From: "Keith J. Grant" <barkeep@keithjgrant.com>
Date: Mon, 12 Apr 2021 09:42:36 -0700
Subject: [PATCH] fix Schedule tests

---
 .../CodeEditor/VariablesDetail.test.jsx       |  4 +-
 .../src/components/Schedule/Schedule.test.jsx |  4 +-
 .../ScheduleList/ScheduleListItem.test.jsx    | 54 +++++++++++--------
 3 files changed, 35 insertions(+), 27 deletions(-)

diff --git a/awx/ui_next/src/components/CodeEditor/VariablesDetail.test.jsx b/awx/ui_next/src/components/CodeEditor/VariablesDetail.test.jsx
index 2705f2b2cf..a9caa9a21f 100644
--- a/awx/ui_next/src/components/CodeEditor/VariablesDetail.test.jsx
+++ b/awx/ui_next/src/components/CodeEditor/VariablesDetail.test.jsx
@@ -42,12 +42,12 @@ describe('<VariablesDetail>', () => {
     expect(input2.prop('value')).toEqual('foo: bar\n');
   });
 
-  test('should render label and value= --- when there are no values', () => {
+  test('should render label and value --- when there are no values', () => {
     const wrapper = mountWithContexts(
       <VariablesDetail value="" label="Variables" />
     );
     expect(wrapper.find('VariablesDetail___StyledCodeEditor').length).toBe(1);
-    expect(wrapper.find('div.pf-c-form__label').text()).toBe('Variables');
+    expect(wrapper.find('.pf-c-form__label').text()).toBe('Variables');
   });
 
   test('should update value if prop changes', () => {
diff --git a/awx/ui_next/src/components/Schedule/Schedule.test.jsx b/awx/ui_next/src/components/Schedule/Schedule.test.jsx
index 280c6af6bb..6082df1a95 100644
--- a/awx/ui_next/src/components/Schedule/Schedule.test.jsx
+++ b/awx/ui_next/src/components/Schedule/Schedule.test.jsx
@@ -121,8 +121,8 @@ describe('<Schedule />', () => {
   });
   test('expect all tabs to exist, including Back to Schedules', async () => {
     expect(
-      wrapper.find('button[link="/templates/job_template/1/schedules"]').length
+      wrapper.find('a[href="/templates/job_template/1/schedules"]').length
     ).toBe(1);
-    expect(wrapper.find('button[aria-label="Details"]').length).toBe(1);
+    expect(wrapper.find('a[aria-label="Details"]').length).toBe(1);
   });
 });
diff --git a/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.test.jsx b/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.test.jsx
index ccf5fa8855..394128a095 100644
--- a/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.test.jsx
+++ b/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.test.jsx
@@ -50,13 +50,17 @@ describe('ScheduleListItem', () => {
   describe('User has edit permissions', () => {
     beforeAll(() => {
       wrapper = mountWithContexts(
-        <ScheduleListItem
-          isSelected={false}
-          onSelect={onSelect}
-          schedule={mockSchedule}
-          isMissingSurvey={false}
-          isMissingInventory={false}
-        />
+        <table>
+          <tbody>
+            <ScheduleListItem
+              isSelected={false}
+              onSelect={onSelect}
+              schedule={mockSchedule}
+              isMissingSurvey={false}
+              isMissingInventory={false}
+            />
+          </tbody>
+        </table>
       );
     });
 
@@ -190,22 +194,26 @@ describe('ScheduleListItem', () => {
   describe('schedule has missing prompt data', () => {
     beforeAll(() => {
       wrapper = mountWithContexts(
-        <ScheduleListItem
-          isSelected={false}
-          onSelect={onSelect}
-          schedule={{
-            ...mockSchedule,
-            summary_fields: {
-              ...mockSchedule.summary_fields,
-              user_capabilities: {
-                edit: false,
-                delete: false,
-              },
-            },
-          }}
-          isMissingInventory="Inventory Error"
-          isMissingSurvey="Survey Error"
-        />
+        <table>
+          <tbody>
+            <ScheduleListItem
+              isSelected={false}
+              onSelect={onSelect}
+              schedule={{
+                ...mockSchedule,
+                summary_fields: {
+                  ...mockSchedule.summary_fields,
+                  user_capabilities: {
+                    edit: false,
+                    delete: false,
+                  },
+                },
+              }}
+              isMissingInventory="Inventory Error"
+              isMissingSurvey="Survey Error"
+            />
+          </tbody>
+        </table>
       );
     });