Skip to content

Update Integration Tests - #241

Merged
JillRegan merged 2 commits into
mainfrom
jil/update-tests
Aug 12, 2026
Merged

Update Integration Tests#241
JillRegan merged 2 commits into
mainfrom
jil/update-tests

Conversation

@JillRegan

@JillRegan JillRegan commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

The old test account is being deleted, so the integration test now points at a new vault. Matches the equivalent change in the JS SDK.

  • src/onepassword/test_client.py: updated the hardcoded secret reference to the new vault and item.
  • .github/workflows/validate.yml — the fork job's example step referenced TEST_SERVICE_ACCOUNT_VAULT_ID, which doesn't exist as a repo secret, so that step has never worked. Repointed it at EXAMPLE_TESTS_OP_SERVICE_ACCOUNT_TOKEN / EXAMPLE_TESTS_OP_VAULT_ID, matching the trusted job.

@JillRegan
JillRegan marked this pull request as ready for review August 7, 2026 15:53
@JillRegan
JillRegan requested review from bertrmz and rishiy15 August 7, 2026 15:53
@JillRegan
JillRegan merged commit f301751 into main Aug 12, 2026
18 checks passed
@JillRegan
JillRegan deleted the jil/update-tests branch August 12, 2026 19:20
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