commit e8142afbf471575f9c4a4547098bf44d281acd01
author Spencer Low <CompareAndSwap@gmail.com> Mon Sep 14 18:50:13 2015 -0700
committer Spencer Low <CompareAndSwap@gmail.com> Mon Sep 14 18:53:25 2015 -0700
tree ac8d797acd67806e4710b73020ba351ceb720ffa
parent e73bed87f585738729edd5128c474b47abd2d6ce

adb unittest: make test_sync properly cleanup after itself

It wasn't deleting the tempfile.mkdtemp() dir that it made.

Change-Id: I59c5f98aa8297c7b28d38799dd21ffe9566f2145
Signed-off-by: Spencer Low <CompareAndSwap@gmail.com>

adb/test_device.py

diff --git a/adb/test_device.py b/adb/test_device.py
index fedd2d7..0cfc41f 100644
--- a/adb/test_device.py
+++ b/adb/test_device.py
@@ -470,7 +470,8 @@
             self.assertEqual(temp_file.checksum, dev_md5)
 
         self.device.shell(['rm', '-rf', self.DEVICE_TEMP_DIR])
-        shutil.rmtree(base_dir + self.DEVICE_TEMP_DIR)
+        if base_dir is not None:
+            shutil.rmtree(base_dir)
 
     def test_unicode_paths(self):
         """Ensure that we can support non-ASCII paths, even on Windows."""