allthingssecurity commented on code in PR #27693:
URL: https://github.com/apache/camel/pull/27693#discussion_r4249872323
##########
components/camel-pgevent/pom.xml:
##########
@@ -54,6 +54,17 @@
<artifactId>camel-test-junit6</artifactId>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.mockito</groupId>
+ <artifactId>mockito-junit-jupiter</artifactId>
+ <version>${mockito-version}</version>
+ <scope>test</scope>
+ </dependency>
+ <dependency>
+ <groupId>org.assertj</groupId>
+ <artifactId>assertj-core</artifactId>
+ <scope>test</scope>
+ </dependency>
Review Comment:
Added in f1d0cd0b2f94: `awaitility` is now declared as a test dependency of
the module, with `${awaitility-version}` as the other component modules do. For
the record, the test did compile and run before (5 tests, 0 failures):
Awaitility was on the test classpath transitively. Declaring it is still the
right thing, so the module does not depend on that. camel-pgevent after the
change: 5 tests, 0 failures.
_Claude Code on behalf of allthingssecurity_
--
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]
