Skip to content

Commit

Permalink
Polish test review v2 page (#861)
Browse files Browse the repository at this point in the history
* Polish test review v2 page

* Remove unused variable
  • Loading branch information
alflennik authored Dec 11, 2023
1 parent 234a339 commit 135e61b
Showing 1 changed file with 8 additions and 15 deletions.
23 changes: 8 additions & 15 deletions client/components/TestReview/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ const TestReview = () => {
if (loading) {
return (
<PageStatus
title="Loading - Tests for Test Plan Version | ARIA-AT"
heading="Tests for Test Plan Version"
title="Loading - Test Plan Version | ARIA-AT"
heading="Test Plan Version"
/>
);
}
Expand All @@ -51,16 +51,15 @@ const TestReview = () => {
`Failed to find a test plan version with ID ${testPlanVersionId}`;
return (
<PageStatus
title="Tests for Test Plan Version | ARIA-AT"
heading="Tests for Test Plan Version"
title="Test Plan Version | ARIA-AT"
heading="Test Plan Version"
message={errorMessage}
isError
/>
);
}

const testPlanVersion = data.testPlanVersion;
const testCount = testPlanVersion.tests.length;
const atNames = unique(
testPlanVersion.tests.flatMap(test => test.ats.map(at => at.name))
);
Expand Down Expand Up @@ -91,20 +90,14 @@ const TestReview = () => {
return (
<Container id="main" as="main" tabIndex="-1">
<Helmet>
<title>{`Tests for ${testPlanVersion.title} ${testPlanVersion.versionString} | ARIA-AT`}</title>
<title>{`${testPlanVersion.title} Test Plan ${testPlanVersion.versionString} | ARIA-AT`}</title>
</Helmet>
<h1>
{`Tests for ${testPlanVersion.title} ` +
{`${testPlanVersion.title} Test Plan ` +
`${testPlanVersion.versionString}` +
`${testPlanVersion.deprecatedAt ? ' (Deprecated)' : ''}`}
</h1>
<h2>Introduction</h2>
<p>
{`This page contains ${testCount} tests the ARIA-AT team ` +
`has authored for ${testPlanVersion.title}, version ` +
`${testPlanVersion.versionString}.`}
</p>
<h2>Metadata</h2>
<h2>About This Test Plan</h2>
<ul>
<li>
<strong>Phase:&nbsp;</strong>
Expand Down Expand Up @@ -173,7 +166,7 @@ const TestReview = () => {
{testPlanVersion.gitMessage}
</li>
</ul>
<h2>List of Tests</h2>
<h2>Tests</h2>
<FilterButtonContainer>
<FilterButtons
filterLabel="Filter tests by AT"
Expand Down

0 comments on commit 135e61b

Please sign in to comment.