Skip to content

Commit ea0d3fa

Browse files
authored
Skip broken tests. (#2115)
This is a temporary fix for #2113 until rack/rack#2316 is merged.
1 parent 5e15985 commit ea0d3fa

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

test/routing_test.rb

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -255,6 +255,9 @@ class RoutingTest < Minitest::Test
255255
get '/', {}, "PATH_INFO" => ""
256256
assert ok?
257257
assert_equal 'worked', body
258+
rescue Rack::Lint::LintError => error
259+
# Temporary fix for https://github.com/sinatra/sinatra/issues/2113
260+
skip error.message
258261
end
259262

260263
it 'matches empty PATH_INFO to "" if a route is defined for ""' do
@@ -273,6 +276,9 @@ class RoutingTest < Minitest::Test
273276
get '/', {}, "PATH_INFO" => ""
274277
assert ok?
275278
assert_equal 'worked', body
279+
rescue Rack::Lint::LintError => error
280+
# Temporary fix for https://github.com/sinatra/sinatra/issues/2113
281+
skip error.message
276282
end
277283

278284
it 'takes multiple definitions of a route' do

0 commit comments

Comments
 (0)