Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Test Plans to the Test Queue: Test plan dropdown includes two tests that are not in the tests directory of aria-at #581

Closed
mcking65 opened this issue Mar 28, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@mcking65
Copy link

To reproduce:

  1. Open Test Management page
  2. Expand "Add Test Plans to the Test Queue"
  3. Tab to test plan drop down.
  4. Inspect values in the drop down.
  5. Compare to available aria-at test plans.

Expected behavior: The test plans in the drop down match the plans available in the repository.

Actual behavior: There are two extra plans ("Alert" and "Modal Dialog") that appear at the bottom of the dropdown list.

@howard-e
Copy link
Contributor

Actual behavior: There are two extra plans ("Alert" and "Modal Dialog") that appear at the bottom of the dropdown list.

Seems there's a 3rd one here as well, "Checkbox Example (Tri State)" that's also at the bottom of the list. After it was renamed to "Checkbox Example (Mixed State)", looks like it wasn't properly mapped or removed. This "un-syncing" may also explain why it doesn't fall into the alphabetical order of the items above it

@alflennik
Copy link
Contributor

We discussed this as a group this morning and considered some solutions to the issue. We decided that a good way to solve it would be to author a migration to update the obsolete directory names so they match the current directory names.

@ccanash ccanash assigned ccanash and howard-e and unassigned ccanash Apr 12, 2023
@ccanash
Copy link
Contributor

ccanash commented Apr 12, 2023

This has been triaged and assigned to @howard-e

@howard-e
Copy link
Contributor

#600 has been merged which includes a migration for the items no longer being shown in the dropdown.

A follow-up PR will follow which will prevent the chance of this happening in the future.

@howard-e
Copy link
Contributor

howard-e commented Aug 15, 2023

A follow-up PR will follow which will prevent the chance of this happening in the future.

Closing as this is solved and in production, but discussion on the follow-up PR may happen in #748.

@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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
No open projects
Development

No branches or pull requests

4 participants