From 75486c1046f6f5af1af26628823d570bad2c3926 Mon Sep 17 00:00:00 2001 From: Geoff Brown Date: Fri, 9 May 2014 06:43:21 -0600 Subject: [PATCH] Bug 1004668 - Update robocop testUITelemetry; r=rnewman --- mobile/android/base/tests/robocop.ini | 2 -- mobile/android/base/tests/testUITelemetry.js | 15 ++++++++------- 2 files changed, 8 insertions(+), 9 deletions(-) diff --git a/mobile/android/base/tests/robocop.ini b/mobile/android/base/tests/robocop.ini index 132b59cb119d..c1da19d075b3 100644 --- a/mobile/android/base/tests/robocop.ini +++ b/mobile/android/base/tests/robocop.ini @@ -119,8 +119,6 @@ skip-if = android_version == "10" [testSharedPreferences] [testSimpleDiscovery] [testUITelemetry] -# disabled on 2.2, see bug 993813 -skip-if = android_version == "8" [testVideoDiscovery] # Used for Talos, please don't use in mochitest diff --git a/mobile/android/base/tests/testUITelemetry.js b/mobile/android/base/tests/testUITelemetry.js index 5ff8d452b014..850843ca5365 100644 --- a/mobile/android/base/tests/testUITelemetry.js +++ b/mobile/android/base/tests/testUITelemetry.js @@ -48,11 +48,11 @@ add_test(function test_telemetry_events() { let expected = [ ["event", TEST_PREFIX + "enone", "method0", [], null], - ["event", TEST_PREFIX + "efoo", "method1", ["foo"], null], - ["event", TEST_PREFIX + "efoo", "method2", ["foo"], null], - ["event", TEST_PREFIX + "efoobar", "method3", ["foo", "bar"], "foobarextras"], + ["event", TEST_PREFIX + "efoo", "method1", [TEST_PREFIX + "foo"], null], + ["event", TEST_PREFIX + "efoo", "method2", [TEST_PREFIX + "foo"], null], + ["event", TEST_PREFIX + "efoobar", "method3", [TEST_PREFIX + "foo", TEST_PREFIX + "bar"], "foobarextras"], ["session", TEST_PREFIX + "foo", "reasonfoo"], - ["event", TEST_PREFIX + "ebar", "method4", ["bar"], "barextras"], + ["event", TEST_PREFIX + "ebar", "method4", [TEST_PREFIX + "bar"], "barextras"], ["session", TEST_PREFIX + "bar", "reasonbar"], ["event", TEST_PREFIX + "enone", "method5", [], null], ]; @@ -62,12 +62,13 @@ add_test(function test_telemetry_events() { for (let i = 0; i < measurements.length; ++i) { let m = measurements[i]; - let type = m[0]; + let type = m.type; if (type == "event") { let [type, action, method, sessions, extras] = expected[i]; do_check_eq(m.action, action); do_check_eq(m.method, method); - do_check_array_eq(m.sessions, sessions); + // might receive real sessions in addition to the test ones -- remove the real ones + do_check_array_eq(m.sessions.filter(s => TEST_REGEX.test(s)), sessions); do_check_eq(m.extras, extras); continue; } @@ -75,7 +76,7 @@ add_test(function test_telemetry_events() { if (type == "session") { let [type, name, reason] = expected[i]; do_check_eq(m.name, name); - do_check_eq(m.reason, method); + do_check_eq(m.reason, reason); continue; } }