Skip to content

test(api): check durable provider dependency across releases - #1053

Merged
iamgp merged 1 commit into
mainfrom
fix/release-safe-api-dependency-test
Oct 4, 2026
Merged

iamgp merged 1 commit into
mainfrom
fix/release-safe-api-dependency-test

Conversation

@iamgp

@iamgp iamgp commented Oct 4, 2026

Copy link
Copy Markdown
Collaborator

Scope

Make the API dependency contract test valid after release preparation. It currently hard-codes phlo-postgres>=0.16.0,<0.17, so the correctly rewritten >=0.17.0,<0.18 release manifest fails the test.

Parse the mandatory dependencies and assert that PostgreSQL is unconditional. Release-policy and version-drift checks continue to validate the dependency range. No production code, metadata, locks, or excluded PRs change.

This follows #1052, which is merged and has green post-merge release-candidate checks. The release bot refreshed #970 automatically; its only substantive remaining failure is this test. This PR does not merge or edit #970.

Verification

  • Reproduced the original assertion failure against the exact refreshed release manifest.
  • Corrected test passes against that manifest and rejects both missing and optional-only PostgreSQL dependencies.
  • API and release-policy tests: 979 passed, 10 integration tests deselected.
  • Ruff, C901, formatting, module headers, and whitespace checks pass.

Keep this follow-up unmerged until separately authorised.

@iamgp
iamgp marked this pull request as ready for review October 4, 2026 18:19
@coldtea-pr-lens

Copy link
Copy Markdown

Note

The title starts with test(api):, so PR Lens left this pull request undrawn. Comment @pr-lens draw to draw it

github.comment.notice: false in .github/pr-lens.yml turns this note off

@iamgp
iamgp added this pull request to the merge queue Oct 4, 2026
Merged via the queue into main with commit 1c99a68 Oct 4, 2026
37 checks passed
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.

1 participant