Skip to content

Commit 5041086

Browse files
committed
Fix the test-class
1 parent 6b43bd6 commit 5041086

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

parquet-protobuf/src/test/java/org/apache/parquet/proto/TestUtils.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,14 +85,14 @@ public static <T extends MessageOrBuilder> List<T> testData(T... messages) throw
8585
List<Message> outputAsMessages = asMessages(output);
8686
Descriptors.Descriptor messageDescriptor =
8787
Protobufs.getMessageDescriptor(asMessage(messages[0]).getClass());
88-
Descriptors.FileDescriptor.Syntax syntax = messageDescriptor.getFile().getSyntax();
88+
String syntax = messageDescriptor.getFile().toProto().getSyntax();
8989
for (int i = 0; i < messages.length; i++) {
90-
if (Descriptors.FileDescriptor.Syntax.PROTO2.equals(syntax)) {
90+
if (!"proto3".equals(syntax)) {
9191
com.google.common.truth.extensions.proto.ProtoTruth.assertThat(outputAsMessages.get(i))
9292
.ignoringRepeatedFieldOrder()
9393
.reportingMismatchesOnly()
9494
.isEqualTo(asMessage(messages[i]));
95-
} else if (Descriptors.FileDescriptor.Syntax.PROTO3.equals(syntax)) {
95+
} else if ("proto3".equals(syntax)) {
9696
// proto3 will return default values for absent fields which is what is returned in output
9797
// this is why we can ignore absent fields here
9898
com.google.common.truth.extensions.proto.ProtoTruth.assertThat(outputAsMessages.get(i))

0 commit comments

Comments
 (0)