DRIVERS-2900 Test that durations are included on relevant pool events. (

#1562) DRIVERS-2469
mongodb · Apr 19, 2024 · fe86082 · fe86082
1 parent 688b369
commit fe86082
Show file tree

Hide file tree

Showing 17 changed files with 53 additions and 16 deletions.
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/README.md b/source/connection-monitoring-and-pooling/tests/cmap-format/README.md
@@ -46,6 +46,7 @@ All Unit Tests have some of the following fields:
  - `type`: The type of event emitted
  - `address`: The address of the pool emitting the event
  - `connectionId`: The id of a connection associated with the event
+ - `duration`: The event duration
  - `options`: Options used to create the pool
  - `reason`: A reason giving more information on why the event was emitted
 - `ignore`: An array of event names to ignore

diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkin-make-available.json b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkin-make-available.json
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkin-make-available.yml b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkin-make-available.yml
@@ -12,12 +12,14 @@ events:
  - type: ConnectionCheckedOut
  connectionId: 1
  address: 42
+ duration: 42
  - type: ConnectionCheckedIn
  connectionId: 1
  address: 42
  - type: ConnectionCheckedOut
  connectionId: 1
  address: 42
+ duration: 42
 ignore:
  - ConnectionPoolCreated
  - ConnectionPoolReady

diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-connection.json b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-connection.json
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-connection.yml b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-connection.yml
@@ -13,9 +13,11 @@ events:
  - type: ConnectionReady
  connectionId: 1
  address: 42
+ duration: 42
  - type: ConnectionCheckedOut
  connectionId: 1
  address: 42
+ duration: 42
 ignore:
  - ConnectionPoolReady
  - ConnectionPoolCreated
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-error-closed.json b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-error-closed.json
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-error-closed.yml b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-checkout-error-closed.yml
@@ -21,6 +21,7 @@ events:
  - type: ConnectionCheckedOut
  address: 42
  connectionId: 42
+ duration: 42
  - type: ConnectionCheckedIn
  address: 42
  connectionId: 42
@@ -30,6 +31,7 @@ events:
  address: 42
  - type: ConnectionCheckOutFailed
  address: 42
+ duration: 42
  reason: poolClosed
 ignore:
  - ConnectionPoolReady

diff --git a/...nection-monitoring-and-pooling/tests/cmap-format/pool-checkout-maxConnecting-timeout.json b/...nection-monitoring-and-pooling/tests/cmap-format/pool-checkout-maxConnecting-timeout.json
diff --git a/...nnection-monitoring-and-pooling/tests/cmap-format/pool-checkout-maxConnecting-timeout.yml b/...nnection-monitoring-and-pooling/tests/cmap-format/pool-checkout-maxConnecting-timeout.yml
@@ -60,6 +60,7 @@ events:
  - type: ConnectionCheckOutFailed
  reason: timeout
  address: 42
+ duration: 42
 ignore:
  - ConnectionCreated
  - ConnectionCheckedIn

diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-clears-waitqueue.json b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-clears-waitqueue.json
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-clears-waitqueue.yml b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-clears-waitqueue.yml
@@ -38,6 +38,7 @@ events:
  address: 42
  - type: ConnectionCheckedOut
  address: 42
+ duration: 42
  - type: ConnectionCheckOutStarted
  address: 42
  - type: ConnectionCheckOutStarted
@@ -47,12 +48,15 @@ events:
  - type: ConnectionCheckOutFailed
  reason: connectionError
  address: 42
+ duration: 42
  - type: ConnectionCheckOutFailed
  reason: connectionError
  address: 42
+ duration: 42
  - type: ConnectionCheckOutFailed
  reason: connectionError
  address: 42
+ duration: 42
 ignore:
  - ConnectionPoolReady
  - ConnectionPoolCleared

diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-ready.json b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-ready.json
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-ready.yml b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-clear-ready.yml
@@ -20,15 +20,18 @@ events:
  - type: ConnectionCheckedOut
  address: 42
  connectionId: 42
+ duration: 42
  - type: ConnectionPoolCleared
  address: 42
  - type: ConnectionCheckOutFailed
  address: 42
+ duration: 42
  reason: connectionError
  - type: ConnectionPoolReady
  address: 42
  - type: ConnectionCheckedOut
  address: 42
+ duration: 42
 ignore:
  - ConnectionPoolCreated
  - ConnectionReady

diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-ready.json b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-ready.json
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/pool-ready.yml b/source/connection-monitoring-and-pooling/tests/cmap-format/pool-ready.yml
@@ -17,6 +17,7 @@ events:
  - type: ConnectionCheckOutFailed
  reason: connectionError
  address: 42
+ duration: 42
  - type: ConnectionPoolReady
  address: 42
  - type: ConnectionCheckOutStarted
@@ -25,6 +26,7 @@ events:
  address: 42
  - type: ConnectionCheckedOut
  address: 42
+ duration: 42
 ignore:
  - ConnectionPoolCreated
  - ConnectionReady
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/wait-queue-timeout.json b/source/connection-monitoring-and-pooling/tests/cmap-format/wait-queue-timeout.json
diff --git a/source/connection-monitoring-and-pooling/tests/cmap-format/wait-queue-timeout.yml b/source/connection-monitoring-and-pooling/tests/cmap-format/wait-queue-timeout.yml
@@ -32,11 +32,13 @@ events:
  - type: ConnectionCheckedOut
  connectionId: 42
  address: 42
+ duration: 42
  - type: ConnectionCheckOutStarted
  address: 42
  - type: ConnectionCheckOutFailed
  reason: timeout
  address: 42
+ duration: 42
  - type: ConnectionCheckedIn
  connectionId: 42
  address: 42