Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(76)

Unified Diff: extensions/test/data/api_test/audio/remove_nodes/background.js

Issue 2462833002: Refactor audio API tests (Closed)
Patch Set: Refactor audio API tests Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/test/data/api_test/audio/remove_nodes/background.js
diff --git a/extensions/test/data/api_test/audio/remove_nodes/background.js b/extensions/test/data/api_test/audio/remove_nodes/background.js
index b82c99ed20c97efb47272fa01e39d53e6248a152..2eb1e35d82e6cf110fabbd9837570023f9e1eb04 100644
--- a/extensions/test/data/api_test/audio/remove_nodes/background.js
+++ b/extensions/test/data/api_test/audio/remove_nodes/background.js
@@ -2,32 +2,39 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-chrome.audio.OnDevicesChanged.addListener(function (devices) {
- if (devices.length === 2) {
- if (devices[0].id != "40001" ||
- devices[0].stableDeviceId != "90001" ||
- devices[0].isInput != true ||
- devices[0].deviceType != "USB" ||
- devices[0].deviceName != "Jabra Mic" ||
- devices[0].displayName != "Jabra Mic 1") {
- console.error("Got wrong device property for device:" +
- JSON.stringify(devices[0]));
- chrome.test.sendMessage("failure");
- }
- if (devices[1].id != "40002" ||
- devices[1].stableDeviceId != "90002" ||
- devices[1].isInput != true ||
- devices[1].deviceType != "USB" ||
- devices[1].deviceName != "Jabra Mic" ||
- devices[1].displayName != "Jabra Mic 2") {
- console.error("Got wrong device property for device:" +
- JSON.stringify(devices[1]));
- chrome.test.sendMessage("failure");
- }
- chrome.test.sendMessage("success");
- } else {
- console.error("Got unexpected OnNodesChanged event failed");
- chrome.test.sendMessage("failure");
+chrome.test.runTests([
+ function waitForDeviceChangedEventTests() {
+ chrome.test.listenOnce(chrome.audio.OnDevicesChanged, function (devices) {
+ var deviceList = devices.map(function(device) {
+ return {
+ id: device.id,
+ stableDeviceId: device.stableDeviceId,
+ isInput: device.isInput,
+ deviceType: device.deviceType,
+ deviceName: device.deviceName,
+ displayName: device.displayName
+ };
+ }).sort(function(lhs, rhs) {
+ return Number.parseInt(lhs.id) - Number.parseInt(rhs.id);
+ });
+
+ chrome.test.assertEq([{
+ id: '40001',
+ stableDeviceId: '90001',
+ isInput: true,
+ deviceType: 'USB',
+ deviceName: 'Jabra Mic',
+ displayName: 'Jabra Mic 1'
+ }, {
+ id: '40002',
+ stableDeviceId: '90002',
+ isInput: true,
+ deviceType: 'USB',
+ deviceName: 'Jabra Mic',
+ displayName: 'Jabra Mic 2'
+ }], deviceList);
+ });
}
-});
-chrome.test.sendMessage("loaded");
+]);
+
+chrome.test.sendMessage('loaded');
« no previous file with comments | « extensions/test/data/api_test/audio/output_mute_change/background.js ('k') | extensions/test/data/api_test/audio/test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698