Skip to content

feat(CI): add e2e test#72

Merged
kate-goldenring merged 1 commit intospinframework:mainfrom
kate-goldenring:e2e-test
Feb 12, 2026
Merged

feat(CI): add e2e test#72
kate-goldenring merged 1 commit intospinframework:mainfrom
kate-goldenring:e2e-test

Conversation

@kate-goldenring
Copy link
Contributor

Runs e2e tests against a test application that uses an ElasticMQ SQS service. Help us catch runtime errors before releasing and better localdev experience. Thank you copilot for the scripting assistance.

@kate-goldenring
Copy link
Contributor Author

I was hoping we would not need AWS creds since we are not actually using any AWS service; however, CI is failing with:

Creating SQS queue 'test-queue'...

Unable to locate credentials. You can configure credentials by running "aws login".

@itowlson
Copy link
Collaborator

This could be down to us using AWS libraries. I am not sure if we could get away with fake creds? Or if there is a non-AWS-bound library we could use that would still work with SQS?

@kate-goldenring kate-goldenring force-pushed the e2e-test branch 4 times, most recently from d13682e to 3e31b09 Compare February 11, 2026 16:38
@kate-goldenring
Copy link
Contributor Author

@itowlson seems like it was just failing because the expected AWS_* auth env vars were not set. Setting them to dummy values resolved it -- and the test passed 🎉 . Now just a cargo audit issue that is also on main. I'll put up a pr for that

Runs e2e tests against a test application that uses an ElasticMQ SQS service.

Signed-off-by: Kate Goldenring <kate.goldenring@fermyon.com>
Copy link
Contributor

@fibonacci1729 fibonacci1729 left a comment

Choose a reason for hiding this comment

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

🎉

@kate-goldenring kate-goldenring merged commit eb4dd61 into spinframework:main Feb 12, 2026
8 checks passed
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.

3 participants