Skip to content

Conversation

@kraenhansen
Copy link
Contributor

Description

Small update to the connection-form KMS provider test, to make failures easier to debug by failing ealier.

Checklist

  • New tests and/or benchmarks are included
  • Documentation is changed or added
  • If this change updates the UI, screenshots/videos are added and a design review is requested
  • If this change could impact the load on the MongoDB cluster, please describe the expected and worst case impact
  • I have signed the MongoDB Contributor License Agreement (https://www.mongodb.com/legal/contributor-agreement)

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@kraenhansen kraenhansen self-assigned this Nov 17, 2025
@kraenhansen kraenhansen requested a review from a team as a code owner November 17, 2025 13:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds an assertion to the CSFLE tab test to verify that the KMS provider name input field updates correctly after user interaction. The change improves test reliability by catching input update failures earlier in the test execution.

  • Adds type annotation to the input element selector
  • Adds assertion to verify the input value matches the expected value after user interaction

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kraenhansen kraenhansen added no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) no release notes Fix or feature not for release notes labels Nov 17, 2025
@kraenhansen kraenhansen merged commit e6e86b5 into main Nov 17, 2025
64 of 65 checks passed
@kraenhansen kraenhansen deleted the kh/expect-kms-provider-rename branch November 17, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants