Skip to content

Commit 183fe1d

Browse files
committed
Quick fix for test errors on backtrace content
1 parent 42d8168 commit 183fe1d

1 file changed

Lines changed: 17 additions & 3 deletions

File tree

test/test_execjs.rb

Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -385,7 +385,11 @@ def test_exec_thrown_error
385385
flunk
386386
rescue ExecJS::ProgramError => e
387387
assert e
388-
assert e.backtrace.join("\n").include?("(execjs):")
388+
# On newer versions of som OS implementations, we get
389+
# <eval>:1:20
390+
# instead of
391+
# @(execjs):1:20
392+
assert e.backtrace[0].match?(/\(execjs\):|\<eval\>:/)
389393
end
390394
end
391395

@@ -395,7 +399,11 @@ def test_eval_thrown_error
395399
flunk
396400
rescue ExecJS::ProgramError => e
397401
assert e
398-
assert e.backtrace.join("\n").include?("(execjs):")
402+
# On newer versions of som OS implementations, we get
403+
# <eval>:1:20
404+
# instead of
405+
# @(execjs):1:20
406+
assert e.backtrace[0].match?(/\(execjs\):|\<eval\>:/)
399407
end
400408
end
401409

@@ -405,7 +413,13 @@ def test_compile_thrown_error
405413
flunk
406414
rescue ExecJS::ProgramError => e
407415
assert e
408-
assert e.backtrace.join("\n").include?("(execjs):")
416+
# On newer versions of som OS implementations, we get
417+
# <eval>:1:20
418+
# instead of
419+
# @(execjs):1:20
420+
assert e.backtrace[0].match?(/\(execjs\):|\<eval\>:/)
421+
422+
# assert e.backtrace.join("\n").include?("(execjs):")
409423
end
410424
end
411425

0 commit comments

Comments
 (0)