mongo 2.5.3 → 2.6.0

data/spec/support/sdam/sharded/mongos_disconnect.yml

@@ -11,14 +11,18 @@ phases: [
 
                     ok: 1,
                     ismaster: true,
-                    msg: "isdbgrid"
+                    msg: "isdbgrid",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }],
 
                 ["b:27017", {
 
                     ok: 1,
                     ismaster: true,
-                    msg: "isdbgrid"
+                    msg: "isdbgrid",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 
@@ -76,7 +80,9 @@ phases: [
 
                     ok: 1,
                     ismaster: true,
-                    msg: "isdbgrid"
+                    msg: "isdbgrid",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }],
         ],
 

data/spec/support/sdam/sharded/multiple_mongoses.yml

@@ -11,14 +11,18 @@ phases: [
 
                     ok: 1,
                     ismaster: true,
-                    msg: "isdbgrid"
+                    msg: "isdbgrid",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }],
 
                 ["b:27017", {
 
                     ok: 1,
                     ismaster: true,
-                    msg: "isdbgrid"
+                    msg: "isdbgrid",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/sharded/non_mongos_removed.yml

@@ -11,7 +11,9 @@ phases: [
 
                     ok: 1,
                     ismaster: true,
-                    msg: "isdbgrid"
+                    msg: "isdbgrid",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }],
 
                 ["b:27017", {
@@ -19,7 +21,9 @@ phases: [
                     ok: 1,
                     ismaster: true,
                     hosts: ["b:27017"],
-                    setName: "rs"
+                    setName: "rs",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/sharded/too_new.yml

@@ -0,0 +1,36 @@
+description: "Multiple mongoses with large minWireVersion"
+uri: "mongodb://a,b"
+phases: [
+    {
+        responses: [
+                ["a:27017", {
+                    ok: 1,
+                    ismaster: true,
+                    msg: "isdbgrid",
+                    minWireVersion: 999,
+                    maxWireVersion: 1000
+                }],
+                ["b:27017", {
+                    ok: 1,
+                    ismaster: true,
+                    msg: "isdbgrid"
+                }]
+        ],
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "Mongos",
+                    setName:
+                },
+                "b:27017": {
+                    type: "Mongos",
+                    setName:
+                }
+            },
+            topologyType: "Sharded",
+            logicalSessionTimeoutMinutes: null,
+            setName: ,
+            compatible: false
+        }
+    }
+]

data/spec/support/sdam/sharded/too_old.yml

@@ -0,0 +1,36 @@
+description: "Multiple mongoses with default maxWireVersion of 0"
+uri: "mongodb://a,b"
+phases: [
+    {
+        responses: [
+                ["a:27017", {
+                    ok: 1,
+                    ismaster: true,
+                    msg: "isdbgrid",
+                    minWireVersion: 2,
+                    maxWireVersion: 6
+                }],
+                ["b:27017", {
+                    ok: 1,
+                    ismaster: true,
+                    msg: "isdbgrid"
+                }]
+        ],
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "Mongos",
+                    setName:
+                },
+                "b:27017": {
+                    type: "Mongos",
+                    setName:
+                }
+            },
+            topologyType: "Sharded",
+            logicalSessionTimeoutMinutes: null,
+            setName: ,
+            compatible: false
+        }
+    }
+]

data/spec/support/sdam/single/compatible.yml

@@ -0,0 +1,26 @@
+description: "Standalone with large maxWireVersion"
+uri: "mongodb://a"
+phases: [
+    {
+        responses: [
+                ["a:27017", {
+                    ok: 1,
+                    ismaster: true,
+                    minWireVersion: 0,
+                    maxWireVersion: 6
+                }]
+        ],
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "Standalone",
+                    setName:
+                }
+            },
+            topologyType: "Single",
+            logicalSessionTimeoutMinutes: null,
+            setName: ,
+            compatible: true
+        }
+    }
+]

data/spec/support/sdam/single/direct_connection_external_ip.yml

@@ -12,7 +12,9 @@ phases: [
                     ok: 1,
                     ismaster: true,
                     hosts: ["b:27017"],  # Internal IP.
-                    setName: "rs"
+                    setName: "rs",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/direct_connection_mongos.yml

@@ -11,7 +11,9 @@ phases: [
 
                     ok: 1,
                     ismaster: true,
-                    msg: "isdbgrid"
+                    msg: "isdbgrid",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/direct_connection_rsarbiter.yml

@@ -13,7 +13,9 @@ phases: [
                    ismaster: false,
                    arbiterOnly: true,
                    hosts: ["a:27017", "b:27017"],
-                   setName: "rs"
+                   setName: "rs",
+                   minWireVersion: 0,
+                   maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/direct_connection_rsprimary.yml

@@ -12,7 +12,9 @@ phases: [
                    ok: 1,
                    ismaster: true,
                    hosts: ["a:27017", "b:27017"],
-                   setName: "rs"
+                   setName: "rs",
+                   minWireVersion: 0,
+                   maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/direct_connection_rssecondary.yml

@@ -13,7 +13,9 @@ phases: [
                     ismaster: false,
                     secondary: true,
                     hosts: ["a:27017", "b:27017"],
-                    setName: "rs"
+                    setName: "rs",
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/direct_connection_slave.yml

@@ -10,7 +10,9 @@ phases: [
                 ["a:27017", {
 
                     ok: 1,
-                    ismaster: false
+                    ismaster: false,
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/direct_connection_standalone.yml

@@ -10,7 +10,9 @@ phases: [
                 ["a:27017", {
 
                     ok: 1,
-                    ismaster: true
+                    ismaster: true,
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/not_ok_response.yml

@@ -10,13 +10,17 @@ phases: [
                 ["a:27017", {
 
                     ok: 1,
-                    ismaster: true
+                    ismaster: true,
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }],
 
                 ["a:27017", {
 
                     ok: 0,
-                    ismaster: true
+                    ismaster: true,
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/standalone_removed.yml

@@ -10,7 +10,9 @@ phases: [
                 ["a:27017", {
 
                     ok: 1,
-                    ismaster: true
+                    ismaster: true,
+                    minWireVersion: 0,
+                    maxWireVersion: 6
                 }]
         ],
 

data/spec/support/sdam/single/too_new.yml

@@ -0,0 +1,26 @@
+description: "Standalone with large minWireVersion"
+uri: "mongodb://a"
+phases: [
+    {
+        responses: [
+                ["a:27017", {
+                    ok: 1,
+                    ismaster: true,
+                    minWireVersion: 999,
+                    maxWireVersion: 1000
+                }]
+        ],
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "Standalone",
+                    setName:
+                }
+            },
+            topologyType: "Single",
+            logicalSessionTimeoutMinutes: null,
+            setName: ,
+            compatible: false
+        }
+    }
+]

data/spec/support/sdam/single/too_old.yml

@@ -0,0 +1,24 @@
+description: "Standalone with default maxWireVersion of 0"
+uri: "mongodb://a"
+phases: [
+    {
+        responses: [
+                ["a:27017", {
+                    ok: 1,
+                    ismaster: true
+                }]
+        ],
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "Standalone",
+                    setName:
+                }
+            },
+            topologyType: "Single",
+            logicalSessionTimeoutMinutes: null,
+            setName: ,
+            compatible: false
+        }
+    }
+]

data/spec/support/shared/session.rb

@@ -756,3 +756,110 @@ shared_examples 'an operation updating cluster time' do
     end
   end
 end
+
+shared_examples 'an operation not using a session' do
+  require_sessions
+
+  describe 'operation execution' do
+
+    context 'when the client has a session' do
+
+      let(:session) do
+        client.start_session
+      end
+
+      let(:server_session) do
+        session.instance_variable_get(:@server_session)
+      end
+
+      let!(:before_last_use) do
+        server_session.last_use
+      end
+
+      let!(:before_operation_time) do
+        session.operation_time
+      end
+
+      let!(:operation_result) do
+        operation
+      end
+
+      after do
+        session.end_session
+      end
+
+      it 'does not send session id in command' do
+        expect(command).not_to have_key('lsid')
+      end
+
+      it 'does not update the last use value' do
+        expect(server_session.last_use).to eq(before_last_use)
+      end
+
+      it 'does not update the operation time value' do
+        expect(session.operation_time).to eq(before_operation_time)
+      end
+
+      it 'does not close the session when the operation completes' do
+        expect(session.ended?).to be(false)
+      end
+    end
+
+    context 'when the session is ended before it is used' do
+      let(:session) do
+        client.start_session
+      end
+
+      before do
+        session.end_session
+      end
+
+      let(:operation_result) do
+        operation
+      end
+
+      it 'does not raise an exception' do
+        expect {
+          operation_result
+        }.not_to raise_exception(Mongo::Error::InvalidSession)
+      end
+    end
+  end
+end
+
+shared_examples 'a failed operation not using a session' do
+  require_sessions
+
+  context 'when the operation fails' do
+
+    let!(:before_last_use) do
+      session.instance_variable_get(:@server_session).last_use
+    end
+
+    let!(:before_operation_time) do
+      session.operation_time
+    end
+
+    let!(:operation_result) do
+      sleep 0.2
+      begin; failed_operation; rescue => e; e; end
+    end
+
+    let(:session) do
+      client.start_session
+    end
+
+    it 'raises an error' do
+      expect([Mongo::Error::OperationFailure,
+              Mongo::Error::BulkWriteError]).to include(operation_result.class)
+    end
+
+    it 'does not update the last use value' do
+      expect(session.instance_variable_get(:@server_session).last_use).to eq(before_last_use)
+    end
+
+    it 'does not update the operation time value' do
+      expect(session.operation_time).to eq(before_operation_time)
+    end
+  end
+end