Fix adding servers without existing servers

Author: Josh

internal/generator/path.go

@@ -87,6 +87,15 @@ func (g *Generator) addPathsToDoc(doc *openapi3.T, services []*protogen.Service)
 		}
 
 		if len(serviceOptions.AddServers) > 0 {
+			if len(servers) == 0 {
+				// If there are no servers defined on the service level, but we
+				// have servers being added, we need to ensure both are on each
+				// method.
+				for _, s := range doc.Servers {
+					servers = append(servers, s)
+				}
+			}
+
 			for _, addServer := range serviceOptions.AddServers {
 				server, err := NewServer(addServer.Url)
 				if err != nil {

test/service_test.proto

@@ -55,6 +55,22 @@ service TestService {
   }
 }
 
+service Test2Service {
+  option (oapi.v1.service) = {
+    add_servers {
+      url: "api.added.io"
+    }
+    x_display_name: "Test 2 Service"
+    x_tag_group: "Test 2 Group"
+  };
+
+  rpc TestGet(TestGetRequest) returns (TestGetResponse) {
+    option (oapi.v1.method) = {
+      get: "Test2Get"
+    };
+  };
+}
+
 message TestGetRequest{
   string value = 1;
 }

test/service_test_openapi.yaml

@@ -48,6 +48,25 @@ paths:
       tags:
         - TestService
       security: [ ]
+  /v1/Test2Get:
+    get:
+      operationId: Test2Service_TestGet
+      responses:
+        '200':
+          content:
+            application/json:
+              schema:
+                properties:
+                  value:
+                    type: string
+          description: ''
+        default:
+          $ref: '#/components/responses/default'
+      servers:
+        - url: https://swagger.io
+        - url: https://api.added.io
+      tags:
+        - Test2Service
 
 components:
   responses:
@@ -76,11 +95,17 @@ servers:
 tags:
   - name: TestService
     x-displayName: Test Service
+  - name: Test2Service
+    x-displayName: Test 2 Service
+
 
 x-tagGroups:
   - name: Test Group
     tags:
       - TestService
+  - name: Test 2 Group
+    tags:
+      - Test2Service
 
 security:
   - bearer_auth: