FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

PYTHON-5715 Add appName to OIDC test failpoints by blink1073 · Pull Request #2697 · mongodb/mongo-python-driver · GitHub

Repository navigation

PYTHON-5715 Add appName to OIDC test failpoints - #2697

Merged
blink1073 merged 1 commit into
mongodb:masterfrom
blink1073:DRIVERS-3197
Feb 9, 2026
Merged

blink1073 merged 1 commit into
mongodb:masterfrom
blink1073:DRIVERS-3197

Conversation

blink1073 commented Feb 5, 2026 •
edited by aclark4life
Loading

Copy link
Copy Markdown
Member

PYTHON-5715

Changes in this PR

Ensure appName is used in the failCommands in the OIDC tests and sync to the updated OIDC test.

Test Plan

https://spruce.mongodb.com/version/6984eb71f7849a0007a963e9/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC

Checklist

Checklist for Author

  • Did you update the changelog (if necessary)?
  • Is there test coverage?
  • Is any followup work tracked in a JIRA ticket? If so, add link(s).

Checklist for Reviewer

  • Does the title of the PR reference a JIRA Ticket?
  • Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
  • Is all relevant documentation (README or docstring) updated?

aclark4life left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM! Why dict over SON?

Copy link
Copy Markdown
Member Author

SON is redundant now that dictionaries are ordered by default, we had previously migrated some internal usages of SON to dict but missed this one.

blink1073 merged commit 0441761 into mongodb:master Feb 9, 2026
79 of 81 checks passed
blink1073 deleted the DRIVERS-3197 branch February 9, 2026 20:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL