cros_sdk_lib: Make CleanupChrootMount delete consistent.
cros_sdk_lib.CleanupChrootMount has a delete_image option. If
delete_image is True, there is no case where rm'ing the chroot
directory doesn't also make sense, especially since we call this
function for chroots which don't use chroot.img. So... change
'delete_image' to 'delete' and also cleanup the chroot directory.
BUG=None
TEST=run_tests
Change-Id: I845069ab5c5a78e969cafc2b8da168d65dd11b5a
Reviewed-on: https://chromium-review.googlesource.com/1119658
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Don Garrett <dgarrett@chromium.org>
Reviewed-by: Jason Clinton <jclinton@chromium.org>
diff --git a/scripts/cbuildbot_launch_unittest.py b/scripts/cbuildbot_launch_unittest.py
index 9a11612..23a61f9 100644
--- a/scripts/cbuildbot_launch_unittest.py
+++ b/scripts/cbuildbot_launch_unittest.py
@@ -335,7 +335,7 @@
self.root, '.cbuildbot_build_state.json')
self.buildroot = os.path.join(self.root, 'buildroot')
self.repo = os.path.join(self.buildroot, '.repo/repo')
- self.chroot = os.path.join(self.buildroot, 'chroot/chroot')
+ self.chroot = os.path.join(self.buildroot, 'chroot')
self.general = os.path.join(self.buildroot, 'general/general')
self.cache = os.path.join(self.buildroot, '.cache')
self.distfiles = os.path.join(self.cache, 'distfiles')
@@ -450,11 +450,10 @@
self.assertEqual(new_summary, build_state)
self.assertExists(self.repo)
- self.assertNotExists(self.chroot)
self.assertExists(self.general)
self.assertNotExists(self.distfiles)
self.assertExists(self.previous_build_state)
- m.assert_called()
+ m.assert_called_with(self.chroot, delete=True)
def testBuildrootBranchMatch(self):
"""Test CleanBuildRoot with no change in branch."""