Skip to content

Commit

Permalink
Merge pull request #7900 from claui/syntax-test-add-hint
Browse files Browse the repository at this point in the history
In syntax tests, amend `skip` directives to carry the reason for skipping
  • Loading branch information
rolandwalker committed Dec 8, 2014
2 parents 22dbba6 + 5db030e commit bc9304f
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions test/syntax_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,13 @@
flags.unshift '--disable-all' if major_version.to_f > 1.8 # perf
backend_files.each do |file|
it "#{file} is valid Ruby" do
skip unless interpreter.exist?
skip("Ruby #{ major_version } not found") unless interpreter.exist?

# Reason for this hack is unknown. Travis appears to freeze,
# but only in one section of the build matrix.
skip if ENV.key?('TRAVIS_JOB_ID') and TestHelper.ruby18?
if ENV.key?('TRAVIS_JOB_ID') and TestHelper.ruby18?
skip("This test might cause Travis to freeze under Ruby 1.8")
end

args = flags + [ '--', file ]
shutup do
Expand Down

0 comments on commit bc9304f

Please sign in to comment.