CSHARP-5061: Enable tests that require failCommand with appName on initial handshake before 4.9 (#1530)

adelinowona · web-flow · commit 4a669d3f84b6 · 2024-11-06T17:22:23.000-05:00
diff --git a/specifications/load-balancers/tests/sdam-error-handling.json b/specifications/load-balancers/tests/sdam-error-handling.json
@@ -263,7 +263,7 @@
       "description": "errors during the initial connection hello are ignored",
       "runOnRequirements": [
         {
-          "minServerVersion": "4.9"
+          "minServerVersion": "4.4.7"
         }
       ],
       "operations": [
diff --git a/specifications/load-balancers/tests/sdam-error-handling.yml b/specifications/load-balancers/tests/sdam-error-handling.yml
@@ -141,9 +141,8 @@ tests:
   # to the same mongos on which the failpoint is set.
   - description: errors during the initial connection hello are ignored
     runOnRequirements:
-      # Server version 4.9+ is needed to set a fail point on the initial
-      # connection handshake with the appName filter due to SERVER-49336.
-      - minServerVersion: '4.9'
+      # Require SERVER-49336 for failCommand + appName on the initial handshake.
+      - minServerVersion: '4.4.7'
     operations:
       - name: failPoint
         object: testRunner
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/hello-command-error.json b/specifications/server-discovery-and-monitoring/tests/unified/hello-command-error.json
@@ -3,7 +3,7 @@
   "schemaVersion": "1.10",
   "runOnRequirements": [
     {
-      "minServerVersion": "4.9",
+      "minServerVersion": "4.4.7",
       "serverless": "forbid",
       "topologies": [
         "single",
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/hello-command-error.yml b/specifications/server-discovery-and-monitoring/tests/unified/hello-command-error.yml
@@ -4,8 +4,8 @@ description: hello-command-error
 schemaVersion: "1.10"
 
 runOnRequirements:
-    # failCommand appName requirements
-  - minServerVersion: "4.9"
+  # Require SERVER-49336 for failCommand + appName on the initial handshake.
+  - minServerVersion: "4.4.7"
     serverless: forbid
     topologies: [ single, replicaset, sharded ]
 
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/hello-network-error.json b/specifications/server-discovery-and-monitoring/tests/unified/hello-network-error.json
@@ -3,7 +3,7 @@
   "schemaVersion": "1.10",
   "runOnRequirements": [
     {
-      "minServerVersion": "4.9",
+      "minServerVersion": "4.4.7",
       "serverless": "forbid",
       "topologies": [
         "single",
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/hello-network-error.yml b/specifications/server-discovery-and-monitoring/tests/unified/hello-network-error.yml
@@ -4,8 +4,8 @@ description: hello-network-error
 schemaVersion: "1.10"
 
 runOnRequirements:
-    # failCommand appName requirements
-  - minServerVersion: "4.9"
+  # Require SERVER-49336 for failCommand + appName on the initial handshake.
+  - minServerVersion: "4.4.7"
     serverless: forbid
     topologies: [ single, replicaset, sharded ]
 
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/interruptInUse-pool-clear.json b/specifications/server-discovery-and-monitoring/tests/unified/interruptInUse-pool-clear.json
@@ -3,7 +3,7 @@
   "schemaVersion": "1.11",
   "runOnRequirements": [
     {
-      "minServerVersion": "4.9",
+      "minServerVersion": "4.4",
       "serverless": "forbid",
       "topologies": [
         "replicaset",
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/interruptInUse-pool-clear.yml b/specifications/server-discovery-and-monitoring/tests/unified/interruptInUse-pool-clear.yml
@@ -5,7 +5,7 @@ schemaVersion: "1.11"
 
 runOnRequirements:
   # failCommand appName requirements
-  - minServerVersion: "4.9"
+  - minServerVersion: "4.4"
     serverless: forbid
     topologies: [ replicaset, sharded ]
 
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/minPoolSize-error.json b/specifications/server-discovery-and-monitoring/tests/unified/minPoolSize-error.json
@@ -3,7 +3,7 @@
   "schemaVersion": "1.10",
   "runOnRequirements": [
     {
-      "minServerVersion": "4.9",
+      "minServerVersion": "4.4.7",
       "serverless": "forbid",
       "topologies": [
         "single"
diff --git a/specifications/server-discovery-and-monitoring/tests/unified/minPoolSize-error.yml b/specifications/server-discovery-and-monitoring/tests/unified/minPoolSize-error.yml
@@ -4,8 +4,8 @@ description: minPoolSize-error
 schemaVersion: "1.10"
 
 runOnRequirements:
-    # failCommand appName requirements
-  - minServerVersion: "4.9"
+  # Require SERVER-49336 for failCommand + appName on the initial handshake.
+  - minServerVersion: "4.4.7"
     serverless: forbid
     topologies:
       - single

Original file line number	Diff line number	Diff line change
`@@ -263,7 +263,7 @@`
`263`	`263`	`"description": "errors during the initial connection hello are ignored",`
`264`	`264`	`"runOnRequirements": [`
`265`	`265`	`{`
`266`		`- "minServerVersion": "4.9"`
	`266`	`+ "minServerVersion": "4.4.7"`
`267`	`267`	`}`
`268`	`268`	`],`
`269`	`269`	`"operations": [`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`"schemaVersion": "1.10",`
`4`	`4`	`"runOnRequirements": [`
`5`	`5`	`{`
`6`		`- "minServerVersion": "4.9",`
	`6`	`+ "minServerVersion": "4.4.7",`
`7`	`7`	`"serverless": "forbid",`
`8`	`8`	`"topologies": [`
`9`	`9`	`"single",`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`"schemaVersion": "1.11",`
`4`	`4`	`"runOnRequirements": [`
`5`	`5`	`{`
`6`		`- "minServerVersion": "4.9",`
	`6`	`+ "minServerVersion": "4.4",`
`7`	`7`	`"serverless": "forbid",`
`8`	`8`	`"topologies": [`
`9`	`9`	`"replicaset",`