Merge pull request #1234 from w3c/releases

October 9, 2024 Production Release Includes changes recently included in the [releases branch](https://github.com/w3c/aria-at-app/tree/releases) through #1233. [Latest CHANGELOG.md update: v1.9.0](https://github.com/w3c/aria-at-app/blob/releases/CHANGELOG.md#190-2024-10-09)
w3c · Oct 9, 2024 · 1f87b26 · 1f87b26
2 parents 553c905 + 19074d1
commit 1f87b26
Show file tree

Hide file tree

Showing 53 changed files with 6,250 additions and 812 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,16 @@
+## [1.9.0](https://github.com/w3c/aria-at-app/compare/v1.8.1...v1.9.0) (2024-10-09)
+
+
+### Features
+
+* Vendor verification ([#1208](https://github.com/w3c/aria-at-app/issues/1208)) ([b8b13d4](https://github.com/w3c/aria-at-app/commit/b8b13d48a5e05e31613bfea54678a30b525d18c6))
+
+
+### Bug Fixes
+
+* Add tests for retry cancelled collection jobs ([#1214](https://github.com/w3c/aria-at-app/issues/1214)) ([34d7aac](https://github.com/w3c/aria-at-app/commit/34d7aac53492b5077fe487ef51e2a7bd5c536afc))
+* Assertion with EXCLUDE priority causing test result submit errors ([#1229](https://github.com/w3c/aria-at-app/issues/1229)) ([d7cc1fd](https://github.com/w3c/aria-at-app/commit/d7cc1fd6efe3a72b3df6158a5d1fb9580f3f6de2))
+
 ### [1.8.1](https://github.com/w3c/aria-at-app/compare/v1.8.0...v1.8.1) (2024-09-23)
 
 

diff --git a/client/components/CandidateReview/TestPlans/index.jsx b/client/components/CandidateReview/TestPlans/index.jsx
@@ -21,6 +21,7 @@ import ClippedProgressBar from '@components/common/ClippedProgressBar';
 import { dates } from 'shared';
 import './TestPlans.css';
 import { calculations } from 'shared';
+import { UserPropType } from '../../common/proptypes';
 
 const FullHeightContainer = styled(Container)`
   min-height: calc(100vh - 64px);
@@ -177,7 +178,7 @@ const None = styled.span`
   }
 `;
 
-const TestPlans = ({ testPlanVersions }) => {
+const TestPlans = ({ testPlanVersions, me }) => {
   const [atExpandTableItems, setAtExpandTableItems] = useState({
     1: true,
     2: true,
@@ -503,20 +504,39 @@ const TestPlans = ({ testPlanVersions }) => {
                     .filter(t => t.isCandidateReview === true)
                     .filter(t => uniqueFilter(t, uniqueLinks, 'link'));
 
+                  const canReview =
+                    me.roles.includes('ADMIN') ||
+                    (me.roles.includes('VENDOR') &&
+                      me.company.ats.some(at => at.id === atId));
+
+                  const getTitleEl = () => {
+                    if (canReview) {
+                      return (
+                        <Link
+                          to={`/candidate-test-plan/${testPlanVersion.id}/${atId}`}
+                        >
+                          {getTestPlanVersionTitle(testPlanVersion, {
+                            includeVersionString: true
+                          })}{' '}
+                          ({testsCount} Test
+                          {testsCount === 0 || testsCount > 1 ? `s` : ''})
+                        </Link>
+                      );
+                    }
+                    return (
+                      <>
+                        {getTestPlanVersionTitle(testPlanVersion, {
+                          includeVersionString: true
+                        })}{' '}
+                        ({testsCount} Test
+                        {testsCount === 0 || testsCount > 1 ? `s` : ''})
+                      </>
+                    );
+                  };
                   return (
                     dataExists && (
                       <tr key={testPlanVersion.id}>
-                        <th>
-                          <Link
-                            to={`/candidate-test-plan/${testPlanVersion.id}/${atId}`}
-                          >
-                            {getTestPlanVersionTitle(testPlanVersion, {
-                              includeVersionString: true
-                            })}{' '}
-                            ({testsCount} Test
-                            {testsCount === 0 || testsCount > 1 ? `s` : ''})
-                          </Link>
-                        </th>
+                        <th>{getTitleEl()}</th>
                         <CenteredTd>
                           <i>
                             {dates.convertDateToString(
@@ -778,6 +798,7 @@ TestPlans.propTypes = {
       )
     })
   ).isRequired,
+  me: UserPropType.isRequired,
   triggerPageUpdate: PropTypes.func
 };
 

diff --git a/client/components/CandidateReview/index.jsx b/client/components/CandidateReview/index.jsx
@@ -3,12 +3,16 @@ import { useQuery } from '@apollo/client';
 import PageStatus from '../common/PageStatus';
 import TestPlans from './TestPlans';
 import { CANDIDATE_REVIEW_PAGE_QUERY } from './queries';
+import { ME_QUERY } from '../App/queries';
 
 const CandidateReview = () => {
   const { loading, data, error } = useQuery(CANDIDATE_REVIEW_PAGE_QUERY, {
     fetchPolicy: 'cache-and-network'
   });
 
+  const { data: meData } = useQuery(ME_QUERY);
+  const { me } = meData;
+
   if (error) {
     return (
       <PageStatus
@@ -33,7 +37,7 @@ const CandidateReview = () => {
 
   const testPlanVersions = data.testPlanVersions;
 
-  return <TestPlans testPlanVersions={testPlanVersions} />;
+  return <TestPlans testPlanVersions={testPlanVersions} me={me} />;
 };
 
 export default CandidateReview;
diff --git a/client/components/ConfirmAuth/index.jsx b/client/components/ConfirmAuth/index.jsx
@@ -1,15 +1,17 @@
 import React from 'react';
-import { Navigate } from 'react-router-dom';
+import { Navigate, useParams } from 'react-router-dom';
 import PropTypes from 'prop-types';
 import { useQuery } from '@apollo/client';
 import { ME_QUERY } from '../App/queries';
 import { evaluateAuth } from '../../utils/evaluateAuth';
 
-const ConfirmAuth = ({ children, requiredPermission }) => {
+const ConfirmAuth = ({ children, requiredPermission, requireVendorForAt }) => {
   const { data } = useQuery(ME_QUERY);
+  const { atId } = useParams();
 
   const auth = evaluateAuth(data && data.me ? data.me : {});
   const { roles, username, isAdmin, isSignedIn } = auth;
+  const company = data && data.me ? data.me.company : null;
 
   if (!username) return <Navigate to={{ pathname: '/invalid-request' }} />;
 
@@ -21,6 +23,14 @@ const ConfirmAuth = ({ children, requiredPermission }) => {
     return <Navigate to="/404" replace />;
   }
 
+  // Check if the user's company is the vendor for the associated At
+  if (requireVendorForAt && !isAdmin) {
+    const isVendorForAt = company && company.ats.some(at => at.id === atId);
+    if (!isVendorForAt) {
+      return <Navigate to="/404" replace />;
+    }
+  }
+
   return children;
 };
 
@@ -29,7 +39,8 @@ ConfirmAuth.propTypes = {
     PropTypes.arrayOf(PropTypes.node),
     PropTypes.node
   ]).isRequired,
-  requiredPermission: PropTypes.string
+  requiredPermission: PropTypes.string,
+  requireVendorForAt: PropTypes.bool
 };
 
 export default ConfirmAuth;
diff --git a/client/components/DisclaimerInfo/index.jsx b/client/components/DisclaimerInfo/index.jsx
@@ -104,7 +104,6 @@ const DisclaimerInfo = ({ phase }) => {
     <Container>
       <details>
         <summary
-          id="disclaimer-title"
           aria-expanded={expanded}
           aria-controls="description"
           onClick={() => setExpanded(!expanded)}

diff --git a/client/components/ManageBotRunDialog/queries.js b/client/components/ManageBotRunDialog/queries.js
@@ -52,6 +52,9 @@ export const RETRY_CANCELED_COLLECTIONS = gql`
       retryCanceledCollections {
         id
         status
+        testStatus {
+          status
+        }
       }
     }
   }

diff --git a/client/components/common/fragments/Me.js b/client/components/common/fragments/Me.js
@@ -6,6 +6,12 @@ const ME_FIELDS = gql`
     id
     username
     roles
+    company {
+      id
+      ats {
+        id
+      }
+    }
   }
 `;
 

diff --git a/client/index.js b/client/index.js
@@ -51,7 +51,20 @@ window.signMeInAsTester = username => {
 };
 
 window.signMeInAsVendor = username => {
-  return signMeInCommon({ username, roles: [{ name: 'VENDOR' }] });
+  return signMeInCommon({
+    username,
+    roles: [{ name: 'VENDOR' }],
+    company: {
+      id: '1',
+      name: 'vispero',
+      ats: [
+        {
+          id: '1',
+          name: 'JAWS'
+        }
+      ]
+    }
+  });
 };
 
 window.startTestTransaction = async () => {

diff --git a/client/routes/index.js b/client/routes/index.js
@@ -33,7 +33,7 @@ export default () => (
       exact
       path="/candidate-test-plan/:testPlanVersionId/:atId"
       element={
-        <ConfirmAuth requiredPermission="VENDOR">
+        <ConfirmAuth requiredPermission="VENDOR" requireVendorForAt={true}>
           <CandidateTestPlanRun />
         </ConfirmAuth>
       }

diff --git a/client/tests/__mocks__/GraphQLMocks/TestQueuePageAdminNotPopulatedMock.js b/client/tests/__mocks__/GraphQLMocks/TestQueuePageAdminNotPopulatedMock.js
@@ -9,7 +9,12 @@ export default testQueuePageQuery => [
           __typename: 'User',
           id: '1',
           username: 'foo-bar',
-          roles: ['ADMIN', 'TESTER']
+          roles: ['ADMIN', 'TESTER'],
+          company: {
+            id: '1',
+            name: 'Company',
+            ats: []
+          }
         },
         users: [
           {
@@ -18,23 +23,38 @@ export default testQueuePageQuery => [
             username: 'foo-bar',
             roles: ['ADMIN', 'TESTER'],
             isBot: false,
-            ats: []
+            ats: [],
+            company: {
+              id: '1',
+              name: 'Company',
+              ats: []
+            }
           },
           {
             __typename: 'User',
             id: '4',
             username: 'bar-foo',
             roles: ['TESTER'],
             isBot: false,
-            ats: []
+            ats: [],
+            company: {
+              id: '1',
+              name: 'Company',
+              ats: []
+            }
           },
           {
             __typename: 'User',
             id: '5',
             username: 'boo-far',
             roles: ['TESTER'],
             isBot: false,
-            ats: []
+            ats: [],
+            company: {
+              id: '1',
+              name: 'Company',
+              ats: []
+            }
           }
         ],
         ats: [],

diff --git a/client/tests/__mocks__/GraphQLMocks/TestQueuePageTesterNotPopulatedMock.js b/client/tests/__mocks__/GraphQLMocks/TestQueuePageTesterNotPopulatedMock.js
@@ -9,7 +9,12 @@ export default testQueuePageQuery => [
           __typename: 'User',
           id: '4',
           username: 'bar-foo',
-          roles: ['TESTER']
+          roles: ['TESTER'],
+          company: {
+            id: '1',
+            name: 'Company',
+            ats: []
+          }
         },
         users: [
           {
@@ -18,23 +23,38 @@ export default testQueuePageQuery => [
             username: 'foo-bar',
             roles: ['ADMIN', 'TESTER'],
             isBot: false,
-            ats: []
+            ats: [],
+            company: {
+              id: '1',
+              name: 'Company',
+              ats: []
+            }
           },
           {
             __typename: 'User',
             id: '4',
             username: 'bar-foo',
             roles: ['TESTER'],
             isBot: false,
-            ats: []
+            ats: [],
+            company: {
+              id: '1',
+              name: 'Company',
+              ats: []
+            }
           },
           {
             __typename: 'User',
             id: '5',
             username: 'boo-far',
             roles: ['TESTER'],
             isBot: false,
-            ats: []
+            ats: [],
+            company: {
+              id: '1',
+              name: 'Company',
+              ats: []
+            }
           }
         ],
         ats: [],
-Original file line number
+Diff line change
@@ Expand Up / @@ -52,6 +52,9 @@ export const RETRY_CANCELED_COLLECTIONS = gql` @@
           retryCanceledCollections {
             id
             status
+            testStatus {
+              status
+            }
           }
         }
       }
@@ Expand Down @@