Move command parsing tests into separate class

Change-Id: I6aeba632d69b1867c7d7745f0c8c9fd8f8e275d9
2017-06-16 18:36:16 -07:00 · 2017-06-16 18:36:16 -07:00 · 96ced40ec2
parent 03889e6465
commit 96ced40ec2
2 changed files with 770 additions and 801 deletions
--- a/packager/app/test/packager_app.py
+++ b/packager/app/test/packager_app.py
@ -52,9 +52,10 @@ class PackagerApp(object):
    # Put single-quotes around each entry so that things like '$' signs in
    # segment templates won't be interpreted as shell variables.
    self.packaging_command_line = ' '.join(["'%s'" % entry for entry in cmd])
-    self.packaging_result = subprocess.call(cmd)
-    if self.packaging_result != 0:
+    packaging_result = subprocess.call(cmd)
+    if packaging_result != 0:
      print '%s returned non-0 status' % self.packaging_command_line
+    return packaging_result

  def GetCommandLine(self):
    return self.packaging_command_line
--- a/packager/app/test/packager_test.py
+++ b/packager/app/test/packager_test.py