mongo 2.16.3 → 2.17.2

data/spec/spec_tests/data/sessions_unified/snapshot-sessions.yml

@@ -0,0 +1,482 @@
+description: snapshot-sessions
+
+schemaVersion: "1.0"
+
+runOnRequirements:
+  - minServerVersion: "5.0"
+    topologies: [replicaset, sharded-replicaset]
+
+createEntities:
+  - client:
+      id: &client0 client0
+      observeEvents: [ commandStartedEvent]
+      ignoreCommandMonitoringEvents: [ findAndModify, insert, update ]
+  - database:
+      id: &database0 database0
+      client: *client0
+      databaseName: &database0Name database0
+  - collection:
+      id: &collection0 collection0
+      database: *database0
+      collectionName: &collection0Name collection0
+      collectionOptions:
+        writeConcern: { w: majority }
+  - session:
+      id: session0
+      client: client0
+      sessionOptions:
+        snapshot: true
+  - session:
+      id: session1
+      client: client0
+      sessionOptions:
+        snapshot: true
+
+initialData:
+  - collectionName: *collection0Name
+    databaseName: *database0Name
+    documents:
+      - { _id: 1, x: 11 }
+      - { _id: 2, x: 11 }
+
+tests:
+- description: Find operation with snapshot
+  operations:
+  - name: find
+    object: collection0
+    arguments:
+      session: session0
+      filter: { _id: 1 }
+    expectResult:
+      - {_id: 1, x: 11}
+  - name: findOneAndUpdate
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+      update: { $inc: { x: 1 } }
+      returnDocument: After
+    expectResult: { _id: 1, x: 12 }
+  - name: find
+    object: collection0
+    arguments:
+      session: session1
+      filter: { _id: 1 }
+    expectResult:
+      - { _id: 1, x: 12 }
+  - name: findOneAndUpdate
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+      update: { $inc: { x: 1 } }
+      returnDocument: After
+    expectResult: { _id: 1, x: 13 }
+  - name: find
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+    expectResult:
+    - { _id: 1, x: 13 }
+  - name: find
+    object: collection0
+    arguments:
+      session: session0
+      filter: { _id: 1 }
+    expectResult:
+    - {_id: 1, x: 11}
+  - name: find
+    object: collection0
+    arguments:
+      session: session1
+      filter: { _id: 1 }
+    expectResult:
+    - {_id: 1, x: 12}
+  expectEvents:
+  - client: client0
+    events:
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            "$$exists": false
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+
+- description: Distinct operation with snapshot
+  operations:
+  - name: distinct
+    object: collection0
+    arguments:
+      fieldName: x
+      filter: {}
+      session: session0
+    expectResult:
+    - 11
+  - name: findOneAndUpdate
+    object: collection0
+    arguments:
+      filter: { _id: 2 }
+      update: { $inc: { x: 1 } }
+      returnDocument: After
+    expectResult: { _id: 2, x: 12 }
+  - name: distinct
+    object: collection0
+    arguments:
+      fieldName: x
+      filter: {}
+      session: session1
+    expectResult: [11, 12]
+  - name: findOneAndUpdate
+    object: collection0
+    arguments:
+      filter: { _id: 2 }
+      update: { $inc: { x: 1 } }
+      returnDocument: After
+    expectResult: { _id: 2, x: 13 }
+  - name: distinct
+    object: collection0
+    arguments:
+      fieldName: x
+      filter: {}
+    expectResult: [ 11, 13 ]
+  - name: distinct
+    object: collection0
+    arguments:
+      fieldName: x
+      filter: {}
+      session: session0
+    expectResult: [ 11 ]
+  - name: distinct
+    object: collection0
+    arguments:
+      fieldName: x
+      filter: {}
+      session: session1
+    expectResult: [ 11, 12 ]
+  expectEvents:
+  - client: client0
+    events:
+    - commandStartedEvent:
+        command:
+          distinct: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          distinct: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          distinct: collection0
+          readConcern:
+            "$$exists": false
+    - commandStartedEvent:
+        command:
+          distinct: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+    - commandStartedEvent:
+        command:
+          distinct: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+
+- description: Aggregate operation with snapshot
+  operations:
+  - name: aggregate
+    object: collection0
+    arguments:
+      pipeline:
+      - "$match": { _id: 1 }
+      session: session0
+    expectResult:
+    - { _id: 1, x: 11 }
+  - name: findOneAndUpdate
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+      update: { $inc: { x: 1 } }
+      returnDocument: After
+    expectResult: { _id: 1, x: 12 }
+  - name: aggregate
+    object: collection0
+    arguments:
+      pipeline:
+      - "$match":
+          _id: 1
+      session: session1
+    expectResult:
+    - {_id: 1, x: 12}
+  - name: findOneAndUpdate
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+      update: { $inc: { x: 1 } }
+      returnDocument: After
+    expectResult: { _id: 1, x: 13 }
+  - name: aggregate
+    object: collection0
+    arguments:
+      pipeline:
+      - "$match": { _id: 1 }
+    expectResult:
+    - { _id: 1, x: 13 }
+  - name: aggregate
+    object: collection0
+    arguments:
+      pipeline:
+      - "$match":
+          _id: 1
+      session: session0
+    expectResult:
+    - { _id: 1, x: 11 }
+  - name: aggregate
+    object: collection0
+    arguments:
+      pipeline:
+      - "$match": { _id: 1 }
+      session: session1
+    expectResult:
+    - { _id: 1, x: 12 }
+  expectEvents:
+  - client: client0
+    events:
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            "$$exists": false
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+
+- description: countDocuments operation with snapshot
+  operations:
+  - name: countDocuments
+    object: collection0
+    arguments:
+      filter: {}
+      session: session0
+    expectResult: 2
+  - name: countDocuments
+    object: collection0
+    arguments:
+      filter: {}
+      session: session0
+    expectResult: 2
+  expectEvents:
+  - client: client0
+    events:
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+
+- description: Mixed operation with snapshot
+  operations:
+  - name: find
+    object: collection0
+    arguments:
+      session: session0
+      filter: { _id: 1 }
+    expectResult:
+    - { _id: 1, x: 11 }
+  - name: findOneAndUpdate
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+      update: { $inc: { x: 1 } }
+      returnDocument: After
+    expectResult: { _id: 1, x: 12 }
+  - name: find
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+    expectResult:
+    - { _id: 1, x: 12 }
+  - name: aggregate
+    object: collection0
+    arguments:
+      pipeline:
+      - "$match":
+          _id: 1
+      session: session0
+    expectResult:
+    - { _id: 1, x: 11 }
+  - name: distinct
+    object: collection0
+    arguments:
+      fieldName: x
+      filter: {}
+      session: session0
+    expectResult: [ 11 ] 
+  expectEvents:
+  - client: client0
+    events:
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            "$$exists": false
+    - commandStartedEvent:
+        command:
+          aggregate: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+    - commandStartedEvent:
+        command:
+          distinct: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+
+- description: Write commands with snapshot session do not affect snapshot reads
+  operations:
+  - name: find
+    object: collection0
+    arguments:
+      filter: {}
+      session: session0
+  - name: insertOne
+    object: collection0
+    arguments:
+      document:
+        _id: 22
+        x: 33
+  - name: updateOne
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+      update: { $inc: { x: 1 } }
+  - name: find
+    object: collection0
+    arguments:
+      filter: { _id: 1 }
+      session: session0
+    expectResult:
+    - {_id: 1, x: 11}
+  expectEvents:
+  - client: client0
+    events:
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": true
+
+- description: First snapshot read does not send atClusterTime
+  operations:
+  - name: find
+    object: collection0
+    arguments:
+      filter: {}
+      session: session0
+  expectEvents:
+  - client: client0
+    events:
+    - commandStartedEvent:
+        command:
+          find: collection0
+          readConcern:
+            level: snapshot
+            atClusterTime:
+              "$$exists": false
+        commandName: find
+        databaseName: database0
+
+- description: StartTransaction fails in snapshot session
+  operations:
+  - name: startTransaction
+    object: session0
+    expectError:
+      isError: true
+      isClientError: true
+      errorContains: Transactions are not supported in snapshot sessions

data/spec/spec_tests/sessions_unified_spec.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+
+require 'runners/unified'
+
+base = "#{CURRENT_PATH}/spec_tests/data/sessions_unified"
+SESSIONS_UNIFIED_TESTS = Dir.glob("#{base}/**/*.yml").sort
+
+describe 'Sessions unified spec tests' do
+  define_unified_spec_tests(base, SESSIONS_UNIFIED_TESTS)
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongo
 version: !ruby/object:Gem::Version
-  version: 2.16.3
+  version: 2.17.2
 platform: ruby
 authors:
 - Tyler Brock
@@ -235,6 +235,8 @@ files:
 - lib/mongo/error/server_certificate_revoked.rb
 - lib/mongo/error/session_ended.rb
 - lib/mongo/error/sessions_not_supported.rb
+- lib/mongo/error/snapshot_session_invalid_server_version.rb
+- lib/mongo/error/snapshot_session_transaction_prohibited.rb
 - lib/mongo/error/socket_error.rb
 - lib/mongo/error/socket_timeout_error.rb
 - lib/mongo/error/unchangeable_collection_option.rb
@@ -1019,7 +1021,18 @@ files:
 - spec/spec_tests/data/crud/write/updateOne-collation.yml
 - spec/spec_tests/data/crud/write/updateOne-pre_2.6.yml
 - spec/spec_tests/data/crud/write/updateOne.yml
+- spec/spec_tests/data/crud_unified/aggregate-let.yml
+- spec/spec_tests/data/crud_unified/aggregate-write-readPreference.yml
+- spec/spec_tests/data/crud_unified/db-aggregate-write-readPreference.yml
+- spec/spec_tests/data/crud_unified/deleteMany-let.yml
+- spec/spec_tests/data/crud_unified/deleteOne-let.yml
 - spec/spec_tests/data/crud_unified/estimatedDocumentCount.yml
+- spec/spec_tests/data/crud_unified/find-let.yml
+- spec/spec_tests/data/crud_unified/findOneAndDelete-let.yml
+- spec/spec_tests/data/crud_unified/findOneAndReplace-let.yml
+- spec/spec_tests/data/crud_unified/findOneAndUpdate-let.yml
+- spec/spec_tests/data/crud_unified/updateMany-let.yml
+- spec/spec_tests/data/crud_unified/updateOne-let.yml
 - spec/spec_tests/data/crud_unified/updateWithPipelines.yml
 - spec/spec_tests/data/crud_v2/aggregate-merge.yml
 - spec/spec_tests/data/crud_v2/aggregate-out-readConcern.yml
@@ -1470,6 +1483,10 @@ files:
 - spec/spec_tests/data/server_selection_rtt/value_test_3.yml
 - spec/spec_tests/data/server_selection_rtt/value_test_4.yml
 - spec/spec_tests/data/server_selection_rtt/value_test_5.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions-not-supported-client-error.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions-not-supported-server-error.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions-unsupported-ops.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions.yml
 - spec/spec_tests/data/transactions/abort.yml
 - spec/spec_tests/data/transactions/bulk.yml
 - spec/spec_tests/data/transactions/causal-consistency.yml
@@ -1552,6 +1569,7 @@ files:
 - spec/spec_tests/seed_list_discovery_spec.rb
 - spec/spec_tests/server_selection_rtt_spec.rb
 - spec/spec_tests/server_selection_spec.rb
+- spec/spec_tests/sessions_unified_spec.rb
 - spec/spec_tests/transactions_api_spec.rb
 - spec/spec_tests/transactions_spec.rb
 - spec/spec_tests/transactions_unified_spec.rb
@@ -1655,7 +1673,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -1985,8 +2003,19 @@ test_files:
 - spec/spec_tests/data/sdam_integration/cancel-server-check.yml
 - spec/spec_tests/data/sdam_integration/find-shutdown-error.yml
 - spec/spec_tests/data/sdam_integration/connectTimeoutMS.yml
+- spec/spec_tests/data/crud_unified/findOneAndReplace-let.yml
 - spec/spec_tests/data/crud_unified/updateWithPipelines.yml
+- spec/spec_tests/data/crud_unified/aggregate-write-readPreference.yml
+- spec/spec_tests/data/crud_unified/db-aggregate-write-readPreference.yml
+- spec/spec_tests/data/crud_unified/aggregate-let.yml
+- spec/spec_tests/data/crud_unified/deleteMany-let.yml
+- spec/spec_tests/data/crud_unified/findOneAndDelete-let.yml
 - spec/spec_tests/data/crud_unified/estimatedDocumentCount.yml
+- spec/spec_tests/data/crud_unified/findOneAndUpdate-let.yml
+- spec/spec_tests/data/crud_unified/find-let.yml
+- spec/spec_tests/data/crud_unified/deleteOne-let.yml
+- spec/spec_tests/data/crud_unified/updateMany-let.yml
+- spec/spec_tests/data/crud_unified/updateOne-let.yml
 - spec/spec_tests/data/server_selection_rtt/first_value_zero.yml
 - spec/spec_tests/data/server_selection_rtt/value_test_5.yml
 - spec/spec_tests/data/server_selection_rtt/value_test_1.yml
@@ -2532,6 +2561,10 @@ test_files:
 - spec/spec_tests/data/unified/valid-fail/operation-failure.yml
 - spec/spec_tests/data/transactions_unified/mongos-unpin.yml
 - spec/spec_tests/data/collection_management/timeseries-collection.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions-unsupported-ops.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions-not-supported-client-error.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions-not-supported-server-error.yml
+- spec/spec_tests/data/sessions_unified/snapshot-sessions.yml
 - spec/spec_tests/data/seed_list_discovery/replica-set/longer-parent-in-return.yml
 - spec/spec_tests/data/seed_list_discovery/replica-set/uri-with-auth.yml
 - spec/spec_tests/data/seed_list_discovery/replica-set/not-enough-parts.yml
@@ -2578,6 +2611,7 @@ test_files:
 - spec/spec_tests/command_monitoring_unified_spec.rb
 - spec/spec_tests/client_side_encryption_spec.rb
 - spec/spec_tests/connection_string_spec.rb
+- spec/spec_tests/sessions_unified_spec.rb
 - spec/spec_tests/versioned_api_spec.rb
 - spec/spec_tests/crud_spec.rb
 - spec/spec_tests/change_streams_unified_spec.rb