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

genie client integration tests: throw the exception of setting up the docker container #1211

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

bhou2
Copy link
Contributor

@bhou2 bhou2 commented Apr 8, 2024

genie client integration tests: throw the exception if the docker container is not successfully set up.

The genie client integration tests have passed:

> Task :genie-client:integrationTest

GenieClientIntegrationTest > testUpdateApplication() PASSED

GenieClientIntegrationTest > testGetAllAndDeleteAllApplications() PASSED
...

GenieClientIntegrationTest > testClusterPatchMethod() PASSED

GenieClientIntegrationTest > canSubmitJob() PASSED

@bhou2 bhou2 requested review from stevie9868 and zdong2 April 8, 2024 21:19
Copy link
Collaborator

@zdong2 zdong2 left a comment

Choose a reason for hiding this comment

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

LGTM

@bhou2 bhou2 merged commit 4aaecab into master Apr 8, 2024
2 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.

None yet

2 participants