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

Side by Side Diff: chrome/android/javatests/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridgeTest.java

Issue 2877253002: Convert Notification Tests to JUnit4 (Closed)
Patch Set: address comments Created 3 years, 7 months 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.notifications; 5 package org.chromium.chrome.browser.notifications;
6 6
7 import static org.chromium.base.test.util.ScalableTimeout.scaleTimeout; 7 import static org.chromium.base.test.util.ScalableTimeout.scaleTimeout;
8 8
9 import android.annotation.TargetApi; 9 import android.annotation.TargetApi;
10 import android.app.Notification; 10 import android.app.Notification;
11 import android.app.PendingIntent; 11 import android.app.PendingIntent;
12 import android.app.RemoteInput; 12 import android.app.RemoteInput;
13 import android.content.Context; 13 import android.content.Context;
14 import android.content.Intent; 14 import android.content.Intent;
15 import android.graphics.Bitmap; 15 import android.graphics.Bitmap;
16 import android.graphics.BitmapFactory; 16 import android.graphics.BitmapFactory;
17 import android.graphics.Color; 17 import android.graphics.Color;
18 import android.os.Build; 18 import android.os.Build;
19 import android.os.Bundle; 19 import android.os.Bundle;
20 import android.support.test.InstrumentationRegistry;
20 import android.support.test.filters.LargeTest; 21 import android.support.test.filters.LargeTest;
21 import android.support.test.filters.MediumTest; 22 import android.support.test.filters.MediumTest;
22 23
24 import org.junit.Assert;
25 import org.junit.Before;
26 import org.junit.Rule;
27 import org.junit.Test;
28 import org.junit.runner.RunWith;
29
23 import org.chromium.base.ThreadUtils; 30 import org.chromium.base.ThreadUtils;
24 import org.chromium.base.annotations.SuppressFBWarnings; 31 import org.chromium.base.annotations.SuppressFBWarnings;
25 import org.chromium.base.test.util.CommandLineFlags; 32 import org.chromium.base.test.util.CommandLineFlags;
26 import org.chromium.base.test.util.DisabledTest; 33 import org.chromium.base.test.util.DisabledTest;
27 import org.chromium.base.test.util.Feature; 34 import org.chromium.base.test.util.Feature;
28 import org.chromium.base.test.util.MinAndroidSdkLevel; 35 import org.chromium.base.test.util.MinAndroidSdkLevel;
29 import org.chromium.base.test.util.RetryOnFailure; 36 import org.chromium.base.test.util.RetryOnFailure;
30 import org.chromium.chrome.R; 37 import org.chromium.chrome.R;
38 import org.chromium.chrome.browser.ChromeSwitches;
31 import org.chromium.chrome.browser.engagement.SiteEngagementService; 39 import org.chromium.chrome.browser.engagement.SiteEngagementService;
32 import org.chromium.chrome.browser.infobar.InfoBar; 40 import org.chromium.chrome.browser.infobar.InfoBar;
33 import org.chromium.chrome.browser.preferences.PrefServiceBridge; 41 import org.chromium.chrome.browser.preferences.PrefServiceBridge;
34 import org.chromium.chrome.browser.preferences.website.ContentSetting; 42 import org.chromium.chrome.browser.preferences.website.ContentSetting;
35 import org.chromium.chrome.browser.profiles.Profile; 43 import org.chromium.chrome.browser.profiles.Profile;
36 import org.chromium.chrome.browser.tab.Tab; 44 import org.chromium.chrome.browser.tab.Tab;
37 import org.chromium.chrome.browser.widget.RoundedIconGenerator; 45 import org.chromium.chrome.browser.widget.RoundedIconGenerator;
46 import org.chromium.chrome.test.ChromeActivityTestRule;
47 import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
38 import org.chromium.chrome.test.util.InfoBarUtil; 48 import org.chromium.chrome.test.util.InfoBarUtil;
39 import org.chromium.chrome.test.util.browser.TabTitleObserver; 49 import org.chromium.chrome.test.util.browser.TabTitleObserver;
40 import org.chromium.chrome.test.util.browser.notifications.MockNotificationManag erProxy.NotificationEntry; 50 import org.chromium.chrome.test.util.browser.notifications.MockNotificationManag erProxy.NotificationEntry;
41 import org.chromium.components.url_formatter.UrlFormatter; 51 import org.chromium.components.url_formatter.UrlFormatter;
42 import org.chromium.content.browser.test.util.Criteria; 52 import org.chromium.content.browser.test.util.Criteria;
43 import org.chromium.content.browser.test.util.CriteriaHelper; 53 import org.chromium.content.browser.test.util.CriteriaHelper;
44 54
45 import java.net.URL; 55 import java.net.URL;
46 import java.util.List; 56 import java.util.List;
47 import java.util.concurrent.Callable; 57 import java.util.concurrent.Callable;
48 import java.util.concurrent.ExecutionException; 58 import java.util.concurrent.ExecutionException;
49 import java.util.concurrent.TimeoutException; 59 import java.util.concurrent.TimeoutException;
50 60
51 /** 61 /**
52 * Instrumentation tests for the Notification Bridge. 62 * Instrumentation tests for the Notification Bridge.
53 * 63 *
54 * Web Notifications are only supported on Android JellyBean and beyond. 64 * Web Notifications are only supported on Android JellyBean and beyond.
55 */ 65 */
56 public class NotificationPlatformBridgeTest extends NotificationTestBase { 66 @RunWith(ChromeJUnit4ClassRunner.class)
67 @CommandLineFlags.Add({
68 ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE,
69 // Preconnect causes issues with the single-threaded Java test server.
70 ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG,
71 })
72 public class NotificationPlatformBridgeTest {
73 @Rule
74 public NotificationTestRule mNotificationTestRule = new NotificationTestRule ();
75
57 private static final String NOTIFICATION_TEST_PAGE = 76 private static final String NOTIFICATION_TEST_PAGE =
58 "/chrome/test/data/notifications/android_test.html"; 77 "/chrome/test/data/notifications/android_test.html";
59 private static final int TITLE_UPDATE_TIMEOUT_SECONDS = (int) scaleTimeout(5 ); 78 private static final int TITLE_UPDATE_TIMEOUT_SECONDS = (int) scaleTimeout(5 );
60 79
61 @Override 80 @Before
62 protected void setUp() throws Exception { 81 public void setUp() throws Exception {
63 super.setUp();
64 SiteEngagementService.setParamValuesForTesting(); 82 SiteEngagementService.setParamValuesForTesting();
65 loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE)); 83 mNotificationTestRule.loadUrl(
84 mNotificationTestRule.getTestServer().getURL(NOTIFICATION_TEST_P AGE));
66 } 85 }
67 86
68 private void waitForTitle(String expectedTitle) throws InterruptedException { 87 private void waitForTitle(String expectedTitle) throws InterruptedException {
69 Tab tab = getActivity().getActivityTab(); 88 Tab tab = mNotificationTestRule.getActivity().getActivityTab();
70 TabTitleObserver titleObserver = new TabTitleObserver(tab, expectedTitle ); 89 TabTitleObserver titleObserver = new TabTitleObserver(tab, expectedTitle );
71 try { 90 try {
72 titleObserver.waitForTitleUpdate(TITLE_UPDATE_TIMEOUT_SECONDS); 91 titleObserver.waitForTitleUpdate(TITLE_UPDATE_TIMEOUT_SECONDS);
73 } catch (TimeoutException e) { 92 } catch (TimeoutException e) {
74 // The title is not as expected, this assertion neatly logs what the difference is. 93 // The title is not as expected, this assertion neatly logs what the difference is.
75 assertEquals(expectedTitle, tab.getTitle()); 94 Assert.assertEquals(expectedTitle, tab.getTitle());
76 } 95 }
77 } 96 }
78 97
79 private InfoBar getInfobarBlocking() { 98 private InfoBar getInfobarBlocking() {
80 CriteriaHelper.pollUiThread(new Criteria() { 99 CriteriaHelper.pollUiThread(new Criteria() {
81 @Override 100 @Override
82 public boolean isSatisfied() { 101 public boolean isSatisfied() {
83 return !getInfoBars().isEmpty(); 102 return !mNotificationTestRule.getInfoBars().isEmpty();
84 } 103 }
85 }); 104 });
86 List<InfoBar> infoBars = getInfoBars(); 105 List<InfoBar> infoBars = mNotificationTestRule.getInfoBars();
87 assertEquals(1, infoBars.size()); 106 Assert.assertEquals(1, infoBars.size());
88 return infoBars.get(0); 107 return infoBars.get(0);
89 } 108 }
90 109
91 private void waitForInfobarToClose() { 110 private void waitForInfobarToClose() {
92 CriteriaHelper.pollUiThread(new Criteria() { 111 CriteriaHelper.pollUiThread(new Criteria() {
93 @Override 112 @Override
94 public boolean isSatisfied() { 113 public boolean isSatisfied() {
95 return getInfoBars().isEmpty(); 114 return mNotificationTestRule.getInfoBars().isEmpty();
96 } 115 }
97 }); 116 });
98 assertEquals(0, getInfoBars().size()); 117 Assert.assertEquals(0, mNotificationTestRule.getInfoBars().size());
99 } 118 }
100 119
101 private void checkThatShowNotificationIsDenied() throws Exception { 120 private void checkThatShowNotificationIsDenied() throws Exception {
102 runJavaScriptCodeInCurrentTab("showNotification('MyNotification', {})"); 121 mNotificationTestRule.runJavaScriptCodeInCurrentTab(
122 "showNotification('MyNotification', {})");
103 waitForTitle("TypeError: No notification permission has been granted for this origin."); 123 waitForTitle("TypeError: No notification permission has been granted for this origin.");
104 // Ideally we'd wait a little here, but it's hard to wait for things tha t shouldn't happen. 124 // Ideally we'd wait a little here, but it's hard to wait for things tha t shouldn't happen.
105 assertTrue(getNotificationEntries().isEmpty()); 125 Assert.assertTrue(mNotificationTestRule.getNotificationEntries().isEmpty ());
106 } 126 }
107 127
108 private double getEngagementScoreBlocking() { 128 private double getEngagementScoreBlocking() {
109 try { 129 try {
110 return ThreadUtils.runOnUiThreadBlocking(new Callable<Double>() { 130 return ThreadUtils.runOnUiThreadBlocking(new Callable<Double>() {
111 @Override 131 @Override
112 public Double call() throws Exception { 132 public Double call() throws Exception {
113 return SiteEngagementService.getForProfile(Profile.getLastUs edProfile()) 133 return SiteEngagementService.getForProfile(Profile.getLastUs edProfile())
114 .getScore(getOrigin()); 134 .getScore(mNotificationTestRule.getOrigin());
115 } 135 }
116 }); 136 });
117 } catch (ExecutionException ex) { 137 } catch (ExecutionException ex) {
118 assert false : "Unexpected ExecutionException"; 138 assert false : "Unexpected ExecutionException";
119 } 139 }
120 return 0.0; 140 return 0.0;
121 } 141 }
122 142
123 /** 143 /**
124 * Verifies that notifcations cannot be shown without permission, and that d ismissing or denying 144 * Verifies that notifcations cannot be shown without permission, and that d ismissing or denying
125 * the infobar works correctly. 145 * the infobar works correctly.
126 */ 146 */
127 //@LargeTest 147 //@LargeTest
128 //@Feature({"Browser", "Notifications"}) 148 //@Feature({"Browser", "Notifications"})
129 // crbug.com/707528 149 // crbug.com/707528
150 @Test
130 @DisabledTest 151 @DisabledTest
131 public void testPermissionDenied() throws Exception { 152 public void testPermissionDenied() throws Exception {
132 // Notifications permission should initially be prompt, and showing shou ld fail. 153 // Notifications permission should initially be prompt, and showing shou ld fail.
133 assertEquals("\"default\"", runJavaScriptCodeInCurrentTab("Notification. permission")); 154 Assert.assertEquals("\"default\"",
155 mNotificationTestRule.runJavaScriptCodeInCurrentTab("Notificatio n.permission"));
134 checkThatShowNotificationIsDenied(); 156 checkThatShowNotificationIsDenied();
135 157
136 // Notification.requestPermission() should show the notifications infoba r. 158 // Notification.requestPermission() should show the notifications infoba r.
137 assertEquals(0, getInfoBars().size()); 159 Assert.assertEquals(0, mNotificationTestRule.getInfoBars().size());
138 runJavaScriptCodeInCurrentTab("Notification.requestPermission(sendToTest )"); 160 mNotificationTestRule.runJavaScriptCodeInCurrentTab(
161 "Notification.requestPermission(sendToTest)");
139 InfoBar infoBar = getInfobarBlocking(); 162 InfoBar infoBar = getInfobarBlocking();
140 163
141 // Dismissing the infobar should pass prompt to the requestPermission ca llback. 164 // Dismissing the infobar should pass prompt to the requestPermission ca llback.
142 assertTrue(InfoBarUtil.clickCloseButton(infoBar)); 165 Assert.assertTrue(InfoBarUtil.clickCloseButton(infoBar));
143 waitForInfobarToClose(); 166 waitForInfobarToClose();
144 waitForTitle("default"); // See https://crbug.com/434547. 167 waitForTitle("default"); // See https://crbug.com/434547.
145 168
146 // Notifications permission should still be prompt. 169 // Notifications permission should still be prompt.
147 assertEquals("\"default\"", runJavaScriptCodeInCurrentTab("Notification. permission")); 170 Assert.assertEquals("\"default\"",
171 mNotificationTestRule.runJavaScriptCodeInCurrentTab("Notificatio n.permission"));
148 checkThatShowNotificationIsDenied(); 172 checkThatShowNotificationIsDenied();
149 173
150 // Notification.requestPermission() should show the notifications infoba r again. 174 // Notification.requestPermission() should show the notifications infoba r again.
151 runJavaScriptCodeInCurrentTab("Notification.requestPermission(sendToTest )"); 175 mNotificationTestRule.runJavaScriptCodeInCurrentTab(
176 "Notification.requestPermission(sendToTest)");
152 infoBar = getInfobarBlocking(); 177 infoBar = getInfobarBlocking();
153 178
154 // Denying the infobar should pass denied to the requestPermission callb ack. 179 // Denying the infobar should pass denied to the requestPermission callb ack.
155 assertTrue(InfoBarUtil.clickSecondaryButton(infoBar)); 180 Assert.assertTrue(InfoBarUtil.clickSecondaryButton(infoBar));
156 waitForInfobarToClose(); 181 waitForInfobarToClose();
157 waitForTitle("denied"); 182 waitForTitle("denied");
158 183
159 // This should have caused notifications permission to become denied. 184 // This should have caused notifications permission to become denied.
160 assertEquals("\"denied\"", runJavaScriptCodeInCurrentTab("Notification.p ermission")); 185 Assert.assertEquals("\"denied\"",
186 mNotificationTestRule.runJavaScriptCodeInCurrentTab("Notificatio n.permission"));
161 checkThatShowNotificationIsDenied(); 187 checkThatShowNotificationIsDenied();
162 188
163 // Reload page to ensure the block is persisted. 189 // Reload page to ensure the block is persisted.
164 loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE)); 190 mNotificationTestRule.loadUrl(
191 mNotificationTestRule.getTestServer().getURL(NOTIFICATION_TEST_P AGE));
165 192
166 // Notification.requestPermission() should immediately pass denied to th e callback without 193 // Notification.requestPermission() should immediately pass denied to th e callback without
167 // showing an infobar. 194 // showing an infobar.
168 runJavaScriptCodeInCurrentTab("Notification.requestPermission(sendToTest )"); 195 mNotificationTestRule.runJavaScriptCodeInCurrentTab(
196 "Notification.requestPermission(sendToTest)");
169 waitForTitle("denied"); 197 waitForTitle("denied");
170 assertEquals(0, getInfoBars().size()); 198 Assert.assertEquals(0, mNotificationTestRule.getInfoBars().size());
171 199
172 // Notifications permission should still be denied. 200 // Notifications permission should still be denied.
173 assertEquals("\"denied\"", runJavaScriptCodeInCurrentTab("Notification.p ermission")); 201 Assert.assertEquals("\"denied\"",
202 mNotificationTestRule.runJavaScriptCodeInCurrentTab("Notificatio n.permission"));
174 checkThatShowNotificationIsDenied(); 203 checkThatShowNotificationIsDenied();
175 } 204 }
176 205
177 /** 206 /**
178 * Verifies granting permission via the infobar. 207 * Verifies granting permission via the infobar.
179 */ 208 */
180 //@MediumTest 209 //@MediumTest
181 //@Feature({"Browser", "Notifications"}) 210 //@Feature({"Browser", "Notifications"})
182 // crbug.com/707528 211 // crbug.com/707528
212 @Test
183 @DisabledTest 213 @DisabledTest
184 public void testPermissionGranted() throws Exception { 214 public void testPermissionGranted() throws Exception {
185 // Notifications permission should initially be prompt, and showing shou ld fail. 215 // Notifications permission should initially be prompt, and showing shou ld fail.
186 assertEquals("\"default\"", runJavaScriptCodeInCurrentTab("Notification. permission")); 216 Assert.assertEquals("\"default\"",
217 mNotificationTestRule.runJavaScriptCodeInCurrentTab("Notificatio n.permission"));
187 checkThatShowNotificationIsDenied(); 218 checkThatShowNotificationIsDenied();
188 219
189 // Notification.requestPermission() should show the notifications infoba r. 220 // Notification.requestPermission() should show the notifications infoba r.
190 assertEquals(0, getInfoBars().size()); 221 Assert.assertEquals(0, mNotificationTestRule.getInfoBars().size());
191 runJavaScriptCodeInCurrentTab("Notification.requestPermission(sendToTest )"); 222 mNotificationTestRule.runJavaScriptCodeInCurrentTab(
223 "Notification.requestPermission(sendToTest)");
192 InfoBar infoBar = getInfobarBlocking(); 224 InfoBar infoBar = getInfobarBlocking();
193 225
194 // Accepting the infobar should pass granted to the requestPermission ca llback. 226 // Accepting the infobar should pass granted to the requestPermission ca llback.
195 assertTrue(InfoBarUtil.clickPrimaryButton(infoBar)); 227 Assert.assertTrue(InfoBarUtil.clickPrimaryButton(infoBar));
196 waitForInfobarToClose(); 228 waitForInfobarToClose();
197 waitForTitle("granted"); 229 waitForTitle("granted");
198 230
199 // Reload page to ensure the grant is persisted. 231 // Reload page to ensure the grant is persisted.
200 loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE)); 232 mNotificationTestRule.loadUrl(
233 mNotificationTestRule.getTestServer().getURL(NOTIFICATION_TEST_P AGE));
201 234
202 // Notifications permission should now be granted, and showing should su cceed. 235 // Notifications permission should now be granted, and showing should su cceed.
203 assertEquals("\"granted\"", runJavaScriptCodeInCurrentTab("Notification. permission")); 236 Assert.assertEquals("\"granted\"",
204 showAndGetNotification("MyNotification", "{}"); 237 mNotificationTestRule.runJavaScriptCodeInCurrentTab("Notificatio n.permission"));
238 mNotificationTestRule.showAndGetNotification("MyNotification", "{}");
205 } 239 }
206 240
207 /** 241 /**
208 * Verifies that the intended default properties of a notification will inde ed be set on the 242 * Verifies that the intended default properties of a notification will inde ed be set on the
209 * Notification object that will be send to Android. 243 * Notification object that will be send to Android.
210 */ 244 */
245 @Test
211 @MediumTest 246 @MediumTest
212 @Feature({"Browser", "Notifications"}) 247 @Feature({"Browser", "Notifications"})
213 @RetryOnFailure 248 @RetryOnFailure
214 public void testDefaultNotificationProperties() throws Exception { 249 public void testDefaultNotificationProperties() throws Exception {
215 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 250 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
216 Context context = getInstrumentation().getTargetContext(); 251 Context context = InstrumentationRegistry.getInstrumentation().getTarget Context();
217 252
218 Notification notification = showAndGetNotification("MyNotification", "{ body: 'Hello' }"); 253 Notification notification =
219 String expectedOrigin = 254 mNotificationTestRule.showAndGetNotification("MyNotification", " { body: 'Hello' }");
220 UrlFormatter.formatUrlForSecurityDisplay(getOrigin(), false /* s howScheme */); 255 String expectedOrigin = UrlFormatter.formatUrlForSecurityDisplay(
256 mNotificationTestRule.getOrigin(), false /* showScheme */);
221 257
222 // Validate the contents of the notification. 258 // Validate the contents of the notification.
223 assertEquals("MyNotification", NotificationTestUtil.getExtraTitle(notifi cation)); 259 Assert.assertEquals("MyNotification", NotificationTestUtil.getExtraTitle (notification));
224 assertEquals("Hello", NotificationTestUtil.getExtraText(notification)); 260 Assert.assertEquals("Hello", NotificationTestUtil.getExtraText(notificat ion));
225 assertEquals(expectedOrigin, NotificationTestUtil.getExtraSubText(notifi cation)); 261 Assert.assertEquals(expectedOrigin, NotificationTestUtil.getExtraSubText (notification));
226 262
227 // Verify that the ticker text contains the notification's title and bod y. 263 // Verify that the ticker text contains the notification's title and bod y.
228 String tickerText = notification.tickerText.toString(); 264 String tickerText = notification.tickerText.toString();
229 265
230 assertTrue(tickerText.contains("MyNotification")); 266 Assert.assertTrue(tickerText.contains("MyNotification"));
231 assertTrue(tickerText.contains("Hello")); 267 Assert.assertTrue(tickerText.contains("Hello"));
232 268
233 // On L+, verify the public version of the notification contains the not ification's origin, 269 // On L+, verify the public version of the notification contains the not ification's origin,
234 // and that the body text has been replaced. 270 // and that the body text has been replaced.
235 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) { 271 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
236 assertNotNull(notification.publicVersion); 272 Assert.assertNotNull(notification.publicVersion);
237 assertEquals(context.getString(R.string.notification_hidden_text), 273 Assert.assertEquals(context.getString(R.string.notification_hidden_t ext),
238 NotificationTestUtil.getExtraText(notification.publicVersion )); 274 NotificationTestUtil.getExtraText(notification.publicVersion ));
239 } 275 }
240 if (Build.VERSION.SDK_INT > Build.VERSION_CODES.M) { 276 if (Build.VERSION.SDK_INT > Build.VERSION_CODES.M) {
241 // On N+, origin should be set as the subtext of the public notifica tion. 277 // On N+, origin should be set as the subtext of the public notifica tion.
242 assertEquals(expectedOrigin, 278 Assert.assertEquals(expectedOrigin,
243 NotificationTestUtil.getExtraSubText(notification.publicVers ion)); 279 NotificationTestUtil.getExtraSubText(notification.publicVers ion));
244 } else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) { 280 } else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
245 // On L/M, origin should be the title of the public notification. 281 // On L/M, origin should be the title of the public notification.
246 assertEquals( 282 Assert.assertEquals(
247 expectedOrigin, NotificationTestUtil.getExtraTitle(notificat ion.publicVersion)); 283 expectedOrigin, NotificationTestUtil.getExtraTitle(notificat ion.publicVersion));
248 } 284 }
249 285
250 // Verify that the notification's timestamp is set in the past 60 second s. This number has 286 // Verify that the notification's timestamp is set in the past 60 second s. This number has
251 // no significance, but needs to be high enough to not cause flakiness a s it's set by the 287 // no significance, but needs to be high enough to not cause flakiness a s it's set by the
252 // renderer process on notification creation. 288 // renderer process on notification creation.
253 assertTrue(Math.abs(System.currentTimeMillis() - notification.when) < 60 * 1000); 289 Assert.assertTrue(Math.abs(System.currentTimeMillis() - notification.whe n) < 60 * 1000);
254 290
255 assertNotNull(NotificationTestUtil.getLargeIconFromNotification(context, notification)); 291 Assert.assertNotNull(
292 NotificationTestUtil.getLargeIconFromNotification(context, notif ication));
256 293
257 // Validate the notification's behavior. 294 // Validate the notification's behavior.
258 assertEquals(Notification.DEFAULT_ALL, notification.defaults); 295 Assert.assertEquals(Notification.DEFAULT_ALL, notification.defaults);
259 assertEquals(Notification.PRIORITY_DEFAULT, notification.priority); 296 Assert.assertEquals(Notification.PRIORITY_DEFAULT, notification.priority );
260 } 297 }
261 298
262 /** 299 /**
263 * Verifies that specifying a notification action with type: 'text' results in a notification 300 * Verifies that specifying a notification action with type: 'text' results in a notification
264 * with a remote input on the action. 301 * with a remote input on the action.
265 */ 302 */
303 @Test
266 @CommandLineFlags.Add("enable-experimental-web-platform-features") 304 @CommandLineFlags.Add("enable-experimental-web-platform-features")
267 @MinAndroidSdkLevel(Build.VERSION_CODES.KITKAT_WATCH) 305 @MinAndroidSdkLevel(Build.VERSION_CODES.KITKAT_WATCH)
268 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs were only added in KITKAT_WATCH. 306 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs were only added in KITKAT_WATCH.
269 @MediumTest 307 @MediumTest
270 @Feature({"Browser", "Notifications"}) 308 @Feature({"Browser", "Notifications"})
271 public void testShowNotificationWithTextAction() throws Exception { 309 public void testShowNotificationWithTextAction() throws Exception {
272 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 310 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
273 311
274 Notification notification = showAndGetNotification("MyNotification", "{ " 312 Notification notification = mNotificationTestRule.showAndGetNotification ("MyNotification",
313 "{ "
275 + " actions: [{action: 'myAction', title: 'reply', type: 'text'," 314 + " actions: [{action: 'myAction', title: 'reply', type: 'text',"
276 + " placeholder: 'hi' }]}"); 315 + " placeholder: 'hi' }]}");
277 316
278 // The specified action should be present, as well as a default settings action. 317 // The specified action should be present, as well as a default settings action.
279 assertEquals(2, notification.actions.length); 318 Assert.assertEquals(2, notification.actions.length);
280 319
281 Notification.Action action = notification.actions[0]; 320 Notification.Action action = notification.actions[0];
282 assertEquals("reply", action.title); 321 Assert.assertEquals("reply", action.title);
283 assertNotNull(notification.actions[0].getRemoteInputs()); 322 Assert.assertNotNull(notification.actions[0].getRemoteInputs());
284 assertEquals(1, action.getRemoteInputs().length); 323 Assert.assertEquals(1, action.getRemoteInputs().length);
285 assertEquals("hi", action.getRemoteInputs()[0].getLabel()); 324 Assert.assertEquals("hi", action.getRemoteInputs()[0].getLabel());
286 } 325 }
287 326
288 /** 327 /**
289 * Verifies that setting a reply on the remote input of a notification actio n with type 'text' 328 * Verifies that setting a reply on the remote input of a notification actio n with type 'text'
290 * and triggering the action's intent causes the same reply to be received i n the subsequent 329 * and triggering the action's intent causes the same reply to be received i n the subsequent
291 * notificationclick event on the service worker. Verifies that site engagem ent is incremented 330 * notificationclick event on the service worker. Verifies that site engagem ent is incremented
292 * appropriately. 331 * appropriately.
293 */ 332 */
333 @Test
294 @CommandLineFlags.Add("enable-experimental-web-platform-features") 334 @CommandLineFlags.Add("enable-experimental-web-platform-features")
295 @MinAndroidSdkLevel(Build.VERSION_CODES.KITKAT_WATCH) 335 @MinAndroidSdkLevel(Build.VERSION_CODES.KITKAT_WATCH)
296 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs were only added in KITKAT_WATCH. 336 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs were only added in KITKAT_WATCH.
297 @MediumTest 337 @MediumTest
298 @Feature({"Browser", "Notifications"}) 338 @Feature({"Browser", "Notifications"})
299 public void testReplyToNotification() throws Exception { 339 public void testReplyToNotification() throws Exception {
300 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 340 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
301 Context context = getInstrumentation().getTargetContext(); 341 Context context = InstrumentationRegistry.getInstrumentation().getTarget Context();
302 342
303 // +5 engagement from notification permission and +0.5 from navigating t o the test page. 343 // +5 engagement from notification permission and +0.5 from navigating t o the test page.
304 assertEquals(5.5, getEngagementScoreBlocking()); 344 Assert.assertEquals(5.5, getEngagementScoreBlocking(), 0);
305 Notification notification = showAndGetNotification("MyNotification", "{ " 345 Notification notification = mNotificationTestRule.showAndGetNotification ("MyNotification",
346 "{ "
306 + " actions: [{action: 'myAction', title: 'reply', type: 'text'}]," 347 + " actions: [{action: 'myAction', title: 'reply', type: 'text'}],"
307 + " data: 'ACTION_REPLY'}"); 348 + " data: 'ACTION_REPLY'}");
308 349
309 // Check the action is present with a remote input attached. 350 // Check the action is present with a remote input attached.
310 Notification.Action action = notification.actions[0]; 351 Notification.Action action = notification.actions[0];
311 assertEquals("reply", action.title); 352 Assert.assertEquals("reply", action.title);
312 RemoteInput[] remoteInputs = action.getRemoteInputs(); 353 RemoteInput[] remoteInputs = action.getRemoteInputs();
313 assertNotNull(remoteInputs); 354 Assert.assertNotNull(remoteInputs);
314 355
315 // Set a reply using the action's remote input key and send it on the in tent. 356 // Set a reply using the action's remote input key and send it on the in tent.
316 sendIntentWithRemoteInput(context, action.actionIntent, remoteInputs, 357 Helper.sendIntentWithRemoteInput(context, action.actionIntent, remoteInp uts,
317 remoteInputs[0].getResultKey(), "My Reply" /* reply */); 358 remoteInputs[0].getResultKey(), "My Reply" /* reply */);
318 359
319 // Check reply was received by the service worker (see android_test_work er.js). 360 // Check reply was received by the service worker (see android_test_work er.js).
320 // Expect +1 engagement from interacting with the notification. 361 // Expect +1 engagement from interacting with the notification.
321 waitForTitle("reply: My Reply"); 362 waitForTitle("reply: My Reply");
322 assertEquals(6.5, getEngagementScoreBlocking()); 363 Assert.assertEquals(6.5, getEngagementScoreBlocking(), 0);
323 } 364 }
324 365
325 /** 366 /**
326 * Verifies that setting an empty reply on the remote input of a notificatio n action with type 367 * Verifies that setting an empty reply on the remote input of a notificatio n action with type
327 * 'text' and triggering the action's intent causes an empty reply string to be received in the 368 * 'text' and triggering the action's intent causes an empty reply string to be received in the
328 * subsequent notificationclick event on the service worker. Verifies that s ite engagement is 369 * subsequent notificationclick event on the service worker. Verifies that s ite engagement is
329 * incremented appropriately. 370 * incremented appropriately.
330 */ 371 */
372 @Test
331 @CommandLineFlags.Add("enable-experimental-web-platform-features") 373 @CommandLineFlags.Add("enable-experimental-web-platform-features")
332 @MinAndroidSdkLevel(Build.VERSION_CODES.KITKAT_WATCH) 374 @MinAndroidSdkLevel(Build.VERSION_CODES.KITKAT_WATCH)
333 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs added in KITKAT _WATCH. 375 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs added in KITKAT _WATCH.
334 @MediumTest 376 @MediumTest
335 @Feature({"Browser", "Notifications"}) 377 @Feature({"Browser", "Notifications"})
336 public void testReplyToNotificationWithEmptyReply() throws Exception { 378 public void testReplyToNotificationWithEmptyReply() throws Exception {
337 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 379 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
338 Context context = getInstrumentation().getTargetContext(); 380 Context context = InstrumentationRegistry.getInstrumentation().getTarget Context();
339 381
340 // +5 engagement from notification permission and +0.5 from navigating t o the test page. 382 // +5 engagement from notification permission and +0.5 from navigating t o the test page.
341 assertEquals(5.5, getEngagementScoreBlocking()); 383 Assert.assertEquals(5.5, getEngagementScoreBlocking(), 0);
342 Notification notification = showAndGetNotification("MyNotification", "{ " 384 Notification notification = mNotificationTestRule.showAndGetNotification ("MyNotification",
385 "{ "
343 + " actions: [{action: 'myAction', title: 'reply', type: 'text'}]," 386 + " actions: [{action: 'myAction', title: 'reply', type: 'text'}],"
344 + " data: 'ACTION_REPLY'}"); 387 + " data: 'ACTION_REPLY'}");
345 388
346 // Check the action is present with a remote input attached. 389 // Check the action is present with a remote input attached.
347 Notification.Action action = notification.actions[0]; 390 Notification.Action action = notification.actions[0];
348 assertEquals("reply", action.title); 391 Assert.assertEquals("reply", action.title);
349 RemoteInput[] remoteInputs = action.getRemoteInputs(); 392 RemoteInput[] remoteInputs = action.getRemoteInputs();
350 assertNotNull(remoteInputs); 393 Assert.assertNotNull(remoteInputs);
351 394
352 // Set a reply using the action's remote input key and send it on the in tent. 395 // Set a reply using the action's remote input key and send it on the in tent.
353 sendIntentWithRemoteInput(context, action.actionIntent, remoteInputs, 396 Helper.sendIntentWithRemoteInput(context, action.actionIntent, remoteInp uts,
354 remoteInputs[0].getResultKey(), "" /* reply */); 397 remoteInputs[0].getResultKey(), "" /* reply */);
355 398
356 // Check empty reply was received by the service worker (see android_tes t_worker.js). 399 // Check empty reply was received by the service worker (see android_tes t_worker.js).
357 // Expect +1 engagement from interacting with the notification. 400 // Expect +1 engagement from interacting with the notification.
358 waitForTitle("reply:"); 401 waitForTitle("reply:");
359 assertEquals(6.5, getEngagementScoreBlocking()); 402 Assert.assertEquals(6.5, getEngagementScoreBlocking(), 0);
360 } 403 }
361 404
362 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs added in KITKAT _WATCH. 405 //TODO(yolandyan): remove this after supporting SdkSuppress annotation
363 private static void sendIntentWithRemoteInput(Context context, PendingIntent pendingIntent, 406 //Currently JUnit4 reflection would look for all test methods in a Test clas s, and
364 RemoteInput[] remoteInputs, String resultKey, String reply) 407 //this test uses RemoteInput as input, this would cause NoClassDefFoundError
365 throws PendingIntent.CanceledException { 408 //in lower than L devices
366 Bundle results = new Bundle(); 409 private static class Helper {
367 results.putString(resultKey, reply); 410 @TargetApi(Build.VERSION_CODES.KITKAT_WATCH) // RemoteInputs added in KI TKAT_WATCH.
368 Intent fillInIntent = new Intent().addFlags(Intent.FLAG_RECEIVER_FOREGRO UND); 411 private static void sendIntentWithRemoteInput(Context context, PendingIn tent pendingIntent,
369 RemoteInput.addResultsToIntent(remoteInputs, fillInIntent, results); 412 RemoteInput[] remoteInputs, String resultKey, String reply)
413 throws PendingIntent.CanceledException {
414 Bundle results = new Bundle();
415 results.putString(resultKey, reply);
416 Intent fillInIntent = new Intent().addFlags(Intent.FLAG_RECEIVER_FOR EGROUND);
417 RemoteInput.addResultsToIntent(remoteInputs, fillInIntent, results);
370 418
371 // Send the pending intent filled in with the additional information fro m the new intent. 419 // Send the pending intent filled in with the additional information from the new
372 pendingIntent.send(context, 0 /* code */, fillInIntent); 420 // intent.
421 pendingIntent.send(context, 0 /* code */, fillInIntent);
422 }
373 } 423 }
374 424
375 /** 425 /**
376 * Verifies that *not* setting a reply on the remote input of a notification action with type 426 * Verifies that *not* setting a reply on the remote input of a notification action with type
377 * 'text' and triggering the action's intent causes a null reply to be recei ved in the 427 * 'text' and triggering the action's intent causes a null reply to be recei ved in the
378 * subsequent notificationclick event on the service worker. Verifies that site engagement is 428 * subsequent notificationclick event on the service worker. Verifies that site engagement is
379 * incremented appropriately. 429 * incremented appropriately.
380 */ 430 */
431 @Test
381 @TargetApi(Build.VERSION_CODES.KITKAT) // Notification.Action.actionIntent a dded in Android K. 432 @TargetApi(Build.VERSION_CODES.KITKAT) // Notification.Action.actionIntent a dded in Android K.
382 @CommandLineFlags.Add("enable-experimental-web-platform-features") 433 @CommandLineFlags.Add("enable-experimental-web-platform-features")
383 @MediumTest 434 @MediumTest
384 @Feature({"Browser", "Notifications"}) 435 @Feature({"Browser", "Notifications"})
385 public void testReplyToNotificationWithNoRemoteInput() throws Exception { 436 public void testReplyToNotificationWithNoRemoteInput() throws Exception {
386 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 437 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
387 438
388 // +5 engagement from notification permission and +0.5 from navigating t o the test page. 439 // +5 engagement from notification permission and +0.5 from navigating t o the test page.
389 assertEquals(5.5, getEngagementScoreBlocking()); 440 Assert.assertEquals(5.5, getEngagementScoreBlocking(), 0);
390 Notification notification = showAndGetNotification("MyNotification", "{ " 441 Notification notification = mNotificationTestRule.showAndGetNotification ("MyNotification",
442 "{ "
391 + " actions: [{action: 'myAction', title: 'reply', type: 'text'}]," 443 + " actions: [{action: 'myAction', title: 'reply', type: 'text'}],"
392 + " data: 'ACTION_REPLY'}"); 444 + " data: 'ACTION_REPLY'}");
393 445
394 assertEquals("reply", notification.actions[0].title); 446 Assert.assertEquals("reply", notification.actions[0].title);
395 notification.actions[0].actionIntent.send(); 447 notification.actions[0].actionIntent.send();
396 448
397 // Check reply was received by the service worker (see android_test_work er.js). 449 // Check reply was received by the service worker (see android_test_work er.js).
398 // Expect +1 engagement from interacting with the notification. 450 // Expect +1 engagement from interacting with the notification.
399 waitForTitle("reply: null"); 451 waitForTitle("reply: null");
400 assertEquals(6.5, getEngagementScoreBlocking()); 452 Assert.assertEquals(6.5, getEngagementScoreBlocking(), 0);
401 } 453 }
402 454
403 /** 455 /**
404 * Verifies that the ONLY_ALERT_ONCE flag is not set when renotify is true. 456 * Verifies that the ONLY_ALERT_ONCE flag is not set when renotify is true.
405 */ 457 */
458 @Test
406 @MediumTest 459 @MediumTest
407 @Feature({"Browser", "Notifications"}) 460 @Feature({"Browser", "Notifications"})
408 public void testNotificationRenotifyProperty() throws Exception { 461 public void testNotificationRenotifyProperty() throws Exception {
409 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 462 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
410 463
411 Notification notification = 464 Notification notification = mNotificationTestRule.showAndGetNotification (
412 showAndGetNotification("MyNotification", "{ tag: 'myTag', renoti fy: true }"); 465 "MyNotification", "{ tag: 'myTag', renotify: true }");
413 466
414 assertEquals(0, notification.flags & Notification.FLAG_ONLY_ALERT_ONCE); 467 Assert.assertEquals(0, notification.flags & Notification.FLAG_ONLY_ALERT _ONCE);
415 } 468 }
416 469
417 /** 470 /**
418 * Verifies that notifications created with the "silent" flag do not inherit system defaults 471 * Verifies that notifications created with the "silent" flag do not inherit system defaults
419 * in regards to their sound, vibration and light indicators. 472 * in regards to their sound, vibration and light indicators.
420 */ 473 */
474 @Test
421 @MediumTest 475 @MediumTest
422 @Feature({"Browser", "Notifications"}) 476 @Feature({"Browser", "Notifications"})
423 public void testNotificationSilentProperty() throws Exception { 477 public void testNotificationSilentProperty() throws Exception {
424 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 478 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
425 479
426 Notification notification = showAndGetNotification("MyNotification", "{ silent: true }"); 480 Notification notification =
481 mNotificationTestRule.showAndGetNotification("MyNotification", " { silent: true }");
427 482
428 // Zero indicates that no defaults should be inherited from the system. 483 // Zero indicates that no defaults should be inherited from the system.
429 assertEquals(0, notification.defaults); 484 Assert.assertEquals(0, notification.defaults);
430 } 485 }
431 486
432 private void verifyVibrationNotRequestedWhenDisabledInPrefs(String notificat ionOptions) 487 private void verifyVibrationNotRequestedWhenDisabledInPrefs(String notificat ionOptions)
433 throws Exception { 488 throws Exception {
434 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 489 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
435 490
436 // Disable notification vibration in preferences. 491 // Disable notification vibration in preferences.
437 ThreadUtils.runOnUiThreadBlocking(new Runnable() { 492 ThreadUtils.runOnUiThreadBlocking(new Runnable() {
438 @Override 493 @Override
439 public void run() { 494 public void run() {
440 PrefServiceBridge.getInstance().setNotificationsVibrateEnabled(f alse); 495 PrefServiceBridge.getInstance().setNotificationsVibrateEnabled(f alse);
441 } 496 }
442 }); 497 });
443 498
444 Notification notification = showAndGetNotification("MyNotification", not ificationOptions); 499 Notification notification =
500 mNotificationTestRule.showAndGetNotification("MyNotification", n otificationOptions);
445 501
446 // Vibration should not be in the defaults. 502 // Vibration should not be in the defaults.
447 assertEquals( 503 Assert.assertEquals(
448 Notification.DEFAULT_ALL & ~Notification.DEFAULT_VIBRATE, notifi cation.defaults); 504 Notification.DEFAULT_ALL & ~Notification.DEFAULT_VIBRATE, notifi cation.defaults);
449 505
450 // There should be a custom no-op vibration pattern. 506 // There should be a custom no-op vibration pattern.
451 assertEquals(1, notification.vibrate.length); 507 Assert.assertEquals(1, notification.vibrate.length);
452 assertEquals(0L, notification.vibrate[0]); 508 Assert.assertEquals(0L, notification.vibrate[0]);
453 } 509 }
454 510
455 /** 511 /**
456 * Verifies that when notification vibration is disabled in preferences and no custom pattern is 512 * Verifies that when notification vibration is disabled in preferences and no custom pattern is
457 * specified, no vibration is requested from the framework. 513 * specified, no vibration is requested from the framework.
458 */ 514 */
515 @Test
459 @MediumTest 516 @MediumTest
460 @Feature({"Browser", "Notifications"}) 517 @Feature({"Browser", "Notifications"})
461 @RetryOnFailure 518 @RetryOnFailure
462 public void testNotificationVibratePreferenceDisabledDefault() throws Except ion { 519 public void testNotificationVibratePreferenceDisabledDefault() throws Except ion {
463 verifyVibrationNotRequestedWhenDisabledInPrefs("{}"); 520 verifyVibrationNotRequestedWhenDisabledInPrefs("{}");
464 } 521 }
465 522
466 /** 523 /**
467 * Verifies that when notification vibration is disabled in preferences and a custom pattern is 524 * Verifies that when notification vibration is disabled in preferences and a custom pattern is
468 * specified, no vibration is requested from the framework. 525 * specified, no vibration is requested from the framework.
469 */ 526 */
527 @Test
470 @MediumTest 528 @MediumTest
471 @Feature({"Browser", "Notifications"}) 529 @Feature({"Browser", "Notifications"})
472 @RetryOnFailure 530 @RetryOnFailure
473 public void testNotificationVibratePreferenceDisabledCustomPattern() throws Exception { 531 public void testNotificationVibratePreferenceDisabledCustomPattern() throws Exception {
474 verifyVibrationNotRequestedWhenDisabledInPrefs("{ vibrate: 42 }"); 532 verifyVibrationNotRequestedWhenDisabledInPrefs("{ vibrate: 42 }");
475 } 533 }
476 534
477 /** 535 /**
478 * Verifies that by default the notification vibration preference is enabled , and a custom 536 * Verifies that by default the notification vibration preference is enabled , and a custom
479 * pattern is passed along. 537 * pattern is passed along.
480 */ 538 */
539 @Test
481 @MediumTest 540 @MediumTest
482 @Feature({"Browser", "Notifications"}) 541 @Feature({"Browser", "Notifications"})
483 public void testNotificationVibrateCustomPattern() throws Exception { 542 public void testNotificationVibrateCustomPattern() throws Exception {
484 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 543 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
485 544
486 // By default, vibration is enabled in notifications. 545 // By default, vibration is enabled in notifications.
487 ThreadUtils.runOnUiThreadBlocking(new Runnable() { 546 ThreadUtils.runOnUiThreadBlocking(new Runnable() {
488 @Override 547 @Override
489 public void run() { 548 public void run() {
490 assertTrue(PrefServiceBridge.getInstance().isNotificationsVibrat eEnabled()); 549 Assert.assertTrue(PrefServiceBridge.getInstance().isNotification sVibrateEnabled());
491 } 550 }
492 }); 551 });
493 552
494 Notification notification = showAndGetNotification("MyNotification", "{ vibrate: 42 }"); 553 Notification notification =
554 mNotificationTestRule.showAndGetNotification("MyNotification", " { vibrate: 42 }");
495 555
496 // Vibration should not be in the defaults, a custom pattern was provide d. 556 // Vibration should not be in the defaults, a custom pattern was provide d.
497 assertEquals( 557 Assert.assertEquals(
498 Notification.DEFAULT_ALL & ~Notification.DEFAULT_VIBRATE, notifi cation.defaults); 558 Notification.DEFAULT_ALL & ~Notification.DEFAULT_VIBRATE, notifi cation.defaults);
499 559
500 // The custom pattern should have been passed along. 560 // The custom pattern should have been passed along.
501 assertEquals(2, notification.vibrate.length); 561 Assert.assertEquals(2, notification.vibrate.length);
502 assertEquals(0L, notification.vibrate[0]); 562 Assert.assertEquals(0L, notification.vibrate[0]);
503 assertEquals(42L, notification.vibrate[1]); 563 Assert.assertEquals(42L, notification.vibrate[1]);
504 } 564 }
505 565
506 /** 566 /**
507 * Verifies that on Android M+, notifications which specify a badge will hav e that icon 567 * Verifies that on Android M+, notifications which specify a badge will hav e that icon
508 * fetched and included as the small icon in the notification and public ver sion. 568 * fetched and included as the small icon in the notification and public ver sion.
509 * If the test target is L or below, verifies the small icon (and public sma ll icon on L) is 569 * If the test target is L or below, verifies the small icon (and public sma ll icon on L) is
510 * the expected chrome logo. 570 * the expected chrome logo.
511 */ 571 */
572 @Test
512 @MediumTest 573 @MediumTest
513 @Feature({"Browser", "Notifications"}) 574 @Feature({"Browser", "Notifications"})
514 public void testShowNotificationWithBadge() throws Exception { 575 public void testShowNotificationWithBadge() throws Exception {
515 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 576 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
516 577
517 Notification notification = 578 Notification notification = mNotificationTestRule.showAndGetNotification (
518 showAndGetNotification("MyNotification", "{badge: 'badge.png'}") ; 579 "MyNotification", "{badge: 'badge.png'}");
519 580
520 assertEquals("MyNotification", NotificationTestUtil.getExtraTitle(notifi cation)); 581 Assert.assertEquals("MyNotification", NotificationTestUtil.getExtraTitle (notification));
521 582
522 Context context = getInstrumentation().getTargetContext(); 583 Context context = InstrumentationRegistry.getInstrumentation().getTarget Context();
523 Bitmap smallIcon = NotificationTestUtil.getSmallIconFromNotification(con text, notification); 584 Bitmap smallIcon = NotificationTestUtil.getSmallIconFromNotification(con text, notification);
524 assertNotNull(smallIcon); 585 Assert.assertNotNull(smallIcon);
525 586
526 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) { 587 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
527 // Custom badges are only supported on M+. 588 // Custom badges are only supported on M+.
528 // 1. Check the notification badge. 589 // 1. Check the notification badge.
529 URL badgeUrl = 590 URL badgeUrl = new URL(mNotificationTestRule.getTestServer().getURL(
530 new URL(getTestServer().getURL("/chrome/test/data/notificati ons/badge.png")); 591 "/chrome/test/data/notifications/badge.png"));
531 Bitmap bitmap = BitmapFactory.decodeStream(badgeUrl.openStream()); 592 Bitmap bitmap = BitmapFactory.decodeStream(badgeUrl.openStream());
532 Bitmap expected = bitmap.copy(bitmap.getConfig(), true); 593 Bitmap expected = bitmap.copy(bitmap.getConfig(), true);
533 NotificationBuilderBase.applyWhiteOverlayToBitmap(expected); 594 NotificationBuilderBase.applyWhiteOverlayToBitmap(expected);
534 assertTrue(expected.sameAs(smallIcon)); 595 Assert.assertTrue(expected.sameAs(smallIcon));
535 596
536 // 2. Check the public notification badge. 597 // 2. Check the public notification badge.
537 assertNotNull(notification.publicVersion); 598 Assert.assertNotNull(notification.publicVersion);
538 Bitmap publicSmallIcon = NotificationTestUtil.getSmallIconFromNotifi cation( 599 Bitmap publicSmallIcon = NotificationTestUtil.getSmallIconFromNotifi cation(
539 context, notification.publicVersion); 600 context, notification.publicVersion);
540 assertNotNull(publicSmallIcon); 601 Assert.assertNotNull(publicSmallIcon);
541 assertEquals(expected.getWidth(), publicSmallIcon.getWidth()); 602 Assert.assertEquals(expected.getWidth(), publicSmallIcon.getWidth()) ;
542 assertEquals(expected.getHeight(), publicSmallIcon.getHeight()); 603 Assert.assertEquals(expected.getHeight(), publicSmallIcon.getHeight( ));
543 assertTrue(expected.sameAs(publicSmallIcon)); 604 Assert.assertTrue(expected.sameAs(publicSmallIcon));
544 } else { 605 } else {
545 Bitmap expected = 606 Bitmap expected =
546 BitmapFactory.decodeResource(context.getResources(), R.drawa ble.ic_chrome); 607 BitmapFactory.decodeResource(context.getResources(), R.drawa ble.ic_chrome);
547 assertTrue(expected.sameAs(smallIcon)); 608 Assert.assertTrue(expected.sameAs(smallIcon));
548 609
549 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) { 610 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
550 // Public versions of notifications are only supported on L+. 611 // Public versions of notifications are only supported on L+.
551 assertNotNull(notification.publicVersion); 612 Assert.assertNotNull(notification.publicVersion);
552 Bitmap publicSmallIcon = NotificationTestUtil.getSmallIconFromNo tification( 613 Bitmap publicSmallIcon = NotificationTestUtil.getSmallIconFromNo tification(
553 context, notification.publicVersion); 614 context, notification.publicVersion);
554 assertTrue(expected.sameAs(publicSmallIcon)); 615 Assert.assertTrue(expected.sameAs(publicSmallIcon));
555 } 616 }
556 } 617 }
557 } 618 }
558 619
559 /** 620 /**
560 * Verifies that notifications which specify an icon will have that icon fet ched, converted into 621 * Verifies that notifications which specify an icon will have that icon fet ched, converted into
561 * a Bitmap and included as the large icon in the notification. 622 * a Bitmap and included as the large icon in the notification.
562 */ 623 */
624 @Test
563 @MediumTest 625 @MediumTest
564 @Feature({"Browser", "Notifications"}) 626 @Feature({"Browser", "Notifications"})
565 @RetryOnFailure 627 @RetryOnFailure
566 public void testShowNotificationWithIcon() throws Exception { 628 public void testShowNotificationWithIcon() throws Exception {
567 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 629 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
568 630
569 Notification notification = showAndGetNotification("MyNotification", "{i con: 'red.png'}"); 631 Notification notification =
632 mNotificationTestRule.showAndGetNotification("MyNotification", " {icon: 'red.png'}");
570 633
571 assertEquals("MyNotification", NotificationTestUtil.getExtraTitle(notifi cation)); 634 Assert.assertEquals("MyNotification", NotificationTestUtil.getExtraTitle (notification));
572 635
573 Context context = getInstrumentation().getTargetContext(); 636 Context context = InstrumentationRegistry.getInstrumentation().getTarget Context();
574 Bitmap largeIcon = NotificationTestUtil.getLargeIconFromNotification(con text, notification); 637 Bitmap largeIcon = NotificationTestUtil.getLargeIconFromNotification(con text, notification);
575 assertNotNull(largeIcon); 638 Assert.assertNotNull(largeIcon);
576 assertEquals(Color.RED, largeIcon.getPixel(0, 0)); 639 Assert.assertEquals(Color.RED, largeIcon.getPixel(0, 0));
577 } 640 }
578 641
579 /** 642 /**
580 * Verifies that notifications which don't specify an icon will get an autom atically generated 643 * Verifies that notifications which don't specify an icon will get an autom atically generated
581 * icon based on their origin. The size of these icons are dependent on the resolution of the 644 * icon based on their origin. The size of these icons are dependent on the resolution of the
582 * device the test is being ran on, so we create it again in order to compar e. 645 * device the test is being ran on, so we create it again in order to compar e.
583 */ 646 */
647 @Test
584 @MediumTest 648 @MediumTest
585 @Feature({"Browser", "Notifications"}) 649 @Feature({"Browser", "Notifications"})
586 @RetryOnFailure 650 @RetryOnFailure
587 public void testShowNotificationWithoutIcon() throws Exception { 651 public void testShowNotificationWithoutIcon() throws Exception {
588 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 652 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
589 653
590 Notification notification = showAndGetNotification("NoIconNotification", "{}"); 654 Notification notification =
655 mNotificationTestRule.showAndGetNotification("NoIconNotification ", "{}");
591 656
592 assertEquals("NoIconNotification", NotificationTestUtil.getExtraTitle(no tification)); 657 Assert.assertEquals("NoIconNotification", NotificationTestUtil.getExtraT itle(notification));
593 658
594 Context context = getInstrumentation().getTargetContext(); 659 Context context = InstrumentationRegistry.getInstrumentation().getTarget Context();
595 assertNotNull(NotificationTestUtil.getLargeIconFromNotification(context, notification)); 660 Assert.assertNotNull(
661 NotificationTestUtil.getLargeIconFromNotification(context, notif ication));
596 662
597 NotificationPlatformBridge notificationBridge = 663 NotificationPlatformBridge notificationBridge =
598 NotificationPlatformBridge.getInstanceForTests(); 664 NotificationPlatformBridge.getInstanceForTests();
599 assertNotNull(notificationBridge); 665 Assert.assertNotNull(notificationBridge);
600 666
601 // Create a second rounded icon for the test's origin, and compare its d imensions against 667 // Create a second rounded icon for the test's origin, and compare its d imensions against
602 // those of the icon associated to the notification itself. 668 // those of the icon associated to the notification itself.
603 RoundedIconGenerator generator = 669 RoundedIconGenerator generator =
604 NotificationBuilderBase.createIconGenerator(context.getResources ()); 670 NotificationBuilderBase.createIconGenerator(context.getResources ());
605 671
606 Bitmap generatedIcon = generator.generateIconForUrl(getOrigin()); 672 Bitmap generatedIcon = generator.generateIconForUrl(mNotificationTestRul e.getOrigin());
607 assertNotNull(generatedIcon); 673 Assert.assertNotNull(generatedIcon);
608 assertTrue(generatedIcon.sameAs( 674 Assert.assertTrue(generatedIcon.sameAs(
609 NotificationTestUtil.getLargeIconFromNotification(context, notif ication))); 675 NotificationTestUtil.getLargeIconFromNotification(context, notif ication)));
610 } 676 }
611 677
612 /* 678 /*
613 * Verifies that starting the PendingIntent stored as the notification's con tent intent will 679 * Verifies that starting the PendingIntent stored as the notification's con tent intent will
614 * start up the associated Service Worker, where the JavaScript code will cl ose the notification 680 * start up the associated Service Worker, where the JavaScript code will cl ose the notification
615 * by calling event.notification.close(). 681 * by calling event.notification.close().
616 */ 682 */
683 @Test
617 @LargeTest 684 @LargeTest
618 @Feature({"Browser", "Notifications"}) 685 @Feature({"Browser", "Notifications"})
619 @RetryOnFailure 686 @RetryOnFailure
620 public void testNotificationContentIntentClosesNotification() throws Excepti on { 687 public void testNotificationContentIntentClosesNotification() throws Excepti on {
621 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 688 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
622 // +5 engagement from notification permission and +0.5 from navigating t o the test page. 689 // +5 engagement from notification permission and +0.5 from navigating t o the test page.
623 assertEquals(5.5, getEngagementScoreBlocking()); 690 Assert.assertEquals(5.5, getEngagementScoreBlocking(), 0);
624 691
625 Notification notification = showAndGetNotification("MyNotification", "{} "); 692 Notification notification =
693 mNotificationTestRule.showAndGetNotification("MyNotification", " {}");
626 694
627 // Sending the PendingIntent resembles activating the notification. 695 // Sending the PendingIntent resembles activating the notification.
628 assertNotNull(notification.contentIntent); 696 Assert.assertNotNull(notification.contentIntent);
629 notification.contentIntent.send(); 697 notification.contentIntent.send();
630 698
631 // The Service Worker will close the notification upon receiving the not ificationclick 699 // The Service Worker will close the notification upon receiving the not ificationclick
632 // event. This will eventually bubble up to a call to cancel() in the No tificationManager. 700 // event. This will eventually bubble up to a call to cancel() in the No tificationManager.
633 // Expect +1 engagement from interacting with the notification. 701 // Expect +1 engagement from interacting with the notification.
634 waitForNotificationManagerMutation(); 702 mNotificationTestRule.waitForNotificationManagerMutation();
635 assertTrue(getNotificationEntries().isEmpty()); 703 Assert.assertTrue(mNotificationTestRule.getNotificationEntries().isEmpty ());
636 assertEquals(6.5, getEngagementScoreBlocking()); 704 Assert.assertEquals(6.5, getEngagementScoreBlocking(), 0);
637 } 705 }
638 706
639 /** 707 /**
640 * Verifies that starting the PendingIntent stored as the notification's con tent intent will 708 * Verifies that starting the PendingIntent stored as the notification's con tent intent will
641 * start up the associated Service Worker, where the JavaScript code will cr eate a new tab for 709 * start up the associated Service Worker, where the JavaScript code will cr eate a new tab for
642 * displaying the notification's event to the user. 710 * displaying the notification's event to the user.
643 */ 711 */
712 @Test
644 @LargeTest 713 @LargeTest
645 @Feature({"Browser", "Notifications"}) 714 @Feature({"Browser", "Notifications"})
646 @RetryOnFailure 715 @RetryOnFailure
647 public void testNotificationContentIntentCreatesTab() throws Exception { 716 public void testNotificationContentIntentCreatesTab() throws Exception {
648 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 717 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
649 718
650 assertEquals("Expected the notification test page to be the sole tab in the current model", 719 Assert.assertEquals(
651 1, getActivity().getCurrentTabModel().getCount()); 720 "Expected the notification test page to be the sole tab in the c urrent model", 1,
721 mNotificationTestRule.getActivity().getCurrentTabModel().getCoun t());
652 722
653 Notification notification = 723 Notification notification = mNotificationTestRule.showAndGetNotification (
654 showAndGetNotification("MyNotification", "{ data: 'ACTION_CREATE _TAB' }"); 724 "MyNotification", "{ data: 'ACTION_CREATE_TAB' }");
655 725
656 // Sending the PendingIntent resembles activating the notification. 726 // Sending the PendingIntent resembles activating the notification.
657 assertNotNull(notification.contentIntent); 727 Assert.assertNotNull(notification.contentIntent);
658 notification.contentIntent.send(); 728 notification.contentIntent.send();
659 729
660 // The Service Worker, upon receiving the notificationclick event, will create a new tab 730 // The Service Worker, upon receiving the notificationclick event, will create a new tab
661 // after which it closes the notification. 731 // after which it closes the notification.
662 waitForNotificationManagerMutation(); 732 mNotificationTestRule.waitForNotificationManagerMutation();
663 assertTrue(getNotificationEntries().isEmpty()); 733 Assert.assertTrue(mNotificationTestRule.getNotificationEntries().isEmpty ());
664 734
665 CriteriaHelper.pollInstrumentationThread(new Criteria("Expected a new ta b to be created") { 735 CriteriaHelper.pollInstrumentationThread(new Criteria("Expected a new ta b to be created") {
666 @Override 736 @Override
667 public boolean isSatisfied() { 737 public boolean isSatisfied() {
668 return 2 == getActivity().getCurrentTabModel().getCount(); 738 return 2 == mNotificationTestRule.getActivity().getCurrentTabMod el().getCount();
669 } 739 }
670 }); 740 });
671 } 741 }
672 742
673 /** 743 /**
674 * Verifies that creating a notification with an associated "tag" will cause any previous 744 * Verifies that creating a notification with an associated "tag" will cause any previous
675 * notification with the same tag to be dismissed prior to being shown. 745 * notification with the same tag to be dismissed prior to being shown.
676 */ 746 */
747 @Test
677 @SuppressFBWarnings("DLS_DEAD_LOCAL_STORE") 748 @SuppressFBWarnings("DLS_DEAD_LOCAL_STORE")
678 @MediumTest 749 @MediumTest
679 @Feature({"Browser", "Notifications"}) 750 @Feature({"Browser", "Notifications"})
680 public void testNotificationTagReplacement() throws Exception { 751 public void testNotificationTagReplacement() throws Exception {
681 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 752 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
682 // +5 engagement from notification permission and +0.5 from navigating t o the test page. 753 // +5 engagement from notification permission and +0.5 from navigating t o the test page.
683 assertEquals(5.5, getEngagementScoreBlocking()); 754 Assert.assertEquals(5.5, getEngagementScoreBlocking(), 0);
684 755
685 runJavaScriptCodeInCurrentTab("showNotification('MyNotification', {tag: 'myTag'});"); 756 mNotificationTestRule.runJavaScriptCodeInCurrentTab(
686 waitForNotificationManagerMutation(); 757 "showNotification('MyNotification', {tag: 'myTag'});");
687 List<NotificationEntry> notifications = getNotificationEntries(); 758 mNotificationTestRule.waitForNotificationManagerMutation();
759 List<NotificationEntry> notifications = mNotificationTestRule.getNotific ationEntries();
688 String tag = notifications.get(0).tag; 760 String tag = notifications.get(0).tag;
689 int id = notifications.get(0).id; 761 int id = notifications.get(0).id;
690 762
691 runJavaScriptCodeInCurrentTab("showNotification('SecondNotification', {t ag: 'myTag'});"); 763 mNotificationTestRule.runJavaScriptCodeInCurrentTab(
692 waitForNotificationManagerMutation(); 764 "showNotification('SecondNotification', {tag: 'myTag'});");
765 mNotificationTestRule.waitForNotificationManagerMutation();
693 766
694 // Verify that the notification was successfully replaced. 767 // Verify that the notification was successfully replaced.
695 notifications = getNotificationEntries(); 768 notifications = mNotificationTestRule.getNotificationEntries();
696 assertEquals(1, notifications.size()); 769 Assert.assertEquals(1, notifications.size());
697 assertEquals("SecondNotification", 770 Assert.assertEquals("SecondNotification",
698 NotificationTestUtil.getExtraTitle(notifications.get(0).notifica tion)); 771 NotificationTestUtil.getExtraTitle(notifications.get(0).notifica tion));
699 772
700 // Verify that for replaced notifications their tag was the same. 773 // Verify that for replaced notifications their tag was the same.
701 assertEquals(tag, notifications.get(0).tag); 774 Assert.assertEquals(tag, notifications.get(0).tag);
702 775
703 // Verify that as always, the same integer is used, also for replaced no tifications. 776 // Verify that as always, the same integer is used, also for replaced no tifications.
704 assertEquals(id, notifications.get(0).id); 777 Assert.assertEquals(id, notifications.get(0).id);
705 assertEquals(NotificationPlatformBridge.PLATFORM_ID, notifications.get(0 ).id); 778 Assert.assertEquals(NotificationPlatformBridge.PLATFORM_ID, notification s.get(0).id);
706 779
707 // Engagement should not have changed since we didn't interact. 780 // Engagement should not have changed since we didn't interact.
708 assertEquals(5.5, getEngagementScoreBlocking()); 781 Assert.assertEquals(5.5, getEngagementScoreBlocking(), 0);
709 } 782 }
710 783
711 /** 784 /**
712 * Verifies that multiple notifications without a tag can be opened and clos ed without 785 * Verifies that multiple notifications without a tag can be opened and clos ed without
713 * affecting eachother. 786 * affecting eachother.
714 */ 787 */
788 @Test
715 @LargeTest 789 @LargeTest
716 @Feature({"Browser", "Notifications"}) 790 @Feature({"Browser", "Notifications"})
717 public void testShowAndCloseMultipleNotifications() throws Exception { 791 public void testShowAndCloseMultipleNotifications() throws Exception {
718 setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW); 792 mNotificationTestRule.setNotificationContentSettingForCurrentOrigin(Cont entSetting.ALLOW);
719 // +5 engagement from notification permission and +0.5 from navigating t o the test page. 793 // +5 engagement from notification permission and +0.5 from navigating t o the test page.
720 assertEquals(5.5, getEngagementScoreBlocking()); 794 Assert.assertEquals(5.5, getEngagementScoreBlocking(), 0);
721 795
722 // Open the first notification and verify it is displayed. 796 // Open the first notification and verify it is displayed.
723 runJavaScriptCodeInCurrentTab("showNotification('One');"); 797 mNotificationTestRule.runJavaScriptCodeInCurrentTab("showNotification('O ne');");
724 waitForNotificationManagerMutation(); 798 mNotificationTestRule.waitForNotificationManagerMutation();
725 List<NotificationEntry> notifications = getNotificationEntries(); 799 List<NotificationEntry> notifications = mNotificationTestRule.getNotific ationEntries();
726 assertEquals(1, notifications.size()); 800 Assert.assertEquals(1, notifications.size());
727 Notification notificationOne = notifications.get(0).notification; 801 Notification notificationOne = notifications.get(0).notification;
728 assertEquals("One", NotificationTestUtil.getExtraTitle(notificationOne)) ; 802 Assert.assertEquals("One", NotificationTestUtil.getExtraTitle(notificati onOne));
729 803
730 // Open the second notification and verify it is displayed. 804 // Open the second notification and verify it is displayed.
731 runJavaScriptCodeInCurrentTab("showNotification('Two');"); 805 mNotificationTestRule.runJavaScriptCodeInCurrentTab("showNotification('T wo');");
732 waitForNotificationManagerMutation(); 806 mNotificationTestRule.waitForNotificationManagerMutation();
733 notifications = getNotificationEntries(); 807 notifications = mNotificationTestRule.getNotificationEntries();
734 assertEquals(2, notifications.size()); 808 Assert.assertEquals(2, notifications.size());
735 Notification notificationTwo = notifications.get(1).notification; 809 Notification notificationTwo = notifications.get(1).notification;
736 assertEquals("Two", NotificationTestUtil.getExtraTitle(notificationTwo)) ; 810 Assert.assertEquals("Two", NotificationTestUtil.getExtraTitle(notificati onTwo));
737 811
738 // The same integer id is always used as it is not needed for uniqueness , we rely on the tag 812 // The same integer id is always used as it is not needed for uniqueness , we rely on the tag
739 // for uniqueness when the replacement behavior is not needed. 813 // for uniqueness when the replacement behavior is not needed.
740 assertEquals(NotificationPlatformBridge.PLATFORM_ID, notifications.get(0 ).id); 814 Assert.assertEquals(NotificationPlatformBridge.PLATFORM_ID, notification s.get(0).id);
741 assertEquals(NotificationPlatformBridge.PLATFORM_ID, notifications.get(1 ).id); 815 Assert.assertEquals(NotificationPlatformBridge.PLATFORM_ID, notification s.get(1).id);
742 816
743 // As these notifications were not meant to replace eachother, they must not have the same 817 // As these notifications were not meant to replace eachother, they must not have the same
744 // tag internally. 818 // tag internally.
745 assertFalse(notifications.get(0).tag.equals(notifications.get(1).tag)); 819 Assert.assertFalse(notifications.get(0).tag.equals(notifications.get(1). tag));
746 820
747 // Verify that the PendingIntent for content and delete is different for each notification. 821 // Verify that the PendingIntent for content and delete is different for each notification.
748 assertFalse(notificationOne.contentIntent.equals(notificationTwo.content Intent)); 822 Assert.assertFalse(notificationOne.contentIntent.equals(notificationTwo. contentIntent));
749 assertFalse(notificationOne.deleteIntent.equals(notificationTwo.deleteIn tent)); 823 Assert.assertFalse(notificationOne.deleteIntent.equals(notificationTwo.d eleteIntent));
750 824
751 // Close the first notification and verify that only the second remains. 825 // Close the first notification and verify that only the second remains.
752 // Sending the content intent resembles touching the notification. In re sponse tho this the 826 // Sending the content intent resembles touching the notification. In re sponse tho this the
753 // notificationclick event is fired. The test service worker will close the notification 827 // notificationclick event is fired. The test service worker will close the notification
754 // upon receiving the event. 828 // upon receiving the event.
755 notificationOne.contentIntent.send(); 829 notificationOne.contentIntent.send();
756 waitForNotificationManagerMutation(); 830 mNotificationTestRule.waitForNotificationManagerMutation();
757 notifications = getNotificationEntries(); 831 notifications = mNotificationTestRule.getNotificationEntries();
758 assertEquals(1, notifications.size()); 832 Assert.assertEquals(1, notifications.size());
759 assertEquals("Two", NotificationTestUtil.getExtraTitle(notifications.get (0).notification)); 833 Assert.assertEquals(
834 "Two", NotificationTestUtil.getExtraTitle(notifications.get(0).n otification));
760 835
761 // Expect +1 engagement from interacting with the notification. 836 // Expect +1 engagement from interacting with the notification.
762 assertEquals(6.5, getEngagementScoreBlocking()); 837 Assert.assertEquals(6.5, getEngagementScoreBlocking(), 0);
763 838
764 // Close the last notification and verify that none remain. 839 // Close the last notification and verify that none remain.
765 notifications.get(0).notification.contentIntent.send(); 840 notifications.get(0).notification.contentIntent.send();
766 waitForNotificationManagerMutation(); 841 mNotificationTestRule.waitForNotificationManagerMutation();
767 assertTrue(getNotificationEntries().isEmpty()); 842 Assert.assertTrue(mNotificationTestRule.getNotificationEntries().isEmpty ());
768 843
769 // Expect +1 engagement from interacting with the notification. 844 // Expect +1 engagement from interacting with the notification.
770 assertEquals(7.5, getEngagementScoreBlocking()); 845 Assert.assertEquals(7.5, getEngagementScoreBlocking(), 0);
771 } 846 }
772 } 847 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698