Skip to content

Commit 09ec5db

Browse files
addaleaxFishrock123
authored andcommitted
test: fail for missing output files
Instead of ignoring missing `.out` files for message/pseudo-tty tests, raise an error to indicate that something is not quite right. Ref: #10037 PR-URL: #10150 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
1 parent 40b0ca1 commit 09ec5db

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

test/message/testcfg.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,7 @@ def ListTests(self, current_path, path, arch, mode):
127127
file_path = file_prefix + ".js"
128128
output_path = file_prefix + ".out"
129129
if not exists(output_path):
130-
print "Could not find %s" % output_path
131-
continue
130+
raise Exception("Could not find %s" % output_path)
132131
result.append(MessageTestCase(test, file_path, output_path,
133132
arch, mode, self.context, self))
134133
return result

test/pseudo-tty/testcfg.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,7 @@ def ListTests(self, current_path, path, arch, mode):
142142
file_path = file_prefix + ".js"
143143
output_path = file_prefix + ".out"
144144
if not exists(output_path):
145-
print "Could not find %s" % output_path
146-
continue
145+
raise Exception("Could not find %s" % output_path)
147146
result.append(TTYTestCase(test, file_path, output_path,
148147
arch, mode, self.context, self))
149148
return result

0 commit comments

Comments
 (0)