New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 626428 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Write policy_NotificationsBlockedForUrls test using EP Framework

Project Member Reported by scunning...@chromium.org, Jul 7 2016

Issue description

Write automated test for policy_NotificationsBlockedForUrls using EP Framework. Note that test also should verify DefaultNotificationsSetting=1 (ask) and 3 (allow).
 
Components: -Test>Missing Tests>Missing
Owner: trapti@chromium.org
Cc: krishna...@chromium.org
Owner: krishna...@chromium.org
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/07df7da6f10d48d4a55b069906d3f4a414bf5315

commit 07df7da6f10d48d4a55b069906d3f4a414bf5315
Author: Krishna Gavini <krishnargv@chromium.org>
Date: Wed Mar 01 21:19:50 2017

Implement policy_NotificationsBlockedForUrls test.

BUG=chromium:626428
TEST=test_that <IP> policy_NotificationsBlockedForUrls --args='case=SiteBlocked_Block'

Change-Id: I3acca4eeee75924935a61a8f7d7a5823d8d2ca08
Reviewed-on: https://chromium-review.googlesource.com/447202
Commit-Ready: Krishna Gavini <krishnargv@chromium.org>
Tested-by: Krishna Gavini <krishnargv@chromium.org>
Reviewed-by: Vinayak Suley <vsuley@chromium.org>

[add] https://crrev.com/07df7da6f10d48d4a55b069906d3f4a414bf5315/client/site_tests/policy_NotificationsBlockedForUrls/control.notset_show
[add] https://crrev.com/07df7da6f10d48d4a55b069906d3f4a414bf5315/client/site_tests/policy_NotificationsBlockedForUrls/control
[add] https://crrev.com/07df7da6f10d48d4a55b069906d3f4a414bf5315/client/site_tests/policy_NotificationsBlockedForUrls/control.siteblocked_block
[add] https://crrev.com/07df7da6f10d48d4a55b069906d3f4a414bf5315/client/site_tests/policy_NotificationsBlockedForUrls/control.siteallowed_show
[add] https://crrev.com/07df7da6f10d48d4a55b069906d3f4a414bf5315/client/site_tests/policy_NotificationsBlockedForUrls/policy_NotificationsBlockedForUrls.py
[add] https://crrev.com/07df7da6f10d48d4a55b069906d3f4a414bf5315/client/site_tests/policy_NotificationsBlockedForUrls/src/Makefile

Project Member

Comment 6 by bugdroid1@chromium.org, Mar 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/d0a27bb2d56bcb4e2a2abf6b2c23b51175a3c776

commit d0a27bb2d56bcb4e2a2abf6b2c23b51175a3c776
Author: Krishna Gavini <krishnargv@chromium.org>
Date: Fri Mar 03 02:34:10 2017

Add policy_NotificationsBlockedForUrls test to telemetry suite.

BUG=chromium:626428
TEST=test_that <IP> policy_NotificationsBlockedForUrls --args='case=SiteBlocked_Block'

Change-Id: Iaa322668f62e51a521952f9dceee7ef3c5d84477
Reviewed-on: https://chromium-review.googlesource.com/448876
Commit-Ready: Krishna Gavini <krishnargv@chromium.org>
Tested-by: Krishna Gavini <krishnargv@chromium.org>
Reviewed-by: Katherine Threlkeld <kathrelkeld@chromium.org>

[modify] https://crrev.com/d0a27bb2d56bcb4e2a2abf6b2c23b51175a3c776/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

Sign in to add a comment