[test_build] meta-iotqa: add qa tests for audio #288
Conversation
|
please test. |
|
Can one of the admins verify this patch? |
|
ok to test |
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
|
can you share hw card and device number to record and playback ? |
Signed-off-by: nvelux <[email protected]>
|
please test pulse audio recording and playback. |
| time.sleep(3) | ||
| self.assertEqual(status, 0, msg="Error not recorded: %s" % output) | ||
| # Stop pulseaudio daemon | ||
| (status, output) = self.target.run("pulseaudio -k") |
There was a problem hiding this comment.
I know that parecord doesn't have time parameter to stop the recording, but maybe you could dig out or save the previously started parecord process id and kill thaq (instead of killing pulseaudio server)
| (status, output) = self.target.run("pulseaudio -D") | ||
| self.assertEqual(status, 0, msg="Error pulseaudio not started: %s" % output) | ||
| # Checking recorded file present | ||
| (status, output) = self.target.run("ls /tmp/ |grep 'rec.wav'") |
There was a problem hiding this comment.
Do you have to do the grep "rec.wav", could you just ls?
There was a problem hiding this comment.
just confirming the recorded file present.
| (status, output) = self.target.run("ls /tmp/ |grep 'rec.wav'") | ||
| self.assertEqual(status, 0, msg="Error file not found: %s" % output) | ||
| # Playing audio | ||
| (status, output) = self.target.run("paplay /tmp/rec.wav &") |
There was a problem hiding this comment.
Have you actually tried that you can record and play the audio with these commands (like in your own pc)?
There was a problem hiding this comment.
yes, it's playing the audio.
| self.assertEqual(status, 0, msg="Error not played: %s" % output) | ||
| # Audio running states checking | ||
| (status, output) = self.target.run("cat /proc/asound/card1/pcm3p/sub0/status |grep 'state: RUNNING'") | ||
| self.assertEqual(status, 0, msg="Error not running: %s" % output) |
There was a problem hiding this comment.
I'm not sure how portable the alsa card path here is...? You could do "pactl list sinks" and grep for "RUNNING" and so on.
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
|
corrected as per your suggestion , please test. |
|
Apply proper time delay to copy sinks file, please test. |
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
|
"paplay /tmp/rec.wav &" command is working here in own PC and state: RUNNING. |
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
|
Script is running successfully as "State: RUNNING" in own PC here. |
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
|
please can you check "paplay /tmp/rec.wav &" command issue ? it's running successfully in own PC, and state is RUNNING as well. |
|
@nvelux why this test case was separated from oeqa.runtime.multimedia.audio.pulseaudio? Why not include this new test case into existing pulseaudio test? |
|
it's included, please test. |
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Signed-off-by: nvelux <[email protected]>
Add test for audio python interface