-
Notifications
You must be signed in to change notification settings - Fork 65
Add account e2e tests #262
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add account e2e tests #262
Conversation
Could you mention this in Can be done in a followup PR, as I we also need to update there instructions about setting Connect env vars. |
| @sh -c 'unset OP_SERVICE_ACCOUNT_TOKEN; OP_CONNECT_TOKEN="$(OP_CONNECT_TOKEN)" OP_CONNECT_HOST="$(OP_CONNECT_HOST)" TF_ACC=1 go test -v ./test/e2e/... -timeout 30m' | ||
|
|
||
| test-e2e-account: ## Run e2e test using account methodology with Touch ID (requires OP_ACCOUNT, manual only). Creates multiple items to verify biometrics is only prompted once. | ||
| @test -n "$(OP_ACCOUNT)" || (echo "[ERROR] OP_ACCOUNT environment variable is not set."; exit 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add the same check for OP_TEST_VAULT_NAME env var, so it doesn't run tests if it's not set.
97edd98
into
feature/configure-provider-to-use-sdk
✨ Summary
🔗 Resolves:
✅ Checklist
🕵️ Review Notes &⚠️ Risks