Skip to content

Commit dd621a6

Browse files
authored
be more careful when deleting files when testing binaryen methods - the files may not exist, and it is not an error if they do not, we just want them to not be there (#1024)
1 parent 20e1270 commit dd621a6

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

check.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -583,19 +583,19 @@ def break_cashew():
583583
if not success:
584584
break_cashew() # we need cashew
585585
elif method.startswith('interpret-s-expr'):
586-
os.unlink('a.wasm.asm.js') # we should not need the .asm.js
586+
delete_from_orbit('a.wasm.asm.js') # we should not need the .asm.js
587587
if not success:
588-
os.unlink('a.wasm.wast')
588+
delete_from_orbit('a.wasm.wast')
589589
elif method.startswith('asmjs'):
590590
delete_from_orbit('a.wasm.wast') # we should not need the .wast
591591
break_cashew() # we don't use cashew, so ok to break it
592592
if not success:
593-
os.unlink('a.wasm.js')
593+
delete_from_orbit('a.wasm.js')
594594
elif method.startswith('interpret-binary'):
595595
delete_from_orbit('a.wasm.wast') # we should not need the .wast
596-
os.unlink('a.wasm.asm.js') # we should not need the .asm.js
596+
delete_from_orbit('a.wasm.asm.js') # we should not need the .asm.js
597597
if not success:
598-
os.unlink('a.wasm.wasm')
598+
delete_from_orbit('a.wasm.wasm')
599599
else:
600600
1/0
601601
if NODEJS:

0 commit comments

Comments
 (0)