phetsims / qa

Quality Assurance Task Tracking
MIT License
13 stars 8 forks source link

RC test: Gravity Force Lab 2.2.0-rc.2 #480

Closed zepumph closed 4 years ago

zepumph commented 4 years ago

@emily-phet @terracoda @jessegreenberg @arouinfar @kathy-phet @samreid @chrisklus, gravity-force-lab/2.2.0-rc.2 is ready for RC testing. Document issues in https://github.com/phetsims/gravity-force-lab/issues and link to this issue.

This simulation is outfitted with:

Assigning to @ariel-phet for prioritization.

General RC Test

What to Test

- Click every single button. - Test all possible forms of input. - Test all mouse/trackpad inputs. - Test all touchscreen inputs. - If there is sound, make sure it works. - Make sure you can't lose anything. - Play with the sim normally. - Try to break the sim. - Test all query parameters on all platforms. (See [QA Book](https://github.com/phetsims/QA/blob/master/doc/qa-book.md) for a list of query parameters.) - Download HTML on Chrome and iOS. - Make sure the iFrame version of the simulation is working as intended on all platforms. - Make sure the XHTML version of the simulation is working as intended on all platforms. - Complete the test matrix. - Don't forget to make sure the sim works with Legends of Learning. - Check [this](https://docs.google.com/spreadsheets/d/1umIAmhn89WN1nzcHKhYJcv-n3Oj6ps1wITc-CjWYytE/edit#gid=0) LoL spreadsheet and notify AR or AM if it not there. - If this is rc.2 please do a memory test. - When making an issue, check to see if it was in a previously published version - If there is a console available, check for errors and include them in the Problem Description. - As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if there is a problem.

Issues to Verify

- [x] https://github.com/phetsims/studio/issues/94 - [x] https://github.com/phetsims/phet-io-wrappers/issues/319 - [x] https://github.com/phetsims/balancing-act/issues/111 - [x] https://github.com/phetsims/studio/issues/92 - [x] https://github.com/phetsims/studio/issues/19 - [x] https://github.com/phetsims/phet-io-wrappers/issues/321 - [x] https://github.com/phetsims/gravity-force-lab/issues/248 - [x] https://github.com/phetsims/gravity-force-lab/issues/246 - [x] https://github.com/phetsims/gravity-force-lab/issues/245 - [x] https://github.com/phetsims/gravity-force-lab/issues/244 - [x] https://github.com/phetsims/gravity-force-lab/issues/242 - [x] https://github.com/phetsims/gravity-force-lab/issues/240 - [x] https://github.com/phetsims/gravity-force-lab/issues/100 - [ ] https://github.com/phetsims/gravity-force-lab/issues/180 These issues should have the "status:ready-for-qa" 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 and link back to this issue. If the label is "status:ready-for-review" or "status:fixed-pending-deploy" then assign back to the developer when done, even if fixed.

Link(s)

- **[Simulation](https://phet-dev.colorado.edu/html/gravity-force-lab/2.2.0-rc.2/phet/gravity-force-lab_all_phet.html)** - **[iFrame](https://phet-dev.colorado.edu/html/gravity-force-lab/2.2.0-rc.2/phet/gravity-force-lab_all_iframe_phet.html)** - **[XHTML](https://phet-dev.colorado.edu/html/gravity-force-lab/2.2.0-rc.2/phet/xhtml/gravity-force-lab_all.xhtml)** - **[Test Matrix](https://docs.google.com/spreadsheets/d/1dPp2zsj4WW577vQNgCnx9d3cPNxh_JrJj_EQ8CaNqOc/edit#gid=860975363)** - **[Legends of Learning Harness](https://developers.legendsoflearning.com/public-harness/index.html?startPayload=%7B%22languageCode%22%3A%22en%22%7D)**
PhET-iO RC Test

What to Test

- Make sure that public files do not have password protection. Use a private browser for this. - Make sure that private files do have password protection. Use a private browser for this. - Make sure standalone sim is working properly. - Make sure the wrapper index is working properly. - Make sure each wrapper is working properly. - Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube - For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make sure the simulation loads without crashing or throwing errors. - Load the login wrapper just to make sure it works. Do so by adding this link from the sim deployed root: ``` /wrappers/login/?wrapper=record&validationRule=validateDigits&&numberOfDigits=5&promptText=ENTER_A_5_DIGIT_NUMBER ``` Further instructions in QA Book - Please at least load all wrappers except studio in the wrapper suite with internet explorer. - make sure to do a complete metacog recording/playback test. Ask @zepumph if you need help. (this doesn't need to be done as much this time around. Make sure to do a test, and any others relating to issues to verify)

Link(s)

- **[Wrapper Index](https://phet-dev.colorado.edu/html/gravity-force-lab/2.2.0-rc.2/phet-io/)** - **[Test Matrix](https://docs.google.com/spreadsheets/d/15Galf54GoI--GuRreo_p-hzIsZ7MlitRmiwc4dIUctg/edit#gid=1712810511)**
Accessibility (a11y) RC Test

What to Test

- Specific instructions can be found below. - Make sure the a11y feature doesn't negatively affect the sim in any way. - Test all possible forms of input. - Test all mouse/trackpad inputs. - Test all touchscreen inputs. - Test all keyboard navigation inputs (if applicable). - Test all forms of input with a screen reader (if applicable).

Issues to Verify

- [x] [Might be an issue with the radio button group](https://github.com/phetsims/gravity-force-lab/issues/223) These issues should have the "status:ready-for-qa" 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 and link back to this issue. If the label is "status:ready-for-review" or "status:fixed-pending-deploy" then assign back to the developer when done, even if fixed.

Link(s)

- **[a11y View](https://phet-dev.colorado.edu/html/gravity-force-lab/2.2.0-rc.2/phet/gravity-force-lab_a11y_view.html)** - **[Simulation](https://phet-dev.colorado.edu/html/gravity-force-lab/2.2.0-rc.2/phet/gravity-force-lab_all_phet.html)** - **[Test Matrix](https://docs.google.com/spreadsheets/d/1dPp2zsj4WW577vQNgCnx9d3cPNxh_JrJj_EQ8CaNqOc/edit#gid=860975363)**

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](link), 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

- [x] Windows 10 + Latest Chrome + Latest JAWS - [x] Windows 10 + Latest Firefox + Latest NVDA - [x] macOS + Safari + VoiceOver - [x] iOS + Safari + VoiceOver (Please test this! Especially with the Ruler interaction)

Critical Screen Reader Information

We are tracking known screen reader bugs in [this Google Document](https://drive.google.com/open?id=12kTs-g-iKEIH1dyG7Q41_W_oNL4gUKbkW-IQgZjMUBw). 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.

Final Requests

- [x] If this sim is in [this list](https://github.com/phetsims/perennial/blob/master/data/accessibility), make an issue in [website](https://github.com/phetsims/website) to ask if PhET research page links need updating. Please assign to @terracoda and @emily-phet.
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: Test Device blah Operating System blah Browser blah Problem Description blah Steps to Reproduce blah Visuals blah
Troubleshooting Information blah

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: 1. Consult the [QA Book](https://github.com/phetsims/QA/blob/master/doc/qa-book.md). 2. Google it. 3. Ask Katie. 4. Ask a developer. 5. Google it again. 6. Cry.


ariel-phet commented 4 years ago

@KatieWoe priority supersedes #470

KatieWoe commented 4 years ago

@zepumph I noticed that you can drag the ruler from behind the slider boxes. Doesn't cause any problems and I don't think it needs anything done, but if you disagree and want an issue, let me know. shoulddrag

zepumph commented 4 years ago

That seems like a bug, and an issue would be nice here.

KatieWoe commented 4 years ago

QA is done

zepumph commented 4 years ago

I still don't see https://github.com/phetsims/balancing-act/issues/111 checked off, did you get to it? Should I reassign it to someone?

KatieWoe commented 4 years ago

Sorry, I thought one of my workers was looking at it, so I passed it over. I'll check quick.

KatieWoe commented 4 years ago

Done

KatieWoe commented 4 years ago

Reopening for metacog

zepumph commented 4 years ago

@samreid and I don't think that metacog testing is necessary for this second RC. We did a lot of metacog testing last RC, and the only issues that came from that were those that were general record/playback issues and could be confirmed from localFile recording.

zepumph commented 4 years ago

Alright we are ready for a third rc/spot-check. I will go ahead and close this and create a new RC.