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

Issue 626429 link

Starred by 2 users

Issue metadata

Status: Verified
Owner: ----
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Write policy_PluginsAllowedForUrls test using EP Framework

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

Issue description

Write automated test for policy_PluginsAllowedForUrls using EP Framework. Note that test also should verify DefaultPluginsSetting=2 (block) and 3 (click to play).
 
Cc: trapti@chromium.org
Owner: scunning...@chromium.org
Note: Will will verify with dependent policies DisablePluginFinder=True and AllowOutdatedPlugins=True.
Status: Started (was: Assigned)
Note 1 (cont): Will verify with dependent policies AlwaysAuthorizePlugins=true, DisabledPlugins not set, EnabledPlugins not set, and DisabledPluginsExceptions not set.

Since most URL-based tests require JavaScript, Images, and Cookies to be enabled, set DefaultJavaScriptSetting=1 (allow all sites to run JavaScript), DefaultImagesSetting=1 (allow all sites to show images), and DefaultCookiesSetting=1

For debugging purposes, it's also handy to have RestoreOnStartup=4, and RestoreOnStartupURLs=[chrome://policy, chrome://settings].
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 17 2016

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

commit f2da7df7158d1d9226e484aa97eee5037c081668
Author: Scott Cunningham <scunningham@chromium.org>
Date: Fri Sep 02 23:08:21 2016

Implement policy_PluginsAllowedForUrls test.

BUG= chromium:626429 
TEST=test_that <IP> policy_PluginsAllowedForUrls --args='case=SiteAllowed_Run'

Change-Id: I0eeacb6c630cf34bc586b9f1ac60961b9f0673b0
Reviewed-on: https://chromium-review.googlesource.com/380815
Commit-Ready: Scott Cunningham <scunningham@chromium.org>
Tested-by: Scott Cunningham <scunningham@chromium.org>
Reviewed-by: Scott Cunningham <scunningham@chromium.org>
Reviewed-by: danny chan <dchan@chromium.org>
Reviewed-by: Krishna Gavini <krishnargv@chromium.org>

[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/control.siteallowed_run
[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/control.notset_block
[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/policy_PluginsAllowedForUrls.py
[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/control
[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/plugin_status.html
[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/plugin_flash.swf
[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/control.sitenotallowed_block
[add] https://crrev.com/f2da7df7158d1d9226e484aa97eee5037c081668/client/site_tests/policy_PluginsAllowedForUrls/src/Makefile

Project Member

Comment 8 by bugdroid1@chromium.org, Sep 20 2016

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

commit 64c21e232b0766e48ed75063fa5be335c8358776
Author: Scott Cunningham <scunningham@chromium.org>
Date: Mon Sep 19 22:48:18 2016

Add policy_PluginsAllowedForUrls test to telemetry suite.

BUG= chromium:626429 
TEST=test_that <IP> policy_PluginsAllowedForUrls --args='case=SiteAllowed_Run'

Change-Id: I63359c6469c116bdc516097c24467443ccd615c4
Reviewed-on: https://chromium-review.googlesource.com/387225
Tested-by: Scott Cunningham <scunningham@chromium.org>
Trybot-Ready: Scott Cunningham <scunningham@chromium.org>
Reviewed-by: Scott Cunningham <scunningham@chromium.org>
Commit-Queue: Scott Cunningham <scunningham@chromium.org>

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

Status: Fixed (was: Started)
Owner: ----

Sign in to add a comment