Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

Attempt to fix #361 #362

Closed
wants to merge 1 commit into from
Closed

Conversation

jonhyman
Copy link
Contributor

@arthurnn I think that the test that existed for this functionality (below) is broken, but in my timebox for today I can't figure out the trace in the mock replica set. I don't see the auth coming in, but also if this test were valid then I'd expect #361 to not be an issue. Thoughts?

context "when credentials exist on the node" do

          before do
            node.instance_variable_set(:@credentials, { database => [ user, pass ]})
            replica_set_node.unauthorized_on_next_message!
          end

          it "retries the operation" do
            expect(read.execute(node)).to_not be_nil
          end
        end

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.68%) to 93.0% when pulling bfe3ffe on jonhyman:feature/retry-read into 10abbf3 on mongoid:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.68%) to 93.0% when pulling bfe3ffe on jonhyman:feature/retry-read into 10abbf3 on mongoid:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.68%) to 93.0% when pulling bfe3ffe on jonhyman:feature/retry-read into 10abbf3 on mongoid:master.

@jonhyman
Copy link
Contributor Author

#380 merged this in.

@jonhyman jonhyman closed this Jun 16, 2015
@jonhyman jonhyman deleted the feature/retry-read branch June 16, 2015 19:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants