Issue 6600: RunCommand should return an object with various attributes

chromite/lib/cros_build_lib_unittest.py

 #!/usr/bin/python
+#
+# Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
 
 import errno
-import os 
-import unittest 
+import os
 import shutil
-import tempfile 
+import subprocess
+import tempfile
+import unittest
+import cros_build_lib
+import mox
 
-import cros_build_lib
+
+class TestRunCommand(unittest.TestCase):
+
+  def setUp(self):
+    self.mox = mox.Mox()
+    self.mox.StubOutWithMock(subprocess, 'Popen', use_mock_anything=True)
+    self.proc_mock = self.mox.CreateMockAnything()
+    self.cmd = 'test cmd'
+    self.error = 'test error'
+    self.output = 'test output'
+
+  def tearDown(self):
+    self.mox.UnsetStubs()
+    self.mox.VerifyAll()
+
+  def _assertCrEqual(self, expected, actual):
+    """Helper method to compare two CommandResult objects.
+
+    This is needed since assertEqual does not know how to compare two
+    CommandResult objects.
+
+    Args:
+      expected: a CommandResult object, expected result.
+      actual: a CommandResult object, actual result.
+    """
+    self.assertEqual(expected.cmd, actual.cmd)
+    self.assertEqual(expected.error, actual.error)
+    self.assertEqual(expected.output, actual.output)
+    self.assertEqual(expected.returncode, actual.returncode)
+
+  def _testCmd(self, cmd, sp_kv=dict(), rc_kv=dict()):
+    """Factor out common setup logic for testing --cmd.
+
+    Args:
+      cmd: a string or an array of strings.
+      sp_kv: key-value pairs passed to subprocess.Popen().
+      rc_kv: key-value pairs passed to RunCommand().
+    """
+    expected_result = cros_build_lib.CommandResult()
+    expected_result.cmd = self.cmd
+    expected_result.error = self.error
+    expected_result.output = self.output
+    if 'exit_code' in rc_kv:
+      expected_result.returncode = self.proc_mock.returncode
+
+    arg_dict = dict()
+    for attr in 'cwd stdin stdout stderr shell'.split():
+      if attr in sp_kv:
+        arg_dict[attr] = sp_kv[attr]
+      else:
+        if attr == 'shell':
+          arg_dict[attr] = False
+        else:
+          arg_dict[attr] = None
+
+    subprocess.Popen(self.cmd, **arg_dict).AndReturn(self.proc_mock)
+    self.proc_mock.communicate(None).AndReturn((self.output, self.error))
+    self.mox.ReplayAll()
+    actual_result = cros_build_lib.RunCommand(cmd, **rc_kv)
+    self._assertCrEqual(expected_result, actual_result)
+
+  def testReturnCodeZeroWithArrayCmd(self):
+    """--enter_chroot=False and --cmd is an array of strings."""
+    self.proc_mock.returncode = 0
+    cmd_list = ['foo', 'bar', 'roger']
+    self.cmd = 'foo bar roger'
+    self._testCmd(cmd_list, rc_kv=dict(exit_code=True))
+
+  def testReturnCodeZeroWithArrayCmdEnterChroot(self):
+    """--enter_chroot=True and --cmd is an array of strings."""
+    self.proc_mock.returncode = 0
+    cmd_list = ['foo', 'bar', 'roger']
+    self.cmd = './enter_chroot.sh -- %s' % ' '.join(cmd_list)
+    self._testCmd(cmd_list, rc_kv=dict(enter_chroot=True))
+
+  def testReturnCodeNotZeroErrorOkNotRaisesError(self):
+    """Raise error when proc.communicate() returns non-zero."""
+    self.proc_mock.returncode = 1
+    self._testCmd(self.cmd, rc_kv=dict(error_ok=True))
+
+  def testSubprocessCommunicateExceptionRaisesError(self):
+    """Verify error raised by communicate() is caught."""
+    subprocess.Popen(self.cmd, cwd=None, stdin=None, stdout=None, stderr=None,
+                     shell=False).AndReturn(self.proc_mock)
+    self.proc_mock.communicate(None).AndRaise(ValueError)
+    self.mox.ReplayAll()
+    self.assertRaises(ValueError, cros_build_lib.RunCommand, self.cmd)
+
+  def testSubprocessCommunicateExceptionNotRaisesError(self):
+    """Don't re-raise error from communicate() when --error_ok=True."""
+    expected_result = cros_build_lib.CommandResult()
+    cmd_str = './enter_chroot.sh -- %s' % self.cmd
+    expected_result.cmd = cmd_str
+
+    subprocess.Popen(cmd_str, cwd=None, stdin=None, stdout=None, stderr=None,
+                     shell=False).AndReturn(self.proc_mock)
+    self.proc_mock.communicate(None).AndRaise(ValueError)
+    self.mox.ReplayAll()
+    actual_result = cros_build_lib.RunCommand(self.cmd, error_ok=True,
+                                              enter_chroot=True)
+    self._assertCrEqual(expected_result, actual_result)
+
 
 class TestListFiles(unittest.TestCase):
 
   def setUp(self):
     self.root_dir = tempfile.mkdtemp(prefix='listfiles_unittest')
 
   def tearDown(self):
     shutil.rmtree(self.root_dir)
 
   def _createNestedDir(self, dir_structure):
     for entry in dir_structure:
       full_path = os.path.join(os.path.join(self.root_dir, entry))
       # ensure dirs are created
       try:
         os.makedirs(os.path.dirname(full_path))
         if full_path.endswith('/'):
           # we only want to create directories
           return
       except OSError, err:
         if err.errno == errno.EEXIST:
           # we don't care if the dir already exists
           pass
         else:
           raise
       # create dummy files
       tmp = open(full_path, 'w')
       tmp.close()
 
   def testTraverse(self):
-    """
-    Test that we are traversing the directory properly
-    """
+    """Test that we are traversing the directory properly."""
     dir_structure = ['one/two/test.txt', 'one/blah.py',
                      'three/extra.conf']
     self._createNestedDir(dir_structure)
 
     files = cros_build_lib.ListFiles(self.root_dir)
     for file in files:
       file = file.replace(self.root_dir, '').lstrip('/')
       if file not in dir_structure:
         self.fail('%s was not found in %s' % (file, dir_structure))
 
   def testEmptyFilePath(self):
-    """
-    Test that we return nothing when directories are empty
-    """
+    """Test that we return nothing when directories are empty."""
     dir_structure = ['one/', 'two/', 'one/a/']
     self._createNestedDir(dir_structure)
     files = cros_build_lib.ListFiles(self.root_dir)
     self.assertEqual(files, [])
 
   def testNoSuchDir(self):
     try:
       cros_build_lib.ListFiles('/me/no/existe')
     except OSError, err:
       self.assertEqual(err.errno, errno.ENOENT)
-    
+
 
 if __name__ == '__main__':
   unittest.main()