Skip to content
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

test with localstack and github actions #59

Merged
merged 41 commits into from
Aug 22, 2024

Conversation

samuelpcabral
Copy link
Member

No description provided.

Copy link

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
20 0 0 20 100 37.382234s

Passed Tests

Name ⏱️ Duration Suite
Test Wait For Log 0.133 s Cloudwatch
Test Log Insights 30.069 s Cloudwatch
Test Put And Query Item 0.577 s Dynamo
Test Delete And Update Key 0.164 s Dynamo
Test Delete And Update Nested Key 0.094 s Dynamo
Test Delete Item 0.059 s Dynamo
Test Insert New Key 0.067 s Dynamo
Test Put And Query Item With Sort Key 0.077 s Dynamo
Test Delete And Update Key With Sort Key 0.132 s Dynamo
Test Upload And List 0.478 s S3
Test Upload And List With Folder 0.064 s S3
Test Remove And List With Prefix 0.065 s S3
Test Download 0.040 s S3
Test Get Content 0.033 s S3
Test Copy 0.056 s S3
Test Metadata 0.036 s S3
Send and Receive Message 0.116 s Sqs
Send and Receive Messages 0.085 s Sqs
Invalid Queue Name 0.011 s Sqs
Delete Messages 5.025 s Sqs

@samuelpcabral
Copy link
Member Author

Hello @paguilera It was necessary to create a new pull request for the github actions to execute. I will cancel the previous one and leave everything in this one.

There are still some improvements to do in github actions, but at least this way we already have the tests running here.

For now, I left it to run when pushing any branch and when asking for a merge request, (because the tests are fast). Later on we can see if this makes sense or if we just leave it in the master.

@samuelpcabral samuelpcabral changed the title Samuelpcabral test localstack linux test with localstack and github actions Aug 21, 2024
Copy link
Collaborator

@paguilera paguilera left a comment

Choose a reason for hiding this comment

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

With this updated PR, everything works as expected. Looks good to me.

@samuelpcabral samuelpcabral merged commit 0bb1b6f into master Aug 22, 2024
4 checks passed
@samuelpcabral samuelpcabral deleted the samuelpcabral-test_localstack_linux branch August 22, 2024 08:14
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.

Issue with installation add tests for keywords Test development workflow with Localstack
2 participants