Bug 684081 - remove redundant code, a=testonly, DONTBUILD

This commit is contained in:
Jonathan Griffin 2011-09-07 16:38:30 -07:00
parent 1a971d463b
commit 0d60daa7ff
2 changed files with 4 additions and 32 deletions

View File

@ -64,15 +64,6 @@ class TPSFirefoxRunner(object):
if self.installdir:
shutil.rmtree(self.installdir, True)
@property
def names(self):
if sys.platform == 'darwin':
return ['firefox', 'minefield']
if (sys.platform == 'linux2') or (sys.platform in ('sunos5', 'solaris')):
return ['firefox', 'mozilla-firefox', 'minefield']
if os.name == 'nt' or sys.platform == 'cygwin':
return ['firefox']
def download_build(self, installdir='downloadedbuild',
appname='firefox', macAppName='Minefield.app'):
self.installdir = os.path.abspath(installdir)
@ -110,28 +101,6 @@ class TPSFirefoxRunner(object):
return binary
def get_respository_info(self):
"""Read repository information from application.ini and platform.ini."""
import ConfigParser
config = ConfigParser.RawConfigParser()
dirname = os.path.dirname(self.runner.binary)
repository = { }
for entry in [['application', 'App'], ['platform', 'Build']]:
(file, section) = entry
config.read(os.path.join(dirname, '%s.ini' % file))
for entry in [['SourceRepository', 'repository'], ['SourceStamp', 'changeset']]:
(key, id) = entry
try:
repository['%s_%s' % (file, id)] = config.get(section, key);
except:
repository['%s_%s' % (file, id)] = None
return repository
def run(self, profile=None, timeout=PROCESS_TIMEOUT, env=None, args=None):
"""Runs the given FirefoxRunner with the given Profile, waits
for completion, then returns the process exit code

View File

@ -287,7 +287,10 @@ class TPSTestRunner(object):
} [phase.status](phase.errline)
logstr = "\n%s | %s%s\n" % (result[0], testname, (' | %s' % result[1] if result[1] else ''))
repoinfo = self.firefoxRunner.get_respository_info()
try:
repoinfo = self.firefoxRunner.runner.get_repositoryInfo()
except:
repoinfo = {}
apprepo = repoinfo.get('application_repository', '')
appchangeset = repoinfo.get('application_changeset', '')