Skip to content

Commit

Permalink
GODRIVER-2657 Restrict getNonce spec tests to < 6.2. (#1147)
Browse files Browse the repository at this point in the history
  • Loading branch information
benjirewis authored and prestonvasquez committed Apr 4, 2023
1 parent 0351e7f commit a46ad91
Show file tree
Hide file tree
Showing 4 changed files with 49 additions and 0 deletions.
20 changes: 20 additions & 0 deletions testdata/command-monitoring/unified/redacted-commands.json
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,11 @@
},
{
"description": "getnonce",
"runOnRequirements": [
{
"maxServerVersion": "6.1.99"
}
],
"operations": [
{
"name": "runCommand",
Expand Down Expand Up @@ -293,6 +298,11 @@
},
{
"description": "copydbgetnonce",
"runOnRequirements": [
{
"maxServerVersion": "3.6.99"
}
],
"operations": [
{
"name": "runCommand",
Expand Down Expand Up @@ -328,6 +338,11 @@
},
{
"description": "copydbsaslstart",
"runOnRequirements": [
{
"maxServerVersion": "4.0.99"
}
],
"operations": [
{
"name": "runCommand",
Expand Down Expand Up @@ -363,6 +378,11 @@
},
{
"description": "copydb",
"runOnRequirements": [
{
"maxServerVersion": "4.0.99"
}
],
"operations": [
{
"name": "runCommand",
Expand Down
8 changes: 8 additions & 0 deletions testdata/command-monitoring/unified/redacted-commands.yml
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,8 @@ tests:
payload: { $$exists: false }

- description: "getnonce"
runOnRequirements:
- maxServerVersion: 6.1.99 # getnonce removed as of 6.2 via SERVER-71007
operations:
- name: runCommand
object: *database
Expand Down Expand Up @@ -159,6 +161,8 @@ tests:
roles: { $$exists: false }

- description: "copydbgetnonce"
runOnRequirements:
- maxServerVersion: 3.6.99 # copydbgetnonce was removed as of 4.0 via SERVER-32276
operations:
- name: runCommand
object: *database
Expand All @@ -176,6 +180,8 @@ tests:
command: { copydbgetnonce: { $$exists: false } }

- description: "copydbsaslstart"
runOnRequirements:
- maxServerVersion: 4.0.99 # copydbsaslstart was removed as of 4.2 via SERVER-36211
operations:
- name: runCommand
object: *database
Expand All @@ -193,6 +199,8 @@ tests:
command: { copydbsaslstart: { $$exists: false } }

- description: "copydb"
runOnRequirements:
- maxServerVersion: 4.0.99 # copydb was removed as of 4.2 via SERVER-36257
operations:
- name: runCommand
object: *database
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,11 @@
"tests": [
{
"description": "getnonce is observed with observeSensitiveCommands=true",
"runOnRequirements": [
{
"maxServerVersion": "6.1.99"
}
],
"operations": [
{
"name": "runCommand",
Expand Down Expand Up @@ -106,6 +111,11 @@
},
{
"description": "getnonce is not observed with observeSensitiveCommands=false",
"runOnRequirements": [
{
"maxServerVersion": "6.1.99"
}
],
"operations": [
{
"name": "runCommand",
Expand All @@ -127,6 +137,11 @@
},
{
"description": "getnonce is not observed by default",
"runOnRequirements": [
{
"maxServerVersion": "6.1.99"
}
],
"operations": [
{
"name": "runCommand",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ createEntities:

tests:
- description: "getnonce is observed with observeSensitiveCommands=true"
runOnRequirements:
- maxServerVersion: 6.1.99 # getnonce removed as of 6.2 via SERVER-71007
operations:
- name: runCommand
object: *databaseObserveSensitiveCommands
Expand All @@ -57,6 +59,8 @@ tests:
nonce: { $$exists: false }

- description: "getnonce is not observed with observeSensitiveCommands=false"
runOnRequirements:
- maxServerVersion: 6.1.99 # getnonce removed as of 6.2 via SERVER-71007
operations:
- name: runCommand
object: *databaseDoNotObserveSensitiveCommands
Expand All @@ -68,6 +72,8 @@ tests:
events: []

- description: "getnonce is not observed by default"
runOnRequirements:
- maxServerVersion: 6.1.99 # getnonce removed as of 6.2 via SERVER-71007
operations:
- name: runCommand
object: *databaseDoNotObserveSensitiveCommandsByDefault
Expand Down

0 comments on commit a46ad91

Please sign in to comment.