Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 9792aab

Browse files
committed
fix: clarify LRO parsing error messages
1 parent bb9e50c commit 9792aab

File tree

1 file changed

+8
-2
lines changed
  • src/main/java/com/google/api/generator/gapic/protoparser

1 file changed

+8
-2
lines changed

src/main/java/com/google/api/generator/gapic/protoparser/Parser.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -367,10 +367,16 @@ static LongrunningOperation parseLro(
367367
Message responseMessage = messageTypes.get(responseTypeName);
368368
Message metadataMessage = messageTypes.get(metadataTypeName);
369369
Preconditions.checkNotNull(
370-
responseMessage, String.format("LRO response message %s not found", responseTypeName));
370+
responseMessage,
371+
String.format(
372+
"LRO response message %s not found on method %s",
373+
responseTypeName, methodDescriptor.getName()));
371374
// TODO(miraleung): Check that the packages are equal if those strings are not empty.
372375
Preconditions.checkNotNull(
373-
metadataMessage, String.format("LRO metadata message %s not found", metadataTypeName));
376+
metadataMessage,
377+
String.format(
378+
"LRO metadata message %s not found in method %s",
379+
metadataTypeName, methodDescriptor.getName()));
374380

375381
return LongrunningOperation.withTypes(responseMessage.type(), metadataMessage.type());
376382
}

0 commit comments

Comments
 (0)