Automated chrome driver tests for about:plugins functionality tests

functional/PYAUTO_TESTS

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 #
 # This file lists the pyauto tests that run as a part of the functional test
 # suite.
 #
 # Tests can be enabled on a per-platform basis. Tests not listed here will
 # not be run.
 #
@@ skipping 16 matching lines @@
 # Names beginning with '@' will be treated as suite names and will be
 # recursively expanded.
 
 {
   # The CONTINUOUS suite includes tests that we expect to pass on a
   # continuous builder.
   # This is the suite that gets run on 'Chromium' builds.
   'CONTINUOUS': {
     'all': [
       'autofill',
+      'about_plugins_ui.AboutPluginsTest.testAboutPluginDetailInfo',
       'bookmark_bar',
       'bookmarks',
       'browser',
       'browsing_data',
       'codesign',
       'content',
       'cookies',
       'crash_reporter',
       'databases',
       'downloads',
@@ skipping 235 matching lines @@
       # Import tests are invalid on ChromeOS since Chrome is the only browser.
       '-imports',
       # Multi-profile doesn't apply to chromeos yet.
       '-multiprofile',
       # These require a custom setup on ChromeOS and have their own suite for
       # this platform.
       '-policy',
       '-policy_prefs_ui',
       # No Protector on ChromeOS.
       '-protector',
+      # No Web Driver test for ChromeOS.
+      '-aboutplugins_ui',

[Nirnimesh, 2012/02/03 09:05:31]

this is wrong and has caused pyauto tests to fail on chromeos

It should be: about_plugins_ui

[Nirnimesh, 2012/02/03 09:31:40]

You've added only one test in line 37. Why attempt to exclude all here?

[Nirnimesh, 2012/02/03 09:31:40]

I've fixed it in http://src.chromium.org/viewvc/chrome?view=rev&revision=120303

 
       # ==================================================
       # Disabled tests that need to be investigated/fixed.
       # ==================================================
       # crosbug.com/19556
       '-autofill.AutofillTest.testDisplayLineItemForEntriesWithNoCCNum',
       '-autofill.AutofillTest.testPostalCodeAndStateLabelsBasedOnCountry',
       # Downloads panel stays even after declining a download.
       # crosbug.com/14728
       '-downloads.DownloadsTest.testDeclineDangerousDownload',
@@ skipping 123 matching lines @@
   },
 
   # The FULL suite includes CONTINUOUS, and more.
   # This the suite that gets run on 'Google Chrome' builds.
   'FULL': {
     'all': [
       '@CONTINUOUS',
       'media.media_basic_playback',
       'nacl_sdk',
       'translate',
+      'about_plugins_ui',

[Nirnimesh, 2012/02/03 09:31:40]

FULL = CONTINUOUS + extras

You've already specified one test in CONTINUOUS. So now FULL suite will run
testAboutPluginDetailInfo twice!! Not what you want.


It's better to: include 'about_plugins_ui' in CONTINUOUS, and specifically
disable that one test there, and re-add it in FULL

But why did you pick PDF as a part of this UI test?

 
       # Remove when r119687 reaches official builds.  Revisit 01/31/2012
       '-special_tabs.SpecialTabsTest.testSpecialAcceratorTabs',
 
       # ==================================================
       # Disabled tests that need to be investigated/fixed.
       # ==================================================
       # This test fails due to crbug.com/35485
       '-translate.TranslateTest.testBarNotVisibleOnSSLErrorPage',
       # This test fails due to crbug.com/50706
@@ skipping 102 matching lines @@
       '-extensions.ExtensionsTest.testAllowAccessFileURLs',
     ],
 
     'chromeos': [
       'netflix',
       # ===========================
       # Permanently-disabled tests.
       # ===========================
       # No NaCl support on ChromeOS.
       '-nacl_sdk',
+      # No Web Driver test for ChromeOS.
+      '-aboutplugins_ui',

[Nirnimesh, 2012/02/03 09:31:40]

FWIW, webdriver works on chromeos now. So this should not be necessary.

 
       # ==================================================
       # Disabled tests that need to be investigated/fixed.
       # ==================================================
       # Session restore not working with PyAuto.  crosbug.com/12648
       '-translate.TranslateTest.testSessionRestore',
       # Deal with i18n chars.  crosbug.com/12639
       '-translate.TranslateTest.testDownloadsNotTranslated',
     ],
   },
@@ skipping 67 matching lines @@
     ],
   },
 
   # HTML5 media performance tests.
   'AV_PERF': {
      'linux': [
        'media.media_constrained_network_perf',
      ],
   },
 }