Improve the alsa_record_playback_automated test for hearing protection

Bug #1553874 reported by Po-Hsu Lin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Invalid
Undecided
Unassigned
OEM QA Checkbox additions
Fix Released
Undecided
Unassigned
Provider for Plainbox - Canonical Certification (Legacy)
Fix Released
Undecided
Unassigned

Bug Description

As you may hear the high pitching beep sound during our daily stand-up meeting, it was came from the alsa_record_playback_automated and suspend/record_playback_after_suspend test.

I think it will be great to improve this test case for our QA colleagues.
It's not a pleasant thing if you need to sit in the lab and hear that beep sound for many times, which is designed for the machine itself, not human.

Or we can remove it from the manual test plan, as we have already ask testers to check the audio input / output capability, and leave it as-is for automated test plans like SRU.

Revision history for this message
Daniel Manrique (roadmr) wrote :

The frequency can be changed to something less painful. The one we're using was chosen because it seemed to be pretty reliable, but lower frequencies will be friendlier to the ears.

You could run the audio_test with -f and experiment with values to see what works. The one we're using is 5035Hz. I would suggest choosing a value and trying it several times to ensure detection is reliable with that value, once you find one you like, can add the -f parameter to the job definitions or even change it in the script itself (DEFAULT_TEST_FREQUENCY parameter).

Revision history for this message
Pierre Equoy (pieq) wrote :

I'm not sure if this test is useful in a manually controlled environment (i.e. when test is done by a QA engineer), as there are already manual tests in place to ensure both microphone input and headphone outputs work correctly.

There, I would +1 Sam's suggestion to remove these tests from the test plans used by QA engineers or by Cert engineers (but we would keep them for SRU test plans).

Revision history for this message
Jerry Kao (jerry.kao) wrote :

I am not sure the purpose of this test. If it is for functionality of internal microphone and speaker, it should already be covered by other test cases (ie alsa_record_playback_internal, playback_auto)
I agree to remove it if there is no other specific test purpose.

Pierre Equoy (pieq)
Changed in plainbox-provider-checkbox:
milestone: none → 0.28
milestone: 0.28 → none
Changed in plainbox-provider-canonical-certification:
milestone: none → 0.26
milestone: 0.26 → 0.27
Revision history for this message
Pierre Equoy (pieq) wrote :

I pushed a fix for the OEM providers (it will be available next time we do a release).

I also did a merge proposal for the Certification provider, it is available here:

https://code.launchpad.net/~pierre-equoy/plainbox-provider-canonical-certification/+git/plainbox-provider-canonical-certification/+merge/292606

Changed in plainbox-provider-checkbox:
status: New → Invalid
Changed in plainbox-provider-canonical-certification:
status: New → Confirmed
status: Confirmed → Triaged
Changed in oem-qa-checkbox:
status: New → Triaged
status: Triaged → Fix Committed
Changed in plainbox-provider-canonical-certification:
status: Triaged → Fix Committed
Pierre Equoy (pieq)
Changed in plainbox-provider-canonical-certification:
status: Fix Committed → Fix Released
Changed in oem-qa-checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.