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

RC test: Coulomb's Law 1.0.0-rc.4 #214

Closed
3 tasks done
mbarlow12 opened this issue Nov 1, 2018 · 8 comments
Closed
3 tasks done

RC test: Coulomb's Law 1.0.0-rc.4 #214

mbarlow12 opened this issue Nov 1, 2018 · 8 comments
Assignees

Comments

@mbarlow12
Copy link

mbarlow12 commented Nov 1, 2018

@KatieWoe, @arouinfar, @jessegreenberg, coulombs-law/1.0.0-rc.4 is ready for RC testing. Document issues in https://github.com/phetsims/coulombs-law/issues and link to this issue.

Assigning @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 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 LoL spreadsheet and notify AR if it not there.

Issues to Verify

NOTE: keyboard dialog issue has been closed, but I thought it'd be useful to confirm.

Link(s)


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).

Link(s)

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:

  <b>Test Device</b>

  blah

  <b>Operating System</b>

  blah

  <b>Browser</b>

  blah

  <b>Problem Description</b>

  blah

  <b>Steps to Reproduce</b>

  blah

  <b>Visuals</b>

  blah

  <details>
  <summary><b>Troubleshooting Information</b></summary>

  blah

  </details>

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.
  2. Google it.
  3. Ask Katie.
  4. Ask a developer.
  5. Google it again.
  6. Cry.


@ariel-phet
Copy link
Contributor

@KatieWoe top testing priority for the moment

Mentioning @rea-laura @phet-steele @JRomero0613 @lmulhall-phet since Katie is at a conference

@JRomero0613
Copy link

The test matrix link provided does not direct to the test matrix and instead directs to a blank github page. Could a new link be provided so testing progress can be updated?

@ghost
Copy link

ghost commented Nov 1, 2018

I’ll make a matrix in a second! Sorry about that, @JRomero0613.

@ghost
Copy link

ghost commented Nov 1, 2018

Link should be working now.

@KatieWoe
Copy link
Contributor

KatieWoe commented Nov 7, 2018

QA is done.

@KatieWoe
Copy link
Contributor

KatieWoe commented Nov 7, 2018

Sorry, @mbarlow12, noticed one last thing. Apparently the pointer areas for the reset button for touch should be larger. https://github.com/phetsims/friction/issues/65

@mbarlow12
Copy link
Author

mbarlow12 commented Nov 8, 2018

Only issues remaining after latest commits are:
phetsims/scenery#890
and
phetsims/scenery#883

@mbarlow12
Copy link
Author

Closing. New RC issue at #221

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants