commit 8d39c50cd5d746778f911c2e84d666ea967193ae
author Yu-Hsuan Hsu <yuhsuan@google.com> Thu Nov 22 16:58:30 2018 +0800
committer chrome-bot <chrome-bot@chromium.org> Thu Nov 22 19:07:59 2018 -0800
tree 2c71aeb12b1ae56cbfe7ac4694047f3e722d939b
parent 3f24ac2f5dfe650e40b82b887ca86080fc692bf5

[autotest] audio_spec - Add list of boards without sound card

Some other devices do not have sound card and make test fail (such as
gale and veyron_rialto). We need to take them into account.

BUG=chromium:816823
TEST=gale and veyron_rialto can pass audio_Aplay test.

Change-Id: I0b3591f872bbb8240c701d52bbfa967419b870ec
Reviewed-on: https://chromium-review.googlesource.com/1347930
Commit-Ready: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Tested-by: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>

client/cros/audio/audio_spec.py

diff --git a/client/cros/audio/audio_spec.py b/client/cros/audio/audio_spec.py
index 32c8658..aa6bea7 100644
--- a/client/cros/audio/audio_spec.py
+++ b/client/cros/audio/audio_spec.py
@@ -6,6 +6,7 @@
 
 _BOARD_TYPE_CHROMEBOX = 'CHROMEBOX'
 _BOARD_TYPE_CHROMEBIT = 'CHROMEBIT'
+_BOARD_WITHOUT_SOUND_CARD = ['gale', 'veyron_rialto']
 
 def has_internal_speaker(board_type, board_name):
     """Checks if a board has internal speaker.
@@ -17,7 +18,8 @@
 
     """
     if ((board_type == _BOARD_TYPE_CHROMEBOX and board_name != 'stumpy')
-            or board_type == _BOARD_TYPE_CHROMEBIT):
+            or board_type == _BOARD_TYPE_CHROMEBIT
+            or board_name in _BOARD_WITHOUT_SOUND_CARD):
         return False
     return True