New issue
Advanced search Search tips

Issue 803470 link

Starred by 1 user

Issue metadata

Status: Closed
Owner: ----
Closed: Aug 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Improve notifications code test coverage on Android

Project Member Reported by awdf@chromium.org, Jan 18 2018

Issue description

e.g right now the 'getDisplayedNotifications' codepath is not tested via Android instrumentation tests.
 
Owner: ----
Status: Closed (was: Assigned)
Closing - it's true that currently serviceWorker.getNotifications is untested on Android, but the inner-codepath is identical to other platforms since we don't currently query the OS for currently-displayed notifications, so it is sufficiently tested by interactive ui tests & layout tests that run on other platforms. If/when we start querying Android for currently-displayed notifications, a test should indeed be added then.

Sign in to add a comment