Skip to content

test(strr-api): add integration harness and shared seed utilities#1608

Open
jimmypalelil wants to merge 3 commits intobcgov:mainfrom
jimmypalelil:pr2-strr-api-integration-infra
Open

test(strr-api): add integration harness and shared seed utilities#1608
jimmypalelil wants to merge 3 commits intobcgov:mainfrom
jimmypalelil:pr2-strr-api-integration-infra

Conversation

@jimmypalelil
Copy link
Copy Markdown
Collaborator

@jimmypalelil jimmypalelil commented May 5, 2026

Issue: #1543

Summary

  • add integration test harness wiring in tests/conftest.py and tests/integration/conftest.py, using shared reusable fixture plugins from strr_test_utils
  • add shared integration helpers and deterministic seed utilities in tests/integration/helpers.py, tests/integration/registration_seed.py, and tests/integration/application_seed.py
  • include supporting integration config updates in strr-api/Makefile, strr-api/pyproject.toml, and tests/python-test-utils/src/strr_test_utils/parent_fixtures.py
  • all the ci trigger paths have been updated since there is dependency on fixtures which comes from this path. so its best to run these ci checks just to ensure nothings broken with the new changes.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the BC Registry and Digital Services BSD 3-Clause License

Introduce integration test infrastructure (fixtures, helpers, seeds, and test config wiring)
so subsequent endpoint-focused PRs can add scenarios on top of a consistent runtime harness.
@jimmypalelil jimmypalelil force-pushed the pr2-strr-api-integration-infra branch from 7fb6aac to 15c136c Compare May 5, 2026 17:20
@jimmypalelil jimmypalelil marked this pull request as ready for review May 5, 2026 17:23
Add tests/python-test-utils/** to pull_request path filters so packages that depend on strr-test-utils validate on shared fixture updates.
pull_request:
paths:
- "strr-api/**"
- "tests/python-test-utils/**"
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

all the ci trigger paths have been updated since there is dependency on fixtures which comes from this path. so its best to run these ci checks just to ensure nothings broken with the new changes.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

Copy link
Copy Markdown
Collaborator

@JazzarKarim JazzarKarim left a comment

Choose a reason for hiding this comment

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

LGTM

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