fix: Remove TestQueue v1 and update Test Queue e2e tests #1146
Conversation
There was a problem hiding this comment.
Tested manually thoroughly with both mouse + screen and screenreader and didn't encounter any bugs. Nice job removing this. Also really excited about the extensive new end-to-end tests!
Small confirmation comment inline on something that seemed a bit out of place. Otherwise approved!
There was a problem hiding this comment.
Looking at main it seems that this was no longer used and this is clean up because it isn't likely to be used in the future? Adding this comment as a verification measure
There was a problem hiding this comment.
Right! It was meant for a much older implementation of showing the test collection form before the <TestRenderer> was a thing. I was pretty surprised it was still there so using this as a chance to remove it.
howard-e
changed the title
howard-e
changed the title
Includes the following changes: **Features and Fixes** * #1125 * #1135 * #1144 * #1146 * #1150 * #1152 * #1153 * #1160 **Infrastructure changes** * #969 * #1151 * #1148 --------- Co-authored-by: alflennik <[email protected]> Co-authored-by: Mx Corey Frang <[email protected]> Co-authored-by: cypress evelyn masso <[email protected]> Co-authored-by: Stalgia Grigg <[email protected]>
This removes the original
components/TestQueueand related components in favor of the updated Test Queue introduced in #1124.This PR also updates and reorganizes tests related to the the original Test Queue and removes tests done using mocks and replaces them with e2e tests where appropriate.