Skip to content

Relax assertion when checking exception message #1712

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 27, 2025

Conversation

alcaeus
Copy link
Member

@alcaeus alcaeus commented May 27, 2025

This PR updates a test assertion for compatibility with the upcoming minor version of the extension. The newer libmongocrypt version bundled in that version adds the view name to the exception message, so the test would fail. This PR intentionally targets v2.1 as that version is tested with the upcoming minor version of the extension and would cause issues if we have to release a bugfix after merging mongodb/mongo-php-driver#1829.

@alcaeus alcaeus requested a review from GromNaN May 27, 2025 07:47
@alcaeus alcaeus requested a review from a team as a code owner May 27, 2025 07:47
@alcaeus alcaeus merged commit e72d7b8 into mongodb:v2.1 May 27, 2025
31 checks passed
@alcaeus alcaeus deleted the relax-error-message-assertion branch May 27, 2025 08:41
alcaeus added a commit that referenced this pull request Jun 4, 2025
* v2.1: (50 commits)
  Fix missing placeholder replacement in release output (#1717)
  Use more lax assertion when checking exception message (#1712)
  Test against stable 2.1 version of the extension (#1698)
  Use assume_role command before accessing secrets for atlas-data-lake tests
  PHPLIB-1656: Stop testing with MongoDB 4.0 (#1693)
  Make serverless tasks patch-only (#1692)
  Bump tests/specifications from `cea6c7d` to `9d0d3f0` (#1690)
  Bump tests/drivers-evergreen-tools from `21b4109` to `32fd8eb` (#1691)
  PHPLIB-847, PHPLIB-1627: BulkWriteCommand API (#1630)
  Bump tests/specifications from `0c41c8b` to `cea6c7d` (#1688)
  Bump tests/drivers-evergreen-tools from `fa58b39` to `21b4109` (#1689)
  Test next next minor PHP version with PHP 8.4 (#1685)
  Bump tests/drivers-evergreen-tools from `9be5587` to `fa58b39` (#1679)
  Bump tests/specifications from `43d2c7b` to `0c41c8b` (#1680)
  Bump tests/drivers-evergreen-tools from `cec0139` to `9be5587` (#1676)
  Bump tests/drivers-evergreen-tools from `d1e19b1` to `cec0139` (#1675)
  Bump tests/specifications from `6118deb` to `43d2c7b` (#1674)
  Bump tests/drivers-evergreen-tools from `f53da28` to `d1e19b1` (#1673)
  Test against stable 2.0 version of the extension (#1663)
  Bump tests/specifications from `f354960` to `6118deb`
  ...
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