Team testing prior to RC

phetsims / fourier-making-waves

"Fourier: Making Waves" is an educational simulation in HTML5, by PhET Interactive Simulations.

GNU General Public License v3.0

6 stars 3 forks source link

Team testing prior to RC #178

Closed pixelzoom closed 3 years ago

pixelzoom commented 3 years ago

Dev-lit testing https://github.com/phetsims/qa/issues/701 and code review #165 did not reveal any problems with this sim. We're in excellent shape, so we're going to skip dev testing and go right to RC.

Before RC, the consensus was that the team members would exercise the sim. When you have completed testing, please check your name off below. Also please indicate which platforms you tested. If you find any new problems, please create new issues, and link them to this issue.

[x] @arouinfar
[x] @kathy-phet
[x] @ariel-phet
[x] @pixelzoom
[x] QA team

Please complete this task by the end of the day on Monday, 9/27.

kathy-phet commented 3 years ago

A few things I found:

[ ] for the amplitude sliders, when you click in the track, keep mouse down, and move up and down, you do not get the clicking slider sounds (I don't naturally do this on the computer version, but this happens a lot on a touch pad device like iPads)
[ ] Design question (@arouinfar) - Should "Infinite Harmonics" checkbox be available when on "Sinusoidal". It feels like it should be disabled for Sinusoidal, just like it is for custom.
[ ] (iPad only bug?) when Component spacing is set to 0 spacing, k1 (yellow) shows zero which is correct, but a small Lambda1 green box in the lower right of the screen (seems like this is a bug/unitentional). There is also inconsistency as I change the component spacing. I can get the lamda1=infinity to be on the graph. But if I reset, then change spacing to 0, then check k1 and lambda1. I don't see any of this on my Windows machine.

(Gotta go, spent about 30 minutes on this)

pixelzoom commented 3 years ago

From https://github.com/phetsims/fourier-making-waves/issues/178#issue-1005939816:

If you find any new problems, please create new issues, and link them to this issue.

I have created GitHub issues for the checklist items in https://github.com/phetsims/fourier-making-waves/issues/178#issuecomment-926970316. Please do the same for anything else that you find. Putting comments in this issue creates additional work for me, and makes it more difficult to discuss/track these problems.

ariel-phet commented 3 years ago

I finished my testing, I will create a few issues -- nothing is a "show stopper" but there are a few minor issues usability issues that I will document. The wave game works great now, no issues on that screen.

pixelzoom commented 3 years ago

... please check your name off below. Also please indicate which platforms you tested.

@ariel-phet could you please list the platform(s) that you tested? I'd like to know (and document) our coverage.

kathy-phet commented 3 years ago

I invited the QA team to do some testing on master too - to flesh out as many issues as possible in this rapid test.

pixelzoom commented 3 years ago

I've completed my testing, on these devices:

MacBookPro16,1 + macOS 11.5 + Chrome 94.0.4606.61 MacBookPro16,1 + macOS 11.5 + Safari 14.1.2 (16611.3.10.1.3) iPad6 + iOS 14.8 + mobile Safari

pixelzoom commented 3 years ago

I invited the QA team to do some testing on master too - to flesh out as many issues as possible in this rapid test.

@KatieWoe when QA is done, please check the "QA team" item above, and comment on which platforms (machine + OS + browser) you tested.

ariel-phet commented 3 years ago

@pixelzoom testing was on Windows 10 chrome (fast desktop) and no other platforms

KatieWoe commented 3 years ago

iPadOS 15, MacBook Air (m1 chip) 11.5.2 + chrome 93.0.4577.82 and safari 14.1.2, and win10 chrome for various members of QA

arouinfar commented 3 years ago

Here are the configurations I tested: Mid-2014 MacBook Pro, macOS 10.15.7 + Chrome 94.0.4606.54 Mid-2014 MacBook Pro, macOS 10.15.7 + Safari 14.0.2 (15610.3.7.1.10, 15610)

I found one issue with the focus order, referenced above.

kathy-phet commented 3 years ago

Here are my systems: Dell Latitude 7490, Microsoft Windows 10, Version 10.0.19042 Build 19042, Google Chrome Version 93.0.4577.82 (Official Build) (64-bit) iPad (6th generation) iOS 14.4.2 and Safari 14 (Catalina) 14.0.3

pixelzoom commented 3 years ago

Looks like we had decent coverage on a variety of platforms.

I've completed work on all related issues, mostly "polish" stuff. Those issues have either been closed, or are ready for review. When they have all been closed, I'll proceed with RC publications.

Closing.