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

Add Testcontainers GCloud module when GCP messaging and testcontainers are selected #1264

Closed

Conversation

eddumelendez
Copy link
Contributor

Testcontainers offers a GCloud module where PubSubEmulatorContainer
can be found and be used for testing.

…s are selected

Testcontainers offers a GCloud module where `PubSubEmulatorContainer`
can be found and be used for testing.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 31, 2023
@snicoll
Copy link
Contributor

snicoll commented Jul 31, 2023

Thanks again @eddumelendez. Looking at the website the module states:

Testcontainers module for the Google Cloud Platform's Cloud SDK.

Is there a reason why the PR constrains it to only gcp pub sub? At the very least, GCP support should also include it as it is a segway to configure the project with GCP.

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Jul 31, 2023
@eddumelendez
Copy link
Contributor Author

Is there a reason why the PR constrains it to only gcp pub sub? At the very least, GCP support should also include it as it is a segway to configure the project with GCP.

Yeah, make sense now that I think about it. I tried to be specific because GCP Messaging is the one that explicitly can provide the Testcontainers class. However, if as user I know I'll use Datastore, Firestore or Spanner I can select GCP support and Testcontainers which will provide the dependency too and later I can add the proper starter.

I can update the PR later today.

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Jul 31, 2023
@eddumelendez
Copy link
Contributor Author

eddumelendez commented Jul 31, 2023

PR updated. Now, if cloud-gcp is selected, testcontainers gcloud module is also added.

BTW, there is already an issue in spring-cloud-gcp to support ServiceConnections. See GoogleCloudPlatform/spring-cloud-gcp#1780

snicoll pushed a commit that referenced this pull request Aug 1, 2023
Testcontainers offers a GCloud module where `PubSubEmulatorContainer`
can be found and be used for testing.

See gh-1264
snicoll added a commit that referenced this pull request Aug 1, 2023
Testcontainers offers a GCloud module where `PubSubEmulatorContainer`
can be found and be used for testing.

See gh-1264
@snicoll snicoll closed this in 1a5edf1 Aug 1, 2023
@snicoll snicoll added type: enhancement and removed status: waiting-for-triage An issue we've not yet triaged status: feedback-provided Feedback has been provided labels Aug 1, 2023
@snicoll
Copy link
Contributor

snicoll commented Aug 1, 2023

Thanks for the update @eddumelendez. I've polished the proposal to avoid adding the help link twice if both gcp and gcp-pub-sub are selected.

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