Merge pull request #3 from jtjeferreira/no-package-issue

Handle protos without a package
Fadelis · Feb 9, 2022 · 02162e8 · 02162e8
2 parents d5b9b50 + 80eeb68
commit 02162e8
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/protoc-gen-java-optional-test/src/test/resources/no-package-type.proto b/protoc-gen-java-optional-test/src/test/resources/no-package-type.proto
@@ -0,0 +1,12 @@
+syntax = "proto3";
+
+option java_multiple_files = true;
+option java_package = "org.test.datatypes";
+
+message NoPackageTestMessage {
+  message NestedTestMessage {
+    string string = 1;
+    optional string optional_string = 2;
+  }
+}
+
diff --git a/protoc-gen-java-optional/src/main/java/org/grpcmock/protoc/plugin/OptionalGenerator.java b/protoc-gen-java-optional/src/main/java/org/grpcmock/protoc/plugin/OptionalGenerator.java
@@ -104,7 +104,7 @@ private Stream<File> handleMessage(
       String javaPackage
   ) {
     String filePath = javaPackage.replace(".", DIR_SEPARATOR) + DIR_SEPARATOR + fileName + JAVA_EXTENSION;
-    String fullMethodName = protoPackage + "." + messageDescriptor.getName();
+    String fullMethodName = (protoPackage.isEmpty() ? "" : (protoPackage + ".")) + messageDescriptor.getName();
 
     return Stream.concat(
         handleSingleMessage(messageDescriptor, filePath, fullMethodName),