Mercurial > hg
changeset 21329:8ead79ffbc40
run-tests: move blacklist skipping to Test.run()
| author | Gregory Szorc <gregory.szorc@gmail.com> |
|---|---|
| date | Sat, 19 Apr 2014 21:02:51 -0700 |
| parents | 9da0761a22a1 |
| children | 3f79bacbf80b |
| files | tests/run-tests.py |
| diffstat | 1 files changed, 11 insertions(+), 8 deletions(-) [+] |
line wrap: on
line diff
--- a/tests/run-tests.py Sat Apr 19 21:00:24 2014 -0700 +++ b/tests/run-tests.py Sat Apr 19 21:02:51 2014 -0700 @@ -586,6 +586,12 @@ result.skipped = True return self.skip("Doesn't exist") + options = self._options + if not (options.whitelisted and self._test in options.whitelisted): + if options.blacklist and self._test in options.blacklist: + result.skipped = True + return self.skip('blacklisted') + # Remove any previous output files. if os.path.exists(self._errpath): os.remove(self._errpath) @@ -595,7 +601,7 @@ replacements, port = self._getreplacements(testtmp) env = self._getenv(testtmp, port) self._daemonpids.append(env['DAEMON_PIDS']) - createhgrc(env['HGRCPATH'], self._options) + createhgrc(env['HGRCPATH'], options) starttime = time.time() @@ -620,7 +626,7 @@ result.refout = self._refout - if not self._options.keep_tmpdir: + if not options.keep_tmpdir: shutil.rmtree(testtmp) def describe(ret): @@ -648,10 +654,10 @@ return self.fail('timed out', ret) elif out != self._refout: info = {} - if not self._options.nodiff: + if not options.nodiff: iolock.acquire() - if self._options.view: - os.system("%s %s %s" % (self._options.view, self._refpath, + if options.view: + os.system("%s %s %s" % (options.view, self._refpath, self._errpath)) else: info = showdiff(self._refout, out, self._refpath, @@ -1117,9 +1123,6 @@ lctest = test.lower() if not (options.whitelisted and test in options.whitelisted): - if options.blacklist and test in options.blacklist: - return skip("blacklisted") - if options.retest and not os.path.exists(test + ".err"): return ignore("not retesting")