Skip to content
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

Some test plans shown as candidate but with required reports missing #762

Closed
mcking65 opened this issue Aug 30, 2023 · 2 comments
Closed
Labels
bug Something isn't working

Comments

@mcking65
Copy link

The data management page shows some test plans in candidate review that have missing required reports. This should not be possible. three reports are required before a plan can move from draft to candidate review. Once in candidate review, there are no additional reporting requirements. Additional reporting is not required until after a plan is recommended.

In the staging environment, two plans that are in candidate review and showing that required reports are missing are:

  • Action Menu Button Example Using aria-activedescendant
  • Disclosure Navigation Menu Example

image

@howard-e
Copy link
Contributor

@mcking65 this is only slightly related to #758, in that some of the required reports shown as missing could be getting incorrectly shown.

But In the instance that it isn't, then this is another consequence of the migration. It picked up the previous TestPlanReport.status (which no longer exists) as CANDIDATE, so it was marked as final and the parent new TestPlanVersion.phase was pushed to CANDIDATE to support the this new structure. We acknowledged that the phases of some TestPlanVersions won't be possible in the future now that conditions such as required reports will prevent that ,through the server and client operations. We could push both of those TestPlanVersions back to DRAFT for "correctness" in the staging and sandbox environments, but it may not also be necessary soon, given that all the required reports are now included in the production data.

Also yay for an empty Test Queue!

@mcking65
Copy link
Author

OK then, anything that is not addressed by #758 will be overcome by events. We will add the missing reports or bring a new version of the plan forward or both.

@mcking65 mcking65 changed the title It should be impossible to advance to candidate if required reports are missing Some test plans shown as candidate but with required reports missing Sep 26, 2023
@ccanash ccanash moved this from In staging/sandbox to In production / Completed in MVP support of ARIA-AT Working Mode by ARIA-AT App - 2023 H1 Project Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Development

No branches or pull requests

2 participants