diff options
| author | Chinmay <chinmay1dd@gmail.com> | 2020-09-10 09:25:20 -0700 |
|---|---|---|
| committer | Chinmay <chinmay1dd@gmail.com> | 2020-09-10 10:12:31 -0700 |
| commit | 6b036f75273ff4b8ac1130d7524b320be096c15b (patch) | |
| tree | 26b1ea33029973b6229b011316323e1330f8d89d /suite | |
| parent | 167ced17956bbb79aee3a6e6ef88079d822427cb (diff) | |
Fix Windows file deletion issue in tests
Diffstat (limited to 'suite')
| -rw-r--r-- | suite/testcommon.py | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/suite/testcommon.py b/suite/testcommon.py index 14a991a5..036dc8ac 100644 --- a/suite/testcommon.py +++ b/suite/testcommon.py @@ -1323,7 +1323,6 @@ class VerifyBuilder(Builder): # test bndb round trip temp_name = next(tempfile._get_candidate_names()) + ".bndb" bv.create_database(temp_name) - bv.file.close() with binja.open_view(temp_name) as bv: func = bv.get_function_at(0x00008440) @@ -1356,10 +1355,14 @@ class VerifyBuilder(Builder): found = True assert(found) - bv.file.close() - - os.unlink(temp_name) - # Check retrieval of user variable values + for i in range(5): + try: + time.sleep(1) + os.unlink(temp_name) + break + except OSError: + print("Failed to remove file {}".format(temp_name)) + continue return True finally: self.delete_package("helloworld") @@ -1401,9 +1404,15 @@ class VerifyBuilder(Builder): assert(def_ins.get_possible_reg_values_after('r3') == value) - os.unlink(temp_name) + for i in range(5): + try: + time.sleep(1) + os.unlink(temp_name) + break + except OSError: + print("Failed to remove file {}".format(temp_name)) + continue return True - finally: self.delete_package("helloworld") |