diff options
author | nirnimesh@chromium.org <nirnimesh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-17 01:13:15 +0000 |
---|---|---|
committer | nirnimesh@chromium.org <nirnimesh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-17 01:13:15 +0000 |
commit | 0c186d987a01ac3f4e0fb68fd79b4bbc2492e756 (patch) | |
tree | 8b4f6f6c293ad7d6e7b3dace46c592616a55dde5 | |
parent | 583dca288638e576176ea67e040b0f9e2d160703 (diff) | |
download | chromium_src-0c186d987a01ac3f4e0fb68fd79b4bbc2492e756.zip chromium_src-0c186d987a01ac3f4e0fb68fd79b4bbc2492e756.tar.gz chromium_src-0c186d987a01ac3f4e0fb68fd79b4bbc2492e756.tar.bz2 |
Be lenient with exclusions in PYAUTO_TESTS file
If you try to exclude a test that hasn't been included, pyauto will complain
and it won't run anything. Sometimes it's harder to add exclusions per
platform, and it'd be easier to ignore tests that need to be excluded but
haven't been included.
Review URL: http://codereview.chromium.org/5913003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69494 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/test/pyautolib/pyauto.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/chrome/test/pyautolib/pyauto.py b/chrome/test/pyautolib/pyauto.py index d762bc0..4f81f2b 100644 --- a/chrome/test/pyautolib/pyauto.py +++ b/chrome/test/pyautolib/pyauto.py @@ -2262,7 +2262,10 @@ class Main(object): else: args.extend(self._GetTestsFromName(name)) for name in excluded: - args.remove(name) + if name in args: + args.remove(name) + else: + logging.warn('Cannot exclude %s. Not included. Ignoring' % name) if excluded: logging.debug('Excluded %d test(s): %s' % (len(excluded), excluded)) return args |