Skip to content

Commit

Permalink
re-enable ntlm tests
Browse files Browse the repository at this point in the history
  • Loading branch information
pcai committed Jun 6, 2024
1 parent daca176 commit 9bb751d
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 9 deletions.
2 changes: 1 addition & 1 deletion httpi.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ Gem::Specification.new do |s|
s.add_dependency 'base64'
s.add_dependency 'mutex_m'

s.add_development_dependency 'rubyntlm', '~> 0.3.2'
s.add_development_dependency 'rubyntlm', '~> 0.6.4'
s.add_development_dependency 'rake', '~> 13.0'
s.add_development_dependency 'rspec', '~> 3.5'
s.add_development_dependency 'mocha', '~> 0.13'
Expand Down
3 changes: 1 addition & 2 deletions spec/httpi/adapter/httpclient_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -215,8 +215,7 @@
end
end

# TODO: ntlm_openssl3 https://github.com/WinRb/rubyntlm/issues/57
xit "supports NTLM authentication" do
it "supports NTLM authentication" do
request = HTTPI::Request.new(@server.url + "ntlm-auth")

request.auth.ntlm("tester", "vReqSoafRe5O")
Expand Down
9 changes: 3 additions & 6 deletions spec/integration/net_http_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -138,8 +138,7 @@
to raise_error(HTTPI::NotSupportedError, /does not support HTTP digest authentication/)
end

# TODO: ntlm_openssl3 https://github.com/WinRb/rubyntlm/issues/57
xit "supports ntlm authentication" do
it "supports ntlm authentication" do
request = HTTPI::Request.new(@server.url + "ntlm-auth")
request.auth.ntlm("tester", "vReqSoafRe5O")

Expand Down Expand Up @@ -167,8 +166,7 @@
HTTPI::Adapter::NetHTTP.any_instance.unstub(:check_net_ntlm_version!)
end

# TODO: ntlm_openssl3 https://github.com/WinRb/rubyntlm/issues/57
xit "does check ntlm when ntlm authentication is requested" do
it "does check ntlm when ntlm authentication is requested" do
request = HTTPI::Request.new(@server.url + "ntlm-auth")
request.auth.ntlm("tester", "vReqSoafRe5O")

Expand All @@ -185,8 +183,7 @@
HTTPI::Adapter::NetHTTP.any_instance.unstub(:ntlm_version)
end

# TODO: ntlm_openssl3 https://github.com/WinRb/rubyntlm/issues/57
xit "does not crash when authenticate header is missing (on second request)" do
it "does not crash when authenticate header is missing (on second request)" do
request = HTTPI::Request.new(@server.url + "ntlm-auth")
request.auth.ntlm("tester", "vReqSoafRe5O")

Expand Down

0 comments on commit 9bb751d

Please sign in to comment.