Skip to content

Commit

Permalink
address comment: also test with -d:danger, -d:debug
Browse files Browse the repository at this point in the history
  • Loading branch information
timotheecour committed Dec 4, 2020
1 parent 7f83b11 commit d394231
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion tests/exception/t13115.nim
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,14 @@ else:
# remove special case once nodejs updated >= 12.16.2
# refs https://github.com/nim-lang/Nim/pull/16167#issuecomment-738270751
continue
for opt in ["-d:nim_t13115_static", ""]:

# save CI time by avoiding mostly redundant combinations as far as this bug is concerned
var opts = case b
of "c": @["", "-d:nim_t13115_static", "-d:danger", "-d:debug"]
of "js": @["", "-d:nim_t13115_static"]
else: @[""]

for opt in opts:
let cmd = fmt"{nim} r -b:{b} -d:nim_t13115 {opt} --hints:off {file}"
let (outp, exitCode) = execCmdEx(cmd)
when defined windows:
Expand Down

0 comments on commit d394231

Please sign in to comment.