Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1341)

Unified Diff: infra/bots/recipe_modules/flavor/ios_flavor.py

Issue 2215443003: [recipes] Increase test coverage (Closed) Base URL: https://skia.googlesource.com/skia.git@merge_buildbot_spec
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: infra/bots/recipe_modules/flavor/ios_flavor.py
diff --git a/infra/bots/recipe_modules/flavor/ios_flavor.py b/infra/bots/recipe_modules/flavor/ios_flavor.py
index d59b248912ba78a5c8d3d240b559fe48b29a4c17..806926a4b48f00f586470def19724d44afbdfbc9 100644
--- a/infra/bots/recipe_modules/flavor/ios_flavor.py
+++ b/infra/bots/recipe_modules/flavor/ios_flavor.py
@@ -45,16 +45,6 @@ class iOSFlavorUtils(default_flavor.DefaultFlavorUtils):
"""Like os.path.join(), but for paths on a connected iOS device."""
return '/'.join(args)
- def device_path_exists(self, path):
- """Like os.path.exists(), but for paths on a connected device."""
- return self.m.run(
- self.m.step,
- 'exists %s' % path,
- cmd=[self.ios_bin.join('ios_path_exists'), path],
- env=self.default_env,
- infra_step=True,
- ) # pragma: no cover
-
def _remove_device_dir(self, path):
"""Remove the directory on the device."""
return self.m.run(
@@ -106,7 +96,7 @@ class iOSFlavorUtils(default_flavor.DefaultFlavorUtils):
cmd=[self.ios_bin.join('ios_push_file'), host_path, device_path],
env=self.default_env,
infra_step=True,
- ) # pragma: no cover
+ )
def copy_extra_build_products(self, swarming_out_dir):
xcode_dir = self.m.path.join(

Powered by Google App Engine
This is Rietveld 408576698