Backed out changeset 86d0bd2a0233 (bug 992911) for android 2.3 bustage

This commit is contained in:
Carsten "Tomcat" Book 2014-06-02 13:36:51 +02:00
parent 3dcd10150f
commit af448b0363
5 changed files with 42 additions and 168 deletions

View File

@ -51,10 +51,7 @@ registerCleanupFunction(function() {
// Import the GCLI test helper
let testDir = gTestPath.substr(0, gTestPath.lastIndexOf("/"));
testDir = testDir.replace(/\/\//g, '/');
testDir = testDir.replace("chrome:/mochitest", "chrome://mochitest");
let helpersjs = testDir + "/../../commandline/test/helpers.js";
Services.scriptloader.loadSubScript(helpersjs, this);
Services.scriptloader.loadSubScript(testDir + "../../../commandline/test/helpers.js", this);
// Redeclare dbg_assert with a fatal behavior.
function dbg_assert(cond, e) {

View File

@ -163,12 +163,11 @@ function getFileListing(basePath, testPath, dir, srvScope)
var uri = getResolvedURI(basePath);
var chromeDir = getChromeDir(uri);
chromeDir.appendRelativePath(dir);
basePath += '/' + dir.replace(/\\/g, '/');
basePath += '/' + dir;
if (testPath == "false" || testPath == false) {
testPath = "";
}
testPath = testPath.replace(/\\\\/g, '\\').replace(/\\/g, '/');
var ioSvc = Components.classes["@mozilla.org/network/io-service;1"].
getService(Components.interfaces.nsIIOService);

View File

@ -190,7 +190,7 @@ class MochitestRunner(MozbuildObject):
jsdebugger=False, debug_on_failure=False, start_at=None, end_at=None,
e10s=False, dmd=False, dump_output_directory=None,
dump_about_memory_after_test=False, dump_dmd_after_test=False,
install_extension=None, quiet=False, environment=[], app_override=None, runByDir=False,
install_extension=None, quiet=False, environment=[], app_override=None,
useTestMediaDevices=False, **kwargs):
"""Runs a mochitest.
@ -316,7 +316,6 @@ class MochitestRunner(MozbuildObject):
options.dumpOutputDirectory = dump_output_directory
options.quiet = quiet
options.environment = environment
options.runByDir = runByDir
options.useTestMediaDevices = useTestMediaDevices
options.failureFile = failure_file_path
@ -527,12 +526,6 @@ def MochitestCommand(func):
help="Sets the given variable in the application's environment")
func = setenv(func)
runbydir = CommandArgument('--run-by-dir', default=False,
action='store_true',
dest='runByDir',
help='Run each directory in a single browser instance with a fresh profile.')
func = runbydir(func)
test_media = CommandArgument('--use-test-media-devices', default=False,
action='store_true',
dest='useTestMediaDevices',

View File

@ -99,12 +99,6 @@ class MochitestOptions(optparse.OptionParser):
"help": "group tests together in the same chunk that are in the same top chunkByDir directories",
"default": 0,
}],
[["--run-by-dir"],
{ "action": "store_true",
"dest": "runByDir",
"help": "Run each directory in a single browser instance with a fresh profile",
"default": False,
}],
[["--shuffle"],
{ "dest": "shuffle",
"action": "store_true",

View File

@ -417,7 +417,7 @@ class MochitestUtilsMixin(object):
def getTestPath(self, options):
if options.ipcplugins:
return "dom/plugins/test/mochitest"
return "dom/plugins/test"
else:
return options.testPath
@ -451,7 +451,24 @@ class MochitestUtilsMixin(object):
Build a manifest of tests to run and write out a json file for the harness to read
"""
manifest = None
manifest = self.getTestManifest(options)
testRoot = self.getTestRoot(options)
# testdir refers to 'mochitest' here.
testdir = SCRIPT_DIR.split(os.getcwd())[-1]
testdir = testdir.strip(os.sep)
testRootAbs = os.path.abspath(os.path.join(testdir, testRoot))
if isinstance(options.manifestFile, TestManifest):
manifest = options.manifestFile
elif options.manifestFile and os.path.isfile(options.manifestFile):
manifestFileAbs = os.path.abspath(options.manifestFile)
assert manifestFileAbs.startswith(testRootAbs)
manifest = TestManifest([options.manifestFile], strict=False)
else:
masterName = self.getTestFlavor(options) + '.ini'
masterPath = os.path.join(testdir, testRoot, masterName)
if os.path.exists(masterPath):
manifest = TestManifest([masterPath], strict=False)
if manifest:
# Python 2.6 doesn't allow unicode keys to be used for keyword
@ -465,7 +482,6 @@ class MochitestUtilsMixin(object):
# Bug 883858 - return all tests including disabled tests
testPath = self.getTestPath(options)
testPath = testPath.replace('\\', '/')
if testPath.endswith('.html') or \
testPath.endswith('.xhtml') or \
testPath.endswith('.xul') or \
@ -481,8 +497,8 @@ class MochitestUtilsMixin(object):
for test in tests:
pathAbs = os.path.abspath(test['path'])
assert pathAbs.startswith(self.testRootAbs)
tp = pathAbs[len(self.testRootAbs):].replace('\\', '/').strip('/')
assert pathAbs.startswith(testRootAbs)
tp = pathAbs[len(testRootAbs):].replace('\\', '/').strip('/')
# Filter out tests if we are using --test-path
if testPath and not tp.startswith(testPath):
@ -506,7 +522,7 @@ class MochitestUtilsMixin(object):
paths.sort(path_sort)
# Bug 883865 - add this functionality into manifestDestiny
with open(os.path.join(SCRIPT_DIR, 'tests.json'), 'w') as manifestFile:
with open(os.path.join(testdir, 'tests.json'), 'w') as manifestFile:
manifestFile.write(json.dumps({'tests': paths}))
options.manifestFile = 'tests.json'
@ -1049,9 +1065,9 @@ class Mochitest(MochitestUtilsMixin):
return browserEnv
def cleanup(self, options):
def cleanup(self, manifest, options):
""" remove temporary files and profile """
os.remove(self.manifest)
os.remove(manifest)
del self.profile
if options.pidFile != "":
try:
@ -1170,8 +1186,7 @@ class Mochitest(MochitestUtilsMixin):
timeout=-1,
onLaunch=None,
webapprtChrome=False,
screenshotOnFail=False,
testPath=None):
screenshotOnFail=False):
"""
Run the app, log the duration it took to execute, return the status code.
Kills the app if it runs for longer than |maxTime| seconds, or outputs nothing for |timeout| seconds.
@ -1235,7 +1250,7 @@ class Mochitest(MochitestUtilsMixin):
def timeoutHandler():
browserProcessId = outputHandler.browserProcessId
self.handleTimeout(timeout, proc, utilityPath, debuggerInfo, browserProcessId, testPath)
self.handleTimeout(timeout, proc, utilityPath, debuggerInfo, browserProcessId)
kp_kwargs = {'kill_on_timeout': False,
'cwd': SCRIPT_DIR,
'onTimeout': [timeoutHandler]}
@ -1324,58 +1339,6 @@ class Mochitest(MochitestUtilsMixin):
def runTests(self, options, onLaunch=None):
""" Prepare, configure, run tests and cleanup """
# Create variables to count the number of passes, fails, todos.
self.countpass = 0
self.countfail = 0
self.counttodo = 0
self.testRoot = self.getTestRoot(options)
self.testRootAbs = os.path.join(SCRIPT_DIR, self.testRoot)
if not options.runByDir:
return self.doTests(options, onLaunch)
dirs = self.getDirectories(options)
if options.totalChunks > 1:
chunkSize = int(len(dirs) / options.totalChunks) + 1
start = chunkSize * (options.thisChunk-1)
end = chunkSize * (options.thisChunk)
dirs = dirs[start:end]
options.totalChunks = None
options.thisChunk = None
options.chunkByDir = 0
inputTestPath = self.getTestPath(options)
for dir in dirs:
options.manifestFile = None
if inputTestPath and not inputTestPath.startswith(dir):
continue
options.testPath = dir
print "testpath: %s" % options.testPath
options.profilePath = tempfile.mkdtemp()
self.urlOpts = []
self.doTests(options, onLaunch)
# printing total number of tests
if options.browserChrome:
print "TEST-INFO | checking window state"
print "Browser Chrome Test Summary"
print "\tPassed: %s" % self.countpass
print "\tFailed: %s" % self.countfail
print "\tTodo: %s" % self.counttodo
print "*** End BrowserChrome Test Results ***"
else:
print "0 INFO TEST-START | Shutdown"
print "1 INFO Passed: %s" % self.countpass
print "2 INFO Failed: %s" % self.countfail
print "3 INFO Todo: %s" % self.counttodo
print "4 INFO SimpleTest FINISHED"
def doTests(self, options, onLaunch=None):
# get debugger info, a dict of:
# {'path': path to the debugger (string),
# 'interactive': whether the debugger is interactive or not (bool)
@ -1396,31 +1359,22 @@ class Mochitest(MochitestUtilsMixin):
self.leak_report_file = os.path.join(options.profilePath, "runtests_leaks.log")
self.browserEnv = self.buildBrowserEnv(options, debuggerInfo is not None)
if self.browserEnv is None:
browserEnv = self.buildBrowserEnv(options, debuggerInfo is not None)
if browserEnv is None:
return 1
# buildProfile sets self.profile .
# This relies on sideeffects and isn't very stateful:
# https://bugzilla.mozilla.org/show_bug.cgi?id=919300
self.manifest = self.buildProfile(options)
if self.manifest is None:
manifest = self.buildProfile(options)
if manifest is None:
return 1
try:
self.startServers(options, debuggerInfo)
testURL = self.buildTestPath(options)
with open(os.path.join(SCRIPT_DIR, 'tests.json')) as fHandle:
tests = json.load(fHandle)
count = 0
for test in tests['tests']:
count += 1
if count == 0:
return 1
self.buildURLOptions(options, self.browserEnv)
self.buildURLOptions(options, browserEnv)
if self.urlOpts:
testURL += "?" + "&".join(self.urlOpts)
@ -1454,7 +1408,7 @@ class Mochitest(MochitestUtilsMixin):
log.info("runtests.py | Running tests: start.\n")
try:
status = self.runApp(testURL,
self.browserEnv,
browserEnv,
options.app,
profile=self.profile,
extraArgs=options.browserArgs,
@ -1464,8 +1418,7 @@ class Mochitest(MochitestUtilsMixin):
timeout=timeout,
onLaunch=onLaunch,
webapprtChrome=options.webapprtChrome,
screenshotOnFail=options.screenshotOnFail,
testPath=options.testPath
screenshotOnFail=options.screenshotOnFail
)
except KeyboardInterrupt:
log.info("runtests.py | Received keyboard interrupt.\n");
@ -1490,18 +1443,15 @@ class Mochitest(MochitestUtilsMixin):
log.info("runtests.py | Running tests: end.")
if self.manifest is not None:
self.cleanup(options)
if manifest is not None:
self.cleanup(manifest, options)
return status
def handleTimeout(self, timeout, proc, utilityPath, debuggerInfo, browserProcessId, testPath=None):
def handleTimeout(self, timeout, proc, utilityPath, debuggerInfo, browserProcessId):
"""handle process output timeout"""
# TODO: bug 913975 : _processOutput should call self.processOutputLine one more time one timeout (I think)
if testPath:
log.info("TEST-UNEXPECTED-FAIL | %s | application timed out after %d seconds with no output on %s", self.lastTestSeen, int(timeout), testPath)
else:
log.info("TEST-UNEXPECTED-FAIL | %s | application timed out after %d seconds with no output", self.lastTestSeen, int(timeout))
log.info("TEST-UNEXPECTED-FAIL | %s | application timed out after %d seconds with no output", self.lastTestSeen, int(timeout))
browserProcessId = browserProcessId or proc.pid
self.killAndGetStack(browserProcessId, utilityPath, debuggerInfo, dump_screen=not debuggerInfo)
@ -1548,7 +1498,6 @@ class Mochitest(MochitestUtilsMixin):
self.metro_subprocess_id,
self.trackShutdownLeaks,
self.log,
self.countline,
]
def stackFixer(self):
@ -1601,20 +1550,6 @@ class Mochitest(MochitestUtilsMixin):
# output line handlers:
# these take a line and return a line
def countline(self, line):
val = 0
try:
val = int(line.split(':')[-1].strip())
except ValueError, e:
return line
if "Passed:" in line:
self.harness.countpass += val
elif "Failed:" in line:
self.harness.countfail += val
elif "Todo:" in line:
self.harness.counttodo += val
return line
def fix_stack(self, line):
if self.stackFixerFunction:
@ -1664,12 +1599,13 @@ class Mochitest(MochitestUtilsMixin):
"Creates a test configuration file for customizing test execution."
options.logFile = options.logFile.replace("\\", "\\\\")
options.testPath = options.testPath.replace("\\", "\\\\")
testRoot = self.getTestRoot(options)
if "MOZ_HIDE_RESULTS_TABLE" in os.environ and os.environ["MOZ_HIDE_RESULTS_TABLE"] == "1":
options.hideResultsTable = True
d = dict(options.__dict__)
d['testRoot'] = self.testRoot
d['testRoot'] = testRoot
content = json.dumps(d)
with open(os.path.join(options.profilePath, "testConfig.js"), "w") as config:
@ -1705,51 +1641,6 @@ class Mochitest(MochitestUtilsMixin):
for path in self.getExtensionsToInstall(options):
self.installExtensionFromPath(options, path)
def getTestManifest(self, options):
if isinstance(options.manifestFile, TestManifest):
manifest = options.manifestFile
elif options.manifestFile and os.path.isfile(options.manifestFile):
manifestFileAbs = os.path.abspath(options.manifestFile)
assert manifestFileAbs.startswith(SCRIPT_DIR)
manifest = TestManifest([options.manifestFile], strict=False)
elif options.manifestFile and os.path.isfile(os.path.join(SCRIPT_DIR, options.manifestFile)):
manifestFileAbs = os.path.abspath(os.path.join(SCRIPT_DIR, options.manifestFile))
assert manifestFileAbs.startswith(SCRIPT_DIR)
manifest = TestManifest([manifestFileAbs], strict=False)
else:
masterName = self.getTestFlavor(options) + '.ini'
masterPath = os.path.join(SCRIPT_DIR, self.testRoot, masterName)
if os.path.exists(masterPath):
manifest = TestManifest([masterPath], strict=False)
return manifest
def getDirectories(self, options):
"""
Make the list of directories by parsing manifests
"""
info = {}
for k, v in mozinfo.info.items():
if isinstance(k, unicode):
k = k.encode('ascii')
info[k] = v
dirlist = []
manifest = self.getTestManifest(options)
tests = manifest.active_tests(disabled=False, options=options, **info)
for test in tests:
pathAbs = os.path.abspath(test['path'])
assert pathAbs.startswith(self.testRootAbs)
tp = pathAbs[len(self.testRootAbs):].replace('\\', '/').strip('/')
rootdir = '/'.join(tp.split('/')[:-1])
if rootdir not in dirlist:
dirlist.append(rootdir)
dirlist.sort()
return dirlist
def main():