Skip to content

test(integration): use pre-provisioned no-user service account (FB-4691) - #539

Merged
fstr merged 2 commits into
mainfrom
florian/fb-4691-static-sa-fixtures
Oct 7, 2026
Merged

fstr merged 2 commits into
mainfrom
florian/fb-4691-static-sa-fixtures

Conversation

@fstr

@fstr fstr commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

test_account_no_user (sync and async) no longer creates a service account at runtime. It uses the credentials of a pre-provisioned service account without a user, read from SERVICE_ID_NO_USER / SERVICE_SECRET_NO_USER.

Changes

  • Replace the per-test service account fixtures in both V2 conftest.py files with one session fixture in tests/integration/conftest.py.
  • Pass the new secrets through the integration and nightly workflows.

Testing

  • Integration tests pass, including both test_account_no_user tests.

🤖 Generated with Claude Code

test_account_no_user created a service account at runtime, which requires
the CI credential to be an organization admin. Read the credentials of the
static ecosystem_ci_sa_no_user fixture from SERVICE_ID_NO_USER /
SERVICE_SECRET_NO_USER instead, so the CI credential can be reduced to
system_admin on a single account.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fstr
fstr requested a review from a team as a code owner October 7, 2026 11:57
@fstr
fstr requested review from a team and removed request for a team October 7, 2026 12:39
…c-sa-fixtures

# Conflicts:
#	.github/workflows/integration-tests.yml
@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

@fstr
fstr merged commit d697c99 into main Oct 7, 2026
12 checks passed
@fstr
fstr deleted the florian/fb-4691-static-sa-fixtures branch October 7, 2026 12:45
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