Abacn commented on code in PR #37459:
URL: https://github.com/apache/beam/pull/37459#discussion_r2771896748
##########
sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/spanner/changestreams/dofn/ReadChangeStreamPartitionDoFnTest.java:
##########
@@ -149,7 +150,8 @@ public void setUp() {
partitionStartRecordAction,
partitionEndRecordAction,
partitionEventRecordAction,
- metrics))
+ metrics,
+ anyBoolean()))
Review Comment:
testQueryChangeStreamMode
(org.apache.beam.sdk.io.gcp.spanner.changestreams.dofn.ReadChangeStreamPartitionDoFnTest)
failed
sdks/java/io/google-cloud-platform/build/test-results/test/TEST-org.apache.beam.sdk.io.gcp.spanner.changestreams.dofn.ReadChangeStreamPartitionDoFnTest.xml
[took 0s]
```
org.mockito.exceptions.misusing.InvalidUseOfMatchersException:
Invalid use of argument matchers!
12 matchers expected, 1 recorded:
-> at
org.apache.beam.sdk.io.gcp.spanner.changestreams.dofn.ReadChangeStreamPartitionDoFnTest.setUp(ReadChangeStreamPartitionDoFnTest.java:154)
This exception may occur if matchers are combined with raw values:
//incorrect:
someMethod(any(), "raw String");
When using matchers, all arguments have to be provided by matchers.
For example:
//correct:
someMethod(any(), eq("String by matcher"));
For more info see javadoc for Matchers class.
at
app//org.apache.beam.sdk.io.gcp.spanner.changestreams.action.ActionFactory.queryChangeStreamAction(ActionFactory.java:192)
at
app//org.apache.beam.sdk.io.gcp.spanner.changestreams.dofn.ReadChangeStreamPartitionDoFnTest.setUp(ReadChangeStreamPartitionDoFnTest.java:142)
at
[email protected]/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
Method)
```
looks relevant test failure
--
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]