grpc-loader: update interfaces & add new test

grpc · Apr 5, 2024 · 4c185fa · 4c185fa
1 parent d335428
commit 4c185fa
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 12 deletions.
diff --git a/packages/proto-loader/src/index.ts b/packages/proto-loader/src/index.ts
@@ -116,9 +116,9 @@ export interface EnumTypeDefinition extends ProtobufTypeDefinition {
 }
 
 export enum IdempotencyLevel {
-  IDEMPOTENCY_UNKNOWN = 0,
-  NO_SIDE_EFFECTS = 1,
-  IDEMPOTENT = 2
+  IDEMPOTENCY_UNKNOWN = 'IDEMPOTENCY_UNKNOWN',
+  NO_SIDE_EFFECTS = 'NO_SIDE_EFFECTS',
+  IDEMPOTENT = 'IDEMPOTENT'
 }
 
 export interface NamePart {
@@ -136,16 +136,11 @@ export interface UninterpretedOption {
   aggregate_value?: string;
 }
 
-export interface CustomHttpPattern {
-  kind?: string;
-  path?: string;
-}
-
 export interface MethodOptions {
-  deprecated?: (boolean|null);
-  idempotency_level?: (IdempotencyLevel|keyof typeof IdempotencyLevel|null);
-  uninterpreted_option?: (UninterpretedOption[]|null);
-  [k: string]: unknown
+  deprecated?: boolean;
+  idempotency_level?: IdempotencyLevel|keyof typeof IdempotencyLevel;
+  uninterpreted_option?: UninterpretedOption;
+  [k: string]: unknown;
 }
 
 export interface MethodDefinition<RequestType, ResponseType, OutputRequestType=RequestType, OutputResponseType=ResponseType> {

diff --git a/packages/proto-loader/test/descriptor_type_test.ts b/packages/proto-loader/test/descriptor_type_test.ts
@@ -160,5 +160,6 @@ describe('Descriptor types', () => {
       },
       '(google.api.method_signature)': 'bar'
     })
+    assert(service.HelloWithoutOptions.options === undefined)
   })
 });
diff --git a/packages/proto-loader/test_protos/method_options.proto b/packages/proto-loader/test_protos/method_options.proto
@@ -34,4 +34,5 @@ service Hello {
     };
     option (google.api.method_signature) = 'bar';
   }
+  rpc HelloWithoutOptions (Empty) returns (Empty) {}
 }