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.5 #221

Closed
2 of 3 tasks
mbarlow12 opened this issue Nov 13, 2018 · 7 comments
Closed
2 of 3 tasks

RC test: Coulomb's Law 1.0.0-rc.5 #221

mbarlow12 opened this issue Nov 13, 2018 · 7 comments
Assignees

Comments

@mbarlow12
Copy link

mbarlow12 commented Nov 13, 2018

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

The latest round of issues were pretty minor, so I think just briefly checking major browsers/platforms should be sufficient.

Assigning @ariel-phet for prioritization.

General RC Test

What to Test

  • Click every single button.
  • 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.)
  • Complete the test matrix.
  • Check this LoL spreadsheet and notify AR if it not there.

Issues to Verify

Though it's closed, @KatieWoe mentioned in #214 (comment) about the ResetAll button pointer area.

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

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

Link(s)

Keyboard Navigation

This sim supports keyboard navigation. Please make sure it works as intended.


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.


@KatieWoe
Copy link
Contributor

KatieWoe commented Nov 13, 2018

Since this is a spot check, per @mbarlow12 there is not a test matrix. Please do a full rc test on platforms marked full, and about half of the normal tests, including testing all issues, for platforms marked partial. @mbarlow12 let me know if this list works for you.

  • Win 10 chrome full
  • Win 10 firefox partial
  • Win 10 edge full

  • Mac 10.14 chrome partial
  • Mac 10.14 firefox full
  • Mac 10.14 safari full

  • iOS 12 full
  • iOS 11 partial

  • chrome os partial

  • Win 7 IE full
  • Win 7 chrome partial
  • Mac 10.12 safari partial
  • Mac 10.12 firefox partial

@mbarlow12
Copy link
Author

@KatieWoe this looks excellent. Thank you.

@KatieWoe
Copy link
Contributor

Here are the results of a memory test
Start: 27.4
1 min: 35.4
2 min: 36.5
3 min: 37.1
4 min: 37.3
5 min: 37.4
6 min: 37.7
7 min: 37.8
8 min: 38
9 min: 38.2
10 min: 38.3
20 min: 39
30 min: 39.4
40 min: 39.5
50 min: 39.6
60 min: 39.7

@ariel-phet
Copy link
Contributor

Moving to high priority to be finished off

@KatieWoe
Copy link
Contributor

Issue for keyboard nav that was encountered on Edge in the XHTML version but was unable to replicate. The figure on the right in Macro Scale was unable to move to the left past a certain point. I had been in full screen before, the charge of the figure in question was 0, and the point it stopped was a bit to the right of the the middle of the screen. Dragging by mouse was not effected. Unfortunately, I haven't been able to replicate. @jessegreenberg and @mbarlow12 I didn't make an issue since have so little information, but I'm documenting here. Let me know if you want a separate issue made.

@KatieWoe
Copy link
Contributor

KatieWoe commented Nov 28, 2018

QA is done @mbarlow12

@mbarlow12
Copy link
Author

New RC issue at #235.

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

4 participants