-
-
Notifications
You must be signed in to change notification settings - Fork 499
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
Adding mockserver module #2085
Adding mockserver module #2085
Conversation
✅ Deploy Preview for testcontainers-go ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Hi @jespino this LGTM. I added a few notes, all of them are not blockers, but the one regarding the tag of the Docker image. PTAL 🙏
Thanks for adding a new module! The community will appreciate it! 🚀
Co-authored-by: Manuel de la Peña <social.mdelapenya@gmail.com>
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.
LGTM, thanks for adding a new module! 🚀
* main: (25 commits) chore(deps): bump github.com/dvsekhvalnov/jose2go in /modules/pulsar (testcontainers#2136) fix: skip-host-cache option removed in latest MySQL 8.3.0 version (testcontainers#2130) chore: skip assertions for Docker Rootless (testcontainers#2135) pin Docker images version (testcontainers#2129) enable golangci-lint for examples (testcontainers#2128) chore(deps): bump github.com/neo4j/neo4j-go-driver/v5 in /modules/neo4j (testcontainers#2098) enable golangci-lint for redis module (testcontainers#2126) Go install gotestsum and golangci-lint (testcontainers#2127) improve OSSF score (testcontainers#2125) chore: run make lint on new modules (testcontainers#2122) enable golangci-lint for pulsar (testcontainers#2121) lint: enable testifylint (testcontainers#2120) chore(deps): bump github.com/nats-io/nats.go in /modules/nats (testcontainers#2094) chore(deps): bump golang.org/x/sys from 0.15.0 to 0.16.0 (testcontainers#2104) Revert "chore(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (testcontainers#2088)" chore(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (testcontainers#2088) chore(deps): bump cloud.google.com/go/spanner from 1.54.0 to 1.55.0, google.golang.org/api from 0.154.0 to 0.156.0 in /modules/gcloud (testcontainers#2115) chore(deps): bump github.com/aws/aws-sdk-go-v2/config from 1.25.10 to 1.26.3, github.com/aws/aws-sdk-go from 1.48.13 to 1.49.19 in /modules/localstack (testcontainers#2114) chore(deps): bump github.com/docker/go-connections from 0.4.0 to 0.5.0 (testcontainers#2113) Adding mockserver module (testcontainers#2085) ...
What does this PR do?
This PR adds support for the mockserver module.
Why is it important?
This is part of the current modules supported in testcontainers-java, having this in go is an good improvement.
Related issues
Relates #636
How to test this PR
I already added 2 test examples, but you can test manually based on that examples to run a mock server and add some expectations