-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update report status dialog for recommended test plans to include information about required AT/Browser versions #792
Comments
Hi @mcking65 would you be able to add the criterias you mentioned at the end of the issue description that outlines the criteria for adding new row to the table for each AT/Browser combination. |
This issue should be part of the trend reporting project. I tried to find the project and could not. Have you created it? If so, it is not associated with either the aria-at or aria-at-app repos. This is a pretty complex issue and it raises some questions that need discussion. It might be better if I were to consider how to break it out into multiple issues. |
@mcking65 to facilitate future discussions, I’ve summarized some key points of our recent discussions that may have not yet been captured by the top comment, captured elsewhere or just in need of further clarifications: Key points
Example
|
@mcking65 In addition to the current restriction applied to the main point, there was a brief discussion on an additional restriction that if the test plan was in a draft phase, it can only be added with "Any recent" version. Could you confirm if that was understood correctly? And if so what was the decision there? |
Discussed outside this issue - the decision is not to add this additional constraint so give admins and testers can have greater flexibility in results which can be collected. The existing constraint remains if the test plan is in recommended. |
…ress #792 * Add resolver for finding firstRequiredAtVersion for a RECOMMENDED TestPlanVersion, given an atId * Update tests * Fix graphql call when including "firstRequiredAtVersion" under "testPlanVersions" * Update description of firstRequiredAtVersion * Rename resolver
* Add minimum or exact at version to reports * Quick tweak * Revert home copy change * Remove unused field from createTestPlanReport * Fix undefined var * Prevent API from creating duplicate reports * Support primary test plan to be selected * Fix test * Add dialog when marking report as final for an admin to select from probably primary test run options * prioritised -> prioritized typo (british -> american english) * Avoid displaying primary test plan run confirmation when just 1 run option * Add atVersion frontend * Make sure automation dialog always shows when valid * Make sure existing reports have a minimum at version * Formatting * feat: Add resolver for tracking first required AT Version (#1051) Address #792 * Add resolver for finding firstRequiredAtVersion for a RECOMMENDED TestPlanVersion, given an atId * Update tests * Fix graphql call when including "firstRequiredAtVersion" under "testPlanVersions" * Update description of firstRequiredAtVersion * Rename resolver --------- Co-authored-by: alflennik <[email protected]>
…s RECOMMENDED (#1052) * Add minimum or exact at version to reports * Quick tweak * Revert home copy change * Remove unused field from createTestPlanReport * Fix undefined var * Prevent API from creating duplicate reports * Support primary test plan to be selected * Fix test * Add resolver for finding firstRequiredAtVersion for a RECOMMENDED TestPlanVersion, given an atId * Add dialog when marking report as final for an admin to select from probably primary test run options * prioritised -> prioritized typo (british -> american english) * Update tests * Track recommended AT version * Avoid displaying primary test plan run confirmation when just 1 run option * Fix graphql call when including "firstRequiredAtVersion" under "testPlanVersions" * Update description of firstRequiredAtVersion * Add atVersion frontend * Make sure automation dialog always shows when valid * Rename resolver * Make sure existing reports have a minimum at version * Formatting * Fix graphql calls when doing testPlanReports > recommendedAtVersion * feat: Add resolver for tracking first required AT Version (#1051) Address #792 * Add resolver for finding firstRequiredAtVersion for a RECOMMENDED TestPlanVersion, given an atId * Update tests * Fix graphql call when including "firstRequiredAtVersion" under "testPlanVersions" * Update description of firstRequiredAtVersion * Rename resolver * Use exactAtVersionId if available for recommendedAtVersion * Update comment --------- Co-authored-by: alflennik <[email protected]>
…#1087) * Add minimum or exact at version to reports * Quick tweak * Revert home copy change * Remove unused field from createTestPlanReport * Fix undefined var * Prevent API from creating duplicate reports * Support primary test plan to be selected * Fix test * Add dialog when marking report as final for an admin to select from probably primary test run options * prioritised -> prioritized typo (british -> american english) * Avoid displaying primary test plan run confirmation when just 1 run option * Add atVersion frontend * Make sure automation dialog always shows when valid * Make sure existing reports have a minimum at version * Formatting * feat: Add resolver for tracking first required AT Version (#1051) Address #792 * Add resolver for finding firstRequiredAtVersion for a RECOMMENDED TestPlanVersion, given an atId * Update tests * Fix graphql call when including "firstRequiredAtVersion" under "testPlanVersions" * Update description of firstRequiredAtVersion * Rename resolver * Update status dialog for minimum exact AT versions * Recommended phase uses exact AT version * Remove unneeded ternary * Fix issue caused by using draft reports * Reduce number of at versions shown in status dialog * Fix unused var * Fix incorrect required browser * Address PR feedback --------- Co-authored-by: Howard Edwards <[email protected]>
This includes work to support #791 and #792. Includes the following changes: * #1055 * #1001 * #1065 * #1052 * #1087 * #1098 * #1092 * #1131 * #1124 --------- Co-authored-by: Howard Edwards <[email protected]> Co-authored-by: Paul Clue <[email protected]> Co-authored-by: alflennik <[email protected]>
The following requirement is listed in #648 under the heading "Test plan report status dialog Description":
This issue will complete that specification and track development of the requisite changes to the report status dialog.
ARIA-AT goal
One goal of the ARIA-AT working mode specific to test plans in the recommended phase is to keep their interop reports current. That means, for each AT/Browser combo that has a report, generate a new report when a new version of an AT or browser is released. In addition to keeping data current, over time, this will also create data that can be used to publish trend charts.
As of September 27, 2023, the CG plans to update recommended reports for each public release of a supported screen reader; see Minutes for ARIA and Assistive Technologies Community Group Weekly Teleconference – 27 September 2023. Until the ARIA-AT project has sufficient automation and compute resources, new browser releases will not trigger report updates. The browser requirement will be "Any Recent Version". In the event of report run conflicts that are attributable to browser version differences, data from the more recent browser version will be published.
Report status dialog design update objective
When viewing the report status for a recommended test plan, make it easy to determine which AT/Browser version combinations are missing reports and add them to the test queue.
Test queue dependency
The test queue does not currently support version-specific test plan runs. Issue #791 must be resolved at the same time or before this issue.
Primary test plan run dependency
The meet the data processing requirements of this solution, the system will need to enable admins to mark one of the test plan runs as "primary" when finalizing a report. The requirements for updates to the report finalization process are defined in issue ...
Definition of "First Required AT Versions" for a test plan
Incomplete DRAFT CONTENT
The goal is to update reports for all AT versions that are newer than the versions used to generate the data that supported the candidate review process of the test plan. In other words, if new AT versions were released during the review process for which reports were not generated, reports using those versions will need to be generated after the test plan becomes recommended. This prevents important gaps in the history of the AT support levels.
Thus, each recommended test plan will have its own set of "First Required AT Versions" or "First Required AT Version Dates". Whether date or version is used is an implementation detail; either will do. For the purpose of the following definition, version number is used.
A test plan has a "First Required AT Version" for each AT that meets the following two criteria:
The "First Required AT Version" for each AT that meets the above criteria is the earliest version used to generate any of the primary test plan results for any of the reports that are final on the date the test plan is advanced to recommended.
For example, consider a scenario where the command button test plan is advanced to recommended. At the time it is advanced to recommended, It has a report for NVDA and Chrome that was generated during draft review using NVDA 2022.4. This data was used during the review process It also has a report for NVDA and Firefox generated during candidate review using NVDA 2023.1. In this case, the first required NVDA version would be 2022.4. That means that
that was used to generate the primary test plan results shown in is
Report status data changes
When a test plan is advanced to the recommended phase, make the following changes to the data in the table in the report status dialog.
The text was updated successfully, but these errors were encountered: