Remove browsers from query and state in TestQueue and DataManagement …

…in favor of at.browsers
w3c · Nov 16, 2023 · 9319e37 · 9319e37
1 parent 7dd31e1
commit 9319e37
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 26 deletions.
diff --git a/client/components/DataManagement/index.jsx b/client/components/DataManagement/index.jsx
@@ -30,7 +30,6 @@ const DataManagement = () => {
 
     const [pageReady, setPageReady] = useState(false);
     const [ats, setAts] = useState([]);
-    const [browsers, setBrowsers] = useState([]);
     const [testPlans, setTestPlans] = useState([]);
     const [testPlanVersions, setTestPlanVersions] = useState([]);
     const [filter, setFilter] = useState(
@@ -42,14 +41,8 @@ const DataManagement = () => {
 
     useEffect(() => {
         if (data) {
-            const {
-                ats = [],
-                browsers = [],
-                testPlanVersions = [],
-                testPlans = []
-            } = data;
+            const { ats = [], testPlanVersions = [], testPlans = [] } = data;
             setAts(ats);
-            setBrowsers(browsers);
             setTestPlans(testPlans);
             setTestPlanVersions(testPlanVersions);
             setPageReady(true);
@@ -131,7 +124,6 @@ const DataManagement = () => {
 
                     <ManageTestQueue
                         ats={ats}
-                        browsers={browsers}
                         testPlanVersions={testPlanVersions}
                         triggerUpdate={refetch}
                     />

diff --git a/client/components/DataManagement/queries.js b/client/components/DataManagement/queries.js
@@ -26,10 +26,6 @@ export const DATA_MANAGEMENT_PAGE_QUERY = gql`
                 id
             }
         }
-        browsers {
-            id
-            name
-        }
         testPlans {
             id
             directory

diff --git a/client/components/ManageTestQueue/index.jsx b/client/components/ManageTestQueue/index.jsx
@@ -91,7 +91,6 @@ const DisclosureContainer = styled.div`
 
 const ManageTestQueue = ({
     ats = [],
-    browsers = [],
     testPlanVersions = [],
     triggerUpdate = () => {}
 }) => {
@@ -667,9 +666,11 @@ const ManageTestQueue = ({
                                 item => item.id === selectedTestPlanVersionId
                             )}
                             at={ats.find(item => item.id === selectedAtId)}
-                            browser={browsers.find(
-                                item => item.id === selectedBrowserId
-                            )}
+                            browser={ats
+                                .find(at => at.id === selectedAtId)
+                                ?.browsers.find(
+                                    browser => browser.id === selectedBrowserId
+                                )}
                             triggerUpdate={triggerUpdate}
                             disabled={
                                 !selectedTestPlanVersionId ||
@@ -738,7 +739,6 @@ const ManageTestQueue = ({
 
 ManageTestQueue.propTypes = {
     ats: PropTypes.array,
-    browsers: PropTypes.array,
     testPlanVersions: PropTypes.array,
     triggerUpdate: PropTypes.func
 };

diff --git a/client/components/TestQueue/index.jsx b/client/components/TestQueue/index.jsx
@@ -21,7 +21,6 @@ const TestQueue = () => {
     const [pageReady, setPageReady] = useState(false);
     const [testers, setTesters] = useState([]);
     const [ats, setAts] = useState([]);
-    const [browsers, setBrowsers] = useState([]);
     const [testPlanVersions, setTestPlanVersions] = useState([]);
     const [testPlanReports, setTestPlanReports] = useState([]);
     const [latestTestPlanVersions, setLatestTestPlanVersions] = useState([]);
@@ -45,7 +44,6 @@ const TestQueue = () => {
             const {
                 users = [],
                 ats = [],
-                browsers = [],
                 testPlanVersions = [],
                 testPlanReports = [],
                 testPlans = []
@@ -60,7 +58,6 @@ const TestQueue = () => {
             setAts(ats);
             setTestPlanVersions(testPlanVersions);
             setTestPlanReports(testPlanReports);
-            setBrowsers(browsers);
             setLatestTestPlanVersions(testPlans);
             setPageReady(true);
         }
@@ -254,7 +251,6 @@ const TestQueue = () => {
             {isAdmin && (
                 <ManageTestQueue
                     ats={ats}
-                    browsers={browsers}
                     testPlanVersions={testPlanVersions}
                     triggerUpdate={refetch}
                 />

diff --git a/client/components/TestQueue/queries.js b/client/components/TestQueue/queries.js
@@ -33,10 +33,6 @@ export const TEST_QUEUE_PAGE_QUERY = gql`
                 name
             }
         }
-        browsers {
-            id
-            name
-        }
         testPlanVersions {
             id
             title
-Original file line number
+Diff line change
@@ Expand Up / @@ -26,10 +26,6 @@ export const DATA_MANAGEMENT_PAGE_QUERY = gql` @@
                     id
                 }
             }
-            browsers {
-                id
-                name
-            }
             testPlans {
                 id
                 directory
@@ Expand Down @@