domhanak commented on code in PR #3440:
URL: 
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3440#discussion_r1522956691


##########
kogito-test-utils/src/test/java/org/kie/kogito/testcontainers/KogitoMongoDBContainerTest.java:
##########
@@ -51,6 +53,7 @@ void shouldGetMapperPort() {
 
     @Test
     void shouldGetDockerImageName() {
+        assumeTrue(DockerClientFactory.instance().isDockerAvailable());

Review Comment:
   nitpick: `assumeTrue(String message, boolean b)`  with message could be used 
to indicate why we skip it. 
   Thinking that we might miss an issue here if we for some reason do not have 
docker on the machine executing these tests. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to