GODRIVER-2002 Remove 'slaveok' from source code and most spec tests (#713)

Author: benjirewis

data/change-streams/legacy/change-streams-errors.yml

@@ -81,7 +81,7 @@ tests:
       mode: { times: 1 }
       data:
           failCommands: ["getMore"]
-          errorCode: 216 # An error code that's not on the old blacklist or whitelist
+          errorCode: 216 # An error code that's not on the old deny or allow lists
           closeConnection: false
     target: collection
     topology:

data/change-streams/legacy/change-streams-resume-whitelist.json renamed to data/change-streams/legacy/change-streams-resume-allowlist.json

@@ -778,7 +778,7 @@
       }
     },
     {
-      "description": "change stream resumes after NotMaster",
+      "description": "change stream resumes after NotWritablePrimary",
       "minServerVersion": "4.2",
       "maxServerVersion": "4.2.99",
       "failPoint": {
@@ -1069,7 +1069,7 @@
       }
     },
     {
-      "description": "change stream resumes after NotMasterNoSlaveOk",
+      "description": "change stream resumes after NotPrimaryNoSecondaryOk",
       "minServerVersion": "4.2",
       "maxServerVersion": "4.2.99",
       "failPoint": {
@@ -1166,7 +1166,7 @@
       }
     },
     {
-      "description": "change stream resumes after NotMasterOrSecondary",
+      "description": "change stream resumes after NotPrimaryOrSecondary",
       "minServerVersion": "4.2",
       "maxServerVersion": "4.2.99",
       "failPoint": {

data/change-streams/legacy/change-streams-resume-whitelist.yml renamed to data/change-streams/legacy/change-streams-resume-allowlist.yml

@@ -522,7 +522,7 @@ tests:
             x:
               $numberInt: "1"
   -
-    description: "change stream resumes after NotMaster"
+    description: "change stream resumes after NotWritablePrimary"
     minServerVersion: "4.2"
     maxServerVersion: "4.2.99"
     failPoint:
@@ -717,7 +717,7 @@ tests:
             x:
               $numberInt: "1"
   -
-    description: "change stream resumes after NotMasterNoSlaveOk"
+    description: "change stream resumes after NotPrimaryNoSecondaryOk"
     minServerVersion: "4.2"
     maxServerVersion: "4.2.99"
     failPoint:
@@ -782,7 +782,7 @@ tests:
             x:
               $numberInt: "1"
   -
-    description: "change stream resumes after NotMasterOrSecondary"
+    description: "change stream resumes after NotPrimaryOrSecondary"
     minServerVersion: "4.2"
     maxServerVersion: "4.2.99"
     failPoint:

data/change-streams/legacy/change-streams-resume-errorLabels.json

@@ -661,7 +661,7 @@
       }
     },
     {
-      "description": "change stream resumes after NotMaster",
+      "description": "change stream resumes after NotWritablePrimary",
       "minServerVersion": "4.3.1",
       "failPoint": {
         "configureFailPoint": "failGetMoreAfterCursorCheckout",
@@ -943,7 +943,7 @@
       }
     },
     {
-      "description": "change stream resumes after NotMasterNoSlaveOk",
+      "description": "change stream resumes after NotPrimaryNoSecondaryOk",
       "minServerVersion": "4.3.1",
       "failPoint": {
         "configureFailPoint": "failGetMoreAfterCursorCheckout",
@@ -1037,7 +1037,7 @@
       }
     },
     {
-      "description": "change stream resumes after NotMasterOrSecondary",
+      "description": "change stream resumes after NotPrimaryOrSecondary",
       "minServerVersion": "4.3.1",
       "failPoint": {
         "configureFailPoint": "failGetMoreAfterCursorCheckout",

data/change-streams/legacy/change-streams-resume-errorLabels.yml

@@ -451,7 +451,7 @@ tests:
             x:
               $numberInt: "1"
   -
-    description: "change stream resumes after NotMaster"
+    description: "change stream resumes after NotWritablePrimary"
     minServerVersion: "4.3.1"
     failPoint:
       configureFailPoint: failGetMoreAfterCursorCheckout
@@ -643,7 +643,7 @@ tests:
             x:
               $numberInt: "1"
   -
-    description: "change stream resumes after NotMasterNoSlaveOk"
+    description: "change stream resumes after NotPrimaryNoSecondaryOk"
     minServerVersion: "4.3.1"
     failPoint:
       configureFailPoint: failGetMoreAfterCursorCheckout
@@ -707,7 +707,7 @@ tests:
             x:
               $numberInt: "1"
   -
-    description: "change stream resumes after NotMasterOrSecondary"
+    description: "change stream resumes after NotPrimaryOrSecondary"
     minServerVersion: "4.3.1"
     failPoint:
       configureFailPoint: failGetMoreAfterCursorCheckout
@@ -1026,7 +1026,7 @@ tests:
           fullDocument:
             x:
               $numberInt: "1"
-  # The next two tests ensure that the driver only uses the error label, not the whitelist.
+  # The next two tests ensure that the driver only uses the error label, not the allow list.
   -
     description: "change stream resumes if error contains ResumableChangeStreamError"
     minServerVersion: "4.3.1"
@@ -1035,7 +1035,7 @@ tests:
       mode: { times: 1 }
       data:
           failCommands: ["getMore"]
-          errorCode: 50 # Use an error code that does not have the whitelist label by default
+          errorCode: 50 # Use an error code that does not have the allow list label by default
           closeConnection: false
           errorLabels: ["ResumableChangeStreamError"]
     target: collection
@@ -1097,11 +1097,11 @@ tests:
     description: "change stream does not resume if error does not contain ResumableChangeStreamError"
     minServerVersion: "4.3.1"
     failPoint:
-      configureFailPoint: failCommand # failCommand will not add the whitelist error label
+      configureFailPoint: failCommand # failCommand will not add the allow list error label
       mode: { times: 1 }
       data:
           failCommands: ["getMore"]
-          errorCode: 6 # Use an error code that is on the whitelist
+          errorCode: 6 # Use an error code that is on the allow list
           closeConnection: false
     target: collection
     topology:

data/retryable-writes/bulkWrite-errorLabels.json renamed to data/retryable-writes/legacy/bulkWrite-errorLabels.json

@@ -118,7 +118,7 @@
       }
     },
     {
-      "description": "InsertOne succeeds after NotMaster",
+      "description": "InsertOne succeeds after NotWritablePrimary",
       "failPoint": {
         "configureFailPoint": "failCommand",
         "mode": {
@@ -167,7 +167,7 @@
       }
     },
     {
-      "description": "InsertOne succeeds after NotMasterOrSecondary",
+      "description": "InsertOne succeeds after NotPrimaryOrSecondary",
       "failPoint": {
         "configureFailPoint": "failCommand",
         "mode": {
@@ -216,7 +216,7 @@
       }
     },
     {
-      "description": "InsertOne succeeds after NotMasterNoSlaveOk",
+      "description": "InsertOne succeeds after NotPrimaryNoSecondaryOk",
       "failPoint": {
         "configureFailPoint": "failCommand",
         "mode": {

data/retryable-writes/bulkWrite-errorLabels.yml renamed to data/retryable-writes/legacy/bulkWrite-errorLabels.yml

@@ -56,7 +56,7 @@ tests:
                     - { _id: 1, x: 11 }
                     - { _id: 2, x: 22 }
     -
-        description: "InsertOne succeeds after NotMaster"
+        description: "InsertOne succeeds after NotWritablePrimary"
         failPoint:
             configureFailPoint: failCommand
             mode: { times: 1 }
@@ -78,7 +78,7 @@ tests:
                     - { _id: 2, x: 22 }
                     - { _id: 3, x: 33 }
     -
-        description: "InsertOne succeeds after NotMasterOrSecondary"
+        description: "InsertOne succeeds after NotPrimaryOrSecondary"
         failPoint:
             configureFailPoint: failCommand
             mode: { times: 1 }
@@ -100,7 +100,7 @@ tests:
                     - { _id: 2, x: 22 }
                     - { _id: 3, x: 33 }
     -
-        description: "InsertOne succeeds after NotMasterNoSlaveOk"
+        description: "InsertOne succeeds after NotPrimaryNoSecondaryOk"
         failPoint:
             configureFailPoint: failCommand
             mode: { times: 1 }