lukecwik commented on a change in pull request #15685:
URL: https://github.com/apache/beam/pull/15685#discussion_r727570054
##########
File path: sdks/java/core/src/main/java/org/apache/beam/sdk/testing/PAssert.java
##########
@@ -299,6 +299,13 @@ public int hashCode() {
*/
IterableAssert<T> containsInAnyOrder(T... expectedElements);
+ /**
+ * Asserts that the iterable in question matches the provided elements.
+ *
+ * @return the same {@link IterableAssert} builder for further assertions
+ */
+ IterableAssert<T> containsInAnyOrder(SerializableMatcher<? super T>...
expectedElements);
+
Review comment:
There are still additional tests failing:
```
14:59:05 > Task :sdks:java:extensions:sql:zetasql:compileTestJava
14:59:05
/home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_Commit@3/src/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSqlDialectSpecTest.java:1226:
error: reference to containsInAnyOrder is ambiguous
14:59:05 PAssert.that(stream).containsInAnyOrder();
14:59:05 ^
14:59:05 both method containsInAnyOrder(T...) in IterableAssert and method
containsInAnyOrder(SerializableMatcher<? super T>...) in IterableAssert match
14:59:05 where T is a type-variable:
14:59:05 T extends Object declared in interface IterableAssert
14:59:05
/home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_Commit@3/src/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSqlDialectSpecTest.java:1554:
error: reference to containsInAnyOrder is ambiguous
14:59:05 PAssert.that(stream).containsInAnyOrder();
14:59:05 ^
14:59:05 both method containsInAnyOrder(T...) in IterableAssert and method
containsInAnyOrder(SerializableMatcher<? super T>...) in IterableAssert match
14:59:05 where T is a type-variable:
14:59:05 T extends Object declared in interface IterableAssert
14:59:05
/home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_Commit@3/src/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSqlDialectSpecTest.java:1857:
error: reference to containsInAnyOrder is ambiguous
14:59:05 PAssert.that(stream).containsInAnyOrder();
14:59:05 ^
14:59:05 both method containsInAnyOrder(T...) in IterableAssert and method
containsInAnyOrder(SerializableMatcher<? super T>...) in IterableAssert match
14:59:05 where T is a type-variable:
14:59:05 T extends Object declared in interface IterableAssert
14:59:05
/home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_Commit@3/src/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSqlDialectSpecTest.java:2578:
error: reference to containsInAnyOrder is ambiguous
14:59:05 PAssert.that(stream).containsInAnyOrder();
14:59:05 ^
14:59:05 both method containsInAnyOrder(T...) in IterableAssert and method
containsInAnyOrder(SerializableMatcher<? super T>...) in IterableAssert match
14:59:05 where T is a type-variable:
14:59:05 T extends Object declared in interface IterableAssert
14:59:05
/home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_Commit@3/src/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSqlDialectSpecTest.java:2846:
error: reference to containsInAnyOrder is ambiguous
14:59:05 PAssert.that(stream).containsInAnyOrder();
14:59:05 ^
14:59:05 both method containsInAnyOrder(T...) in IterableAssert and method
containsInAnyOrder(SerializableMatcher<? super T>...) in IterableAssert match
14:59:05 where T is a type-variable:
14:59:05 T extends Object declared in interface IterableAssert
14:59:05 Note: Some input files use or override a deprecated API.
14:59:05 Note: Recompile with -Xlint:deprecation for details.
14:59:05 Note:
/home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_Commit@3/src/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/BeamZetaSqlCalcRelTest.java
uses unchecked or unsafe operations.
14:59:05 Note: Recompile with -Xlint:unchecked for details.
14:59:05 5 errors
```
You can verify locally by running `./gradlew compileTestJava`
--
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]
