[DRIVERS-1868] Integration tests for observeSensitiveCommands Created: 26/Jul/21  Updated: 14/Dec/23  Resolved: 14/Dec/23

Status: Closed
Project: Drivers
Component/s: Unified Test Runner
Fix Version/s: None

Type: Task Priority: Unknown
Reporter: Jeremy Mikola Assignee: Jeremy Mikola
Resolution: Done Votes: 0
Labels: size-small, test-runner, yaml-test
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Issue split
split to CDRIVER-4104 Integration tests for observeSensitiv... Closed
split to CSHARP-3769 Integration tests for observeSensitiv... Closed
split to CXX-2325 Integration tests for observeSensitiv... Closed
split to GODRIVER-2107 Integration tests for observeSensitiv... Closed
split to MOTOR-797 Integration tests for observeSensitiv... Closed
split to NODE-3494 Implement support and integrate tests... Closed
split to PHPLIB-703 Integration tests for observeSensitiv... Closed
split to PYTHON-2842 Integration tests for observeSensitiv... Closed
split to RUBY-2752 Integration tests for observeSensitiv... Closed
split to RUST-944 Integration tests for observeSensitiv... Closed
split to JAVA-4264 Integration tests for observeSensitiv... Closed
Related
Driver Changes: Needed
Downstream Changes Summary:

Sync valid-pass tests for the unified test format to mongodb/specifications@6847048. Also note that some additional clarifications were added to the spec itself concerning implementation of observeSensitiveEvents (originally introduced in DRIVERS-1653); however, the behavior of the syntax has not been changed.

Driver Compliance:
Key Status/Resolution FixVersion
CDRIVER-4104 Done
CXX-2325 Done
CSHARP-3769 Duplicate
GODRIVER-2107 Done
JAVA-4264 Fixed 4.4.0
NODE-3494 Fixed 6.4.0
MOTOR-797 Duplicate
PYTHON-2842 Fixed 3.12.1, 4.0
PHPLIB-703 Fixed 1.10.0
RUBY-2752 Fixed 2.19.0
RUST-944 Fixed 2.0.0-beta.3
SWIFT-1288 Won't Do

 Description   

Summary

DRIVERS-1653 introduced observeSensitiveCommands to the unified test runner (schema version 1.5); however, it did not include integration tests for the new syntax. The test cases pertaining to sensitive command redaction are not sufficient to verify that drivers have implemented the new syntax completely. This issue will add integration tests to do so.

Motivation

Who is the affected end user?

Drivers.

How does this affect the end user?

Does not affect the end user.

How likely is it that this problem or use case will occur?

It's quite possible that observeSensitiveCommands can be implemented incorrectly if drivers are only using the redaction tests in DRIVERS-1653 to verify their implementations.

If the problem does occur, what are the consequences and how severe are they?

The risk is an incorrect test runner implementation.

Is this issue urgent?

This should be done alongside DRIVERS-1653, but individual drivers can make that call.

Is this ticket required by a downstream team?

No.

Is this ticket only for tests?

Only tests.



 Comments   
Comment by Githook User [ 29/Jul/21 ]

Author:

{'name': 'Jeremy Mikola', 'email': 'jmikola@gmail.com', 'username': 'jmikola'}

Message: DRIVERS-1868: Integration tests for observeSensitiveCommands (#1043)

Also adds implementation notes for observeSensitiveCommands
Branch: master
https://github.com/mongodb/specifications/commit/6847048c551431c8a328b9c1f0deeec158160d20

Comment by Jeremy Mikola [ 26/Jul/21 ]

https://github.com/mongodb/specifications/pull/1043

Generated at Thu Feb 08 08:24:11 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.