Skip to content

Conversation

@mitchell-as
Copy link
Collaborator

@mitchell-as mitchell-as commented Dec 19, 2025

We no longer have access to PLATFORM_API_TOKEN, which has the ability to delete both projects and users.

Since we're no longer creating users in tests (and needing to delete them), also remove the code for deleting users.

@mitchell-as mitchell-as force-pushed the mitchell/cp-1277 branch 3 times, most recently from be0b599 to bbeec44 Compare December 19, 2025 17:21
@mitchell-as mitchell-as changed the title Try UUID for temporary init project test. When deleting inited projects via integration tests, retry if the first attempt fails. Dec 19, 2025
@mitchell-as mitchell-as force-pushed the mitchell/cp-1277 branch 2 times, most recently from 26e4af7 to 64062e5 Compare December 19, 2025 20:08
@mitchell-as mitchell-as changed the title When deleting inited projects via integration tests, retry if the first attempt fails. Login as the integration test user during teardown. Dec 19, 2025
We no longer have access to the PLATFORM_API_TOKEN that was being used.
Remove references to `PLATFORM_API_TOKEN` secret, which we no longer have.
@mitchell-as mitchell-as requested a review from MDrakos December 19, 2025 20:33
@mitchell-as mitchell-as marked this pull request as ready for review December 19, 2025 20:33
@mitchell-as
Copy link
Collaborator Author

That c-cpp failure is unrelated to this PR. It's a Go repo!

Copy link
Member

@MDrakos MDrakos left a comment

Choose a reason for hiding this comment

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

Looks good!

@mitchell-as mitchell-as merged commit 00d88b9 into master Dec 19, 2025
11 of 12 checks passed
@mitchell-as mitchell-as deleted the mitchell/cp-1277 branch December 19, 2025 21:24
mitchell-as added a commit that referenced this pull request Jan 5, 2026
Login as the integration test user during teardown.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants