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: Balancing Chemical Equations 1.2.0-rc.1 #237

Closed
2 tasks done
pixelzoom opened this issue Dec 10, 2018 · 4 comments
Closed
2 tasks done

RC test: Balancing Chemical Equations 1.2.0-rc.1 #237

pixelzoom opened this issue Dec 10, 2018 · 4 comments
Assignees

Comments

@pixelzoom
Copy link
Contributor

pixelzoom commented Dec 10, 2018

@KatieWoe , @arouinfar, @ariel-phet, Balancing Chemical Equations 1.2.0-rc.1 is ready for RC testing.

This RC is for brand=phet only (no PhET-io, no a11y), and it will not be provided to a client. The deadline mentioned in phetsims/balancing-chemical-equations#130 is "March", presumably 2019.

Document issues in https://github.com/phetsims/balancing-chemical-equations/issues and link to this issue.

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 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.
  • If this is rc.1 please do a memory test.

Focus and Special Instructions

According to phetsims/balancing-chemical-equations#130, the primary motivation for this release is ePub and LoL support. Secondary is to pick up general common-code enhancements, and sim-specific changes identified in "Issues to Verify".

Sim-specific query parameters can be found in BCEQueryParameters. playAll and showReward are both useful for testing.

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)


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

ariel-phet commented Dec 11, 2018

@KatieWoe as discussed over Slack, this sim is aiming to be republished by March. Several high priority tests are likely to come down the pipe, but potentially not until next week, so adding this to the queue, to hopefully get it done sooner than later.

@ariel-phet ariel-phet removed their assignment Dec 11, 2018
@KatieWoe
Copy link
Contributor

Memory Tests:
Start: 20.5 MB
1 min: 59.7 MB
2 min: 83.5 MB
3 min: 104 MB
4 min: 122 MB
5 min: 145 MB
6 min: 174 MB
7 min: 202 MB
8 min: 219 MB
9 min: 238 MB
10 min: Memory tab crashed when trying to take snapshot

@KatieWoe
Copy link
Contributor

QA is done.

@pixelzoom
Copy link
Contributor Author

Thanks @KatieWoe and QA team. Moving on to 1.2.0-rc.2, see #255.

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

3 participants