decouple grpc and non validating full node flags (#1400) (#1414)

(cherry picked from commit 2fffe31)

Co-authored-by: jayy04 <[email protected]>

protocol/app/flags/flags.go

@@ -94,10 +94,6 @@ func (f *Flags) Validate() error {
 		if !f.GrpcEnable {
 			return fmt.Errorf("grpc.enable must be set to true - grpc streaming requires gRPC server")
 		}
-
-		if !f.NonValidatingFullNode {
-			return fmt.Errorf("grpc-streaming-enabled can only be set to true for non-validating full nodes")
-		}
 	}
 	return nil
 }

protocol/app/flags/flags_test.go

@@ -61,19 +61,18 @@ func TestValidate(t *testing.T) {
 				NonValidatingFullNode: true,
 			},
 		},
-		"failure - gRPC disabled": {
-			flags: flags.Flags{
-				GrpcEnable: false,
-			},
-			expectedErr: fmt.Errorf("grpc.enable must be set to true - validating requires gRPC server"),
-		},
-		"failure - gRPC streaming enabled for validating nodes": {
+		"success - gRPC streaming enabled for validating nodes": {
 			flags: flags.Flags{
 				NonValidatingFullNode: false,
 				GrpcEnable:            true,
 				GrpcStreamingEnabled:  true,
 			},
-			expectedErr: fmt.Errorf("grpc-streaming-enabled can only be set to true for non-validating full nodes"),
+		},
+		"failure - gRPC disabled": {
+			flags: flags.Flags{
+				GrpcEnable: false,
+			},
+			expectedErr: fmt.Errorf("grpc.enable must be set to true - validating requires gRPC server"),
 		},
 		"failure - gRPC streaming enabled with gRPC disabled": {
 			flags: flags.Flags{