Skip to content

Read auth token from env vars#7

Merged
carole-lavillonniere merged 2 commits intomainfrom
drg-359-inline-token-authentication-with-environment-variable-for-ci
Feb 3, 2026
Merged

Read auth token from env vars#7
carole-lavillonniere merged 2 commits intomainfrom
drg-359-inline-token-authentication-with-environment-variable-for-ci

Conversation

@carole-lavillonniere
Copy link
Collaborator

Read LOCALSTACK_AUTH_TOKEN from env vars and pass value to localstack-pro container.

We now have 3 test cases:

  • Test: TestStartCommandSucceedsWithValidToken
    Condition: Valid token
    Expected: Container running

  • Test: TestStartCommandFailsWithoutToken
    Condition: No token
    Expected: error "LOCALSTACK_AUTH_TOKEN is not set"

  • Test: TestStartCommandFailsWithInvalidToken
    Condition: Invalid token
    Expected: error"License activation failed"

Testing

LOCALSTACK_AUTH_TOKEN=ls-xxx make test-integration
or
cp env.example .env, set token and make test-integration

Copy link
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

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

LGTM!

Base automatically changed from auth-err to main February 3, 2026 13:42
@carole-lavillonniere carole-lavillonniere force-pushed the drg-359-inline-token-authentication-with-environment-variable-for-ci branch from 6025783 to 0df30e7 Compare February 3, 2026 13:44
@carole-lavillonniere carole-lavillonniere merged commit 1692ad9 into main Feb 3, 2026
@carole-lavillonniere carole-lavillonniere deleted the drg-359-inline-token-authentication-with-environment-variable-for-ci branch February 3, 2026 13:44
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