wip PHPLIB-1228: Run legacy transaction tests on LB topologies

mongodb · Sep 27, 2023 · 340a84d · 340a84d
1 parent bb63143
commit 340a84d
Show file tree

Hide file tree

Showing 33 changed files with 910 additions and 696 deletions.
diff --git a/tests/SpecTests/transactions/abort.json b/tests/SpecTests/transactions/abort.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/bulk.json b/tests/SpecTests/transactions/bulk.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/causal-consistency.json b/tests/SpecTests/transactions/causal-consistency.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/commit.json b/tests/SpecTests/transactions/commit.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/count.json b/tests/SpecTests/transactions/count.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/create-collection.json b/tests/SpecTests/transactions/create-collection.json
@@ -4,7 +4,8 @@
       "minServerVersion": "4.3.4",
       "topology": [
         "replicaset",
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/create-index.json b/tests/SpecTests/transactions/create-index.json
@@ -4,7 +4,8 @@
       "minServerVersion": "4.3.4",
       "topology": [
         "replicaset",
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/delete.json b/tests/SpecTests/transactions/delete.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/error-labels-blockConnection.json b/tests/SpecTests/transactions/error-labels-blockConnection.json
@@ -0,0 +1,160 @@
+{
+  "runOn": [
+    {
+      "minServerVersion": "4.2",
+      "topology": [
+        "replicaset",
+        "sharded",
+        "load-balanced"
+      ]
+    }
+  ],
+  "database_name": "transaction-tests",
+  "collection_name": "test",
+  "data": [],
+  "tests": [
+    {
+      "description": "add RetryableWriteError and UnknownTransactionCommitResult labels to connection errors",
+      "clientOptions": {
+        "socketTimeoutMS": 100
+      },
+      "failPoint": {
+        "configureFailPoint": "failCommand",
+        "mode": {
+          "times": 2
+        },
+        "data": {
+          "failCommands": [
+            "commitTransaction"
+          ],
+          "blockConnection": true,
+          "blockTimeMS": 150
+        }
+      },
+      "operations": [
+        {
+          "name": "startTransaction",
+          "object": "session0"
+        },
+        {
+          "name": "insertOne",
+          "object": "collection",
+          "arguments": {
+            "session": "session0",
+            "document": {
+              "_id": 1
+            }
+          },
+          "result": {
+            "insertedId": 1
+          }
+        },
+        {
+          "name": "commitTransaction",
+          "object": "session0",
+          "result": {
+            "errorLabelsContain": [
+              "RetryableWriteError",
+              "UnknownTransactionCommitResult"
+            ],
+            "errorLabelsOmit": [
+              "TransientTransactionError"
+            ]
+          }
+        },
+        {
+          "name": "commitTransaction",
+          "object": "session0"
+        }
+      ],
+      "expectations": [
+        {
+          "command_started_event": {
+            "command": {
+              "insert": "test",
+              "documents": [
+                {
+                  "_id": 1
+                }
+              ],
+              "ordered": true,
+              "readConcern": null,
+              "lsid": "session0",
+              "txnNumber": {
+                "$numberLong": "1"
+              },
+              "startTransaction": true,
+              "autocommit": false,
+              "writeConcern": null
+            },
+            "command_name": "insert",
+            "database_name": "transaction-tests"
+          }
+        },
+        {
+          "command_started_event": {
+            "command": {
+              "commitTransaction": 1,
+              "lsid": "session0",
+              "txnNumber": {
+                "$numberLong": "1"
+              },
+              "startTransaction": null,
+              "autocommit": false,
+              "writeConcern": null
+            },
+            "command_name": "commitTransaction",
+            "database_name": "admin"
+          }
+        },
+        {
+          "command_started_event": {
+            "command": {
+              "commitTransaction": 1,
+              "lsid": "session0",
+              "txnNumber": {
+                "$numberLong": "1"
+              },
+              "startTransaction": null,
+              "autocommit": false,
+              "writeConcern": {
+                "w": "majority",
+                "wtimeout": 10000
+              }
+            },
+            "command_name": "commitTransaction",
+            "database_name": "admin"
+          }
+        },
+        {
+          "command_started_event": {
+            "command": {
+              "commitTransaction": 1,
+              "lsid": "session0",
+              "txnNumber": {
+                "$numberLong": "1"
+              },
+              "startTransaction": null,
+              "autocommit": false,
+              "writeConcern": {
+                "w": "majority",
+                "wtimeout": 10000
+              }
+            },
+            "command_name": "commitTransaction",
+            "database_name": "admin"
+          }
+        }
+      ],
+      "outcome": {
+        "collection": {
+          "data": [
+            {
+              "_id": 1
+            }
+          ]
+        }
+      }
+    }
+  ]
+}
diff --git a/tests/SpecTests/transactions/error-labels.json b/tests/SpecTests/transactions/error-labels.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ],
       "serverless": "forbid"
     }

diff --git a/tests/SpecTests/transactions/errors-client.json b/tests/SpecTests/transactions/errors-client.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/errors.json b/tests/SpecTests/transactions/errors.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/findOneAndDelete.json b/tests/SpecTests/transactions/findOneAndDelete.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/findOneAndReplace.json b/tests/SpecTests/transactions/findOneAndReplace.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/findOneAndUpdate.json b/tests/SpecTests/transactions/findOneAndUpdate.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/insert.json b/tests/SpecTests/transactions/insert.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/isolation.json b/tests/SpecTests/transactions/isolation.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/mongos-pin-auto.json b/tests/SpecTests/transactions/mongos-pin-auto.json
@@ -3,7 +3,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ],
       "serverless": "forbid"
     }

diff --git a/tests/SpecTests/transactions/mongos-recovery-token.json b/tests/SpecTests/transactions/mongos-recovery-token.json
@@ -3,7 +3,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ],
       "serverless": "forbid"
     }

diff --git a/tests/SpecTests/transactions/pin-mongos.json b/tests/SpecTests/transactions/pin-mongos.json
@@ -3,7 +3,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ],
       "serverless": "forbid"
     }

diff --git a/tests/SpecTests/transactions/read-concern.json b/tests/SpecTests/transactions/read-concern.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/read-pref.json b/tests/SpecTests/transactions/read-pref.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/reads.json b/tests/SpecTests/transactions/reads.json
@@ -9,7 +9,8 @@
     {
       "minServerVersion": "4.1.8",
       "topology": [
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],

diff --git a/tests/SpecTests/transactions/retryable-abort-errorLabels.json b/tests/SpecTests/transactions/retryable-abort-errorLabels.json
@@ -4,7 +4,8 @@
       "minServerVersion": "4.3.1",
       "topology": [
         "replicaset",
-        "sharded"
+        "sharded",
+        "load-balanced"
       ]
     }
   ],