New issue
Advanced search Search tips

Issue 884099 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Use Tast to run alsa conformance test

Project Member Reported by yuhsuan@chromium.org, Sep 14

Issue description

We need to deploy alsa_conformance_test to autotest by using Tast. It will be helpful to monitor this value.

Alsa_conformance_test design doc
https://docs.google.com/document/d/1U1FGeu_7dm6QAvWybm7uaXQtnPFFE4TYqsJtlj3vOBQ/edit?usp=sharing

Alsa_conformance_test guide
https://docs.google.com/document/d/1uA1rChj7ZFg3zO31kpKfKpS5x8zbHUl0Y_IP4YB12YY/edit?usp=sharing
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 10

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/674a855416d05100d86c2967d33da5adcea2aa33

commit 674a855416d05100d86c2967d33da5adcea2aa33
Author: Yu-Hsuan Hsu <yuhsuan@google.com>
Date: Thu Jan 10 12:49:54 2019

tast: Add AlsaConformance into tast

Deploy alsa_conformance_test to tast.

BUG=chromium:884099
TEST=tast -verbose run $DUT_IP audio.AlsaConformance
CQ-DEPEND=CL:1397596

Change-Id: I3941063d2a6e6666dfeaca515c9cd4f0796d1124
Reviewed-on: https://chromium-review.googlesource.com/1395904
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>

[add] https://crrev.com/674a855416d05100d86c2967d33da5adcea2aa33/src/chromiumos/tast/local/bundles/cros/audio/alsa_conformance.go

Project Member

Comment 2 by bugdroid1@chromium.org, Jan 16 (6 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/c3729f7e7124ffeaa2354f9a88eb91c5590dac81

commit c3729f7e7124ffeaa2354f9a88eb91c5590dac81
Author: Yu-Hsuan Hsu <yuhsuan@google.com>
Date: Wed Jan 16 09:46:52 2019

audio.ALSAConformance: Stop CRAS before running test

If we run ALSA test while another stream is playing in CRAS, the test won't
get audio device so that it will return error. We should stop CRAS before
running to make sure this situation will not happen.

BUG=chromium:884099
TEST=tast -verbose run $DUT_IP audio.AlsaConformance

Change-Id: Id458d403f5cc84dc89940b34bd7e6c7c8c9d4c4b
Reviewed-on: https://chromium-review.googlesource.com/1407137
Commit-Ready: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Tested-by: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Reviewed-by: Shuhei Takahashi <nya@chromium.org>

[modify] https://crrev.com/c3729f7e7124ffeaa2354f9a88eb91c5590dac81/src/chromiumos/tast/local/bundles/cros/audio/alsa_conformance.go

Project Member

Comment 3 by bugdroid1@chromium.org, Jan 19 (4 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/069a3774fab5df41a5f3e4beb19e28a2c126ea9f

commit 069a3774fab5df41a5f3e4beb19e28a2c126ea9f
Author: Yu-Hsuan Hsu <yuhsuan@google.com>
Date: Sat Jan 19 08:14:03 2019

audio.ALSAConformance: Not finish until UI selects active nodes

After CRAS starts, UI will select input and output nodes automatically. If we
run another test just after CRAS restarts, these nodes will not be selected and
the test will fail. We need to wait until UI selects active nodes.

BUG=chromium:884099
TEST=tast -verbose run $DUT_IP audio.ALSAConformance audio.Microphone

Change-Id: Ib784d90e231c1f12904761b52d1002dad7ff5f4a
Reviewed-on: https://chromium-review.googlesource.com/1420417
Commit-Ready: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Tested-by: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>

[modify] https://crrev.com/069a3774fab5df41a5f3e4beb19e28a2c126ea9f/src/chromiumos/tast/local/bundles/cros/audio/alsa_conformance.go

Sign in to add a comment