-
Notifications
You must be signed in to change notification settings - Fork 8
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
RC Test: John Travoltage 1.5.0-rc.1 #196
Labels
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@KatieWoe, @emily-phet - John Travoltage 1.5.0-rc.1 is ready for RC testing. Please document issues in https://github.com/phetsims/john-travoltage/issues and link to this issue. The primary purpose of this release is adding sound to the sim, so that should be the primary focus of the testing. The last release of this sim was on 8/15/2018, but the branch it was from (i.e. the 1.3 branch) was created in May of 2017, so the sim has picked up a lot of common code updates since its last publication. The sonification work was pretty concentrated, and is unlikely to have messed up anything major, so I suspect that you can probably limit things like string testing to the major platforms.
Assigning @ariel-phet for prioritization.
General RC Test
What to Test
Focus and Special Instructions
The primary focus, as mentioned above, should be on testing sound generation.
One important note: By default, the sound generation is set to the "basic" level. The "enhanced" level can be turned on through the PhET menu. The only difference is that in "enhanced" mode, there is a popping sound produced as electrons enter and leave the body. Both modes should be tested on every platform (which should be very quick).
On IE, sound should not occur at all, and the sound button should not be visible on the nav bar. There is an issue in the list below about this, but I wanted to make it clear that this is the intended behavior.
Also, there is a new query parameter named 'sound'. There is an issue on the list below about it. The bottom line is that we need to make sure that it works as expected.
Issues to Verify
These issues should have the "status:fixed-pending-testing" label. Check these issues off and close them if they are fixed. Otherwise, post a comment in the issue saying that it wasn't fixed.
Link(s)
Accessibility (a11y) RC Test
What to Test
Focus and Special Instructions
Test keyboard nav and interaction.
Link(s)
Screen Readers
This sim supports screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to screen readers. If you simply need a refresher on screen readers, please consult the QA Book, which should have all of the information you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.
Platforms and Screen Readers to Be Tested
Critical Screen Reader Information
We are tracking known screen reader bugs in this Google Document. If you find a screen reader bug, please check it against this list.
Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a screen reader.
FAQs for QA Members
There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.
How should I format my issue?
Here's a template for making issues:
Who should I assign?
We typically assign the developer who opened the issue in the QA repository.
My question isn't in here... What should I do?
You should:
The text was updated successfully, but these errors were encountered: