Bug 1517738 - Have Raptor use the correct binary name when running on android in production; r=gbrown

Differential Revision: https://phabricator.services.mozilla.com/D17251

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Rob Wood 2019-01-22 19:56:58 +00:00
Родитель 73788254d5
Коммит aa7f384d91
2 изменённых файлов: 5 добавлений и 8 удалений

Просмотреть файл

@ -337,6 +337,11 @@ class Raptor(TestingMixin, MercurialScript, CodeCoverageMixin, AndroidMixin):
if not binary_path: if not binary_path:
self.fatal("Raptor requires a path to the binary.") self.fatal("Raptor requires a path to the binary.")
kw_options['binary'] = binary_path kw_options['binary'] = binary_path
if self.app in["geckoview", "fennec"]:
# in production ensure we have correct app name,
# i.e. fennec_aurora or fennec_release etc.
kw_options['binary'] = self.query_package_name()
self.info("set binary to %s instead of %s" % (kw_options['binary'], binary_path))
else: # running on google chrome else: # running on google chrome
if not self.run_local: if not self.run_local:
# when running locally we already set the chrome binary above in init; here # when running locally we already set the chrome binary above in init; here

Просмотреть файл

@ -314,14 +314,6 @@ class Raptor(object):
raise raise
else: else:
# launch fennec # launch fennec
# when running locally the 'binary' is provided on the mach raptor-test cmd line,
# i.e. `org.mozilla.fennec_aurora` for fennec nightly
# in production the default `.fennec_aurora` comes form the tc config;
# if we're running in production but on beta that needs to be changed to `.fennec`
if not self.config['run_local'] and self.config['is_release_build']:
self.config['binary'] = 'org.mozilla.fennec'
try: try:
# if fennec is already running, shut it down first # if fennec is already running, shut it down first
self.device.stop_application(self.config['binary']) self.device.stop_application(self.config['binary'])