🚨 Linting

Author: pboling

.rubocop_gradual.lock

@@ -1,5 +1,5 @@
 {
-  "lib/omniauth-ldap/adaptor.rb:3925200886": [
+  "lib/omniauth-ldap/adaptor.rb:2775119926": [
     [68, 7, 413, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 105664470]
   ],
   "spec/integration/middleware_spec.rb:2185613788": [
@@ -12,7 +12,7 @@
     [4, 3, 12, "RSpec/BeforeAfterAll: Beware of using `before(:all)` as it may cause state to leak between tests. If you are using `rspec-rails`, and `use_transactional_fixtures` is enabled, then records created in `before(:all)` are not automatically rolled back.", 86334566],
     [70, 16, 5, "RSpec/ExpectActual: Provide the actual value you are testing to `expect(...)`.", 237881235]
   ],
-  "spec/omniauth-ldap/adaptor_spec.rb:1245381318": [
+  "spec/omniauth-ldap/adaptor_spec.rb:1839434473": [
     [3, 1, 38, "RSpec/SpecFilePathFormat: Spec path should end with `omni_auth/ldap/adaptor*_spec.rb`.", 1973618936],
     [239, 9, 26, "RSpec/StubbedMock: Prefer `allow` over `expect` when configuring a response.", 1924417310],
     [240, 9, 26, "RSpec/StubbedMock: Prefer `allow` over `expect` when configuring a response.", 1924417310]

CHANGELOG.md

@@ -32,7 +32,7 @@ Please file a bug if you notice a violation of semantic versioning.
 - Support for JSON bodies
 - Support custom LDAP attributes mapping
 - Raise a distinct error when LDAP server is unreachable
-  - Previously raised an invalid credentials authentication failure error, which is technically incorrect 
+  - Previously raised an invalid credentials authentication failure error, which is technically incorrect
 
 ### Changed
 

spec/omniauth-ldap/adaptor_spec.rb

@@ -311,13 +311,13 @@ def mock_conn(opts = {})
       expect(adaptor.last_password_policy_response.oid).to eq(ppolicy_oid)
     end
 
-    it "should raise a ConnectionError if the bind fails" do
-      adaptor = OmniAuth::LDAP::Adaptor.new({host: "192.168.1.126", method: 'plain', base: 'dc=score, dc=local', port: 389, uid: 'sAMAccountName', bind_dn: 'bind_dn', password: 'password'})
-      expect(adaptor.connection).to receive(:open).and_yield(adaptor.connection)
+    it "raises a ConnectionError if the bind fails" do
+      adaptor = described_class.new({host: "192.168.1.126", method: "plain", base: "dc=score, dc=local", port: 389, uid: "sAMAccountName", bind_dn: "bind_dn", password: "password"})
+      allow(adaptor.connection).to receive(:open).and_yield(adaptor.connection)
       # Net::LDAP#search returns nil if the operation was not successful
-      expect(adaptor.connection).to receive(:search).with(args).and_return(nil)
+      allow(adaptor.connection).to receive(:search).with(args).and_return(nil)
       expect(adaptor.connection).not_to receive(:bind)
-      expect { adaptor.bind_as(args) }.to raise_error OmniAuth::LDAP::Adaptor::ConnectionError
+      expect { adaptor.bind_as(args) }.to raise_error described_class::ConnectionError
     end
   end
 end