Permalink
Browse files

Remove warnings "assigned but unused variable" in spec

  • Loading branch information...
1 parent e3141b9 commit e515908f10c9f780165058902fb11730eb5d9111 @kachick kachick committed with sferik Jan 10, 2013
Showing with 4 additions and 4 deletions.
  1. +4 −4 spec/omniauth/failure_endpoint_spec.rb
@@ -29,24 +29,24 @@
'omniauth.error.strategy' => ExampleStrategy.new({}) } }
it "is a redirect" do
- status, head, body = *subject.call(env)
+ status, _, _ = *subject.call(env)
expect(status).to eq(302)
end
it "includes the SCRIPT_NAME" do
- status, head, body = *subject.call(env.merge('SCRIPT_NAME' => '/random'))
+ _, head, _ = *subject.call(env.merge('SCRIPT_NAME' => '/random'))
expect(head['Location']).to eq('/random/auth/failure?message=invalid_request&strategy=test')
end
it "respects the configured path prefix" do
OmniAuth.config.stub(:path_prefix => '/boo')
- status, head, body = *subject.call(env)
+ _, head, _ = *subject.call(env)
expect(head["Location"]).to eq('/boo/failure?message=invalid_request&strategy=test')
end
it "includes the origin (escaped) if one is provided" do
env.merge! 'omniauth.origin' => '/origin-example'
- status, head, body = *subject.call(env)
+ _, head, _ = *subject.call(env)
expect(head['Location']).to be_include('&origin=%2Forigin-example')
end
end

0 comments on commit e515908

Please sign in to comment.