Merge pull request #21892 from jamieklassen/unsupported-servicelocator-message

fix error message for unsupported serviceLocator

.changeset/friendly-cheetahs-rescue.md

@@ -0,0 +1,6 @@
+---
+'@backstage/plugin-kubernetes-backend': patch
+---
+
+Fixed an issue where a misleading error message would be logged when an
+unsupported service locator method was specified.

plugins/kubernetes-backend/src/service/KubernetesBuilder.test.ts

@@ -206,6 +206,7 @@ describe('KubernetesBuilder', () => {
       });
     });
   });
+
   describe('post /services/:serviceId', () => {
     it('happy path: lists kubernetes objects without auth in request body', async () => {
       const response = await request(app).post(
@@ -780,4 +781,24 @@ metadata:
       });
     });
   });
+
+  it('fails when an unsupported serviceLocator type is specified', () => {
+    return expect(() =>
+      startTestBackend({
+        features: [
+          mockServices.rootConfig.factory({
+            data: {
+              kubernetes: {
+                serviceLocatorMethod: { type: 'unsupported' },
+                clusterLocatorMethods: [{ type: 'config', clusters: [] }],
+              },
+            },
+          }),
+          import('@backstage/plugin-kubernetes-backend/alpha'),
+        ],
+      }),
+    ).rejects.toThrow(
+      'Unsupported kubernetes.serviceLocatorMethod "unsupported"',
+    );
+  });
 });

plugins/kubernetes-backend/src/service/KubernetesBuilder.ts

@@ -288,7 +288,7 @@ export class KubernetesBuilder {
         break;
       default:
         throw new Error(
-          `Unsupported kubernetes.clusterLocatorMethod "${method}"`,
+          `Unsupported kubernetes.serviceLocatorMethod "${method}"`,
         );
     }