fix error message for unsupported serviceLocator

Signed-off-by: Jamie Klassen <jklassen@vmware.com>
This commit is contained in:
Jamie Klassen
2023-12-15 13:12:58 -05:00
committed by Jamie Klassen
parent 2cfd6f55e3
commit 7233f57136
3 changed files with 28 additions and 1 deletions
+6
View File
@@ -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.
@@ -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"',
);
});
});
@@ -288,7 +288,7 @@ export class KubernetesBuilder {
break;
default:
throw new Error(
`Unsupported kubernetes.clusterLocatorMethod "${method}"`,
`Unsupported kubernetes.serviceLocatorMethod "${method}"`,
);
}