mirror of
https://github.com/lensapp/lens.git
synced 2025-05-20 05:10:56 +00:00
chore: Adapt unit tests for more strict API parsing
Signed-off-by: Janne Savolainen <janne.savolainen@live.fi>
This commit is contained in:
parent
befbe62e63
commit
f2cb51884f
@ -58,7 +58,7 @@ describe("ApiManager", () => {
|
||||
|
||||
describe("registerApi", () => {
|
||||
it("re-register store if apiBase changed", () => {
|
||||
const apiBase = "apis/v1/foo";
|
||||
const apiBase = "api/v1/foo";
|
||||
const fallbackApiBase = "/apis/extensions/v1beta1/foo";
|
||||
const kubeApi = new TestApi({
|
||||
logger: di.inject(loggerInjectionToken),
|
||||
|
||||
@ -81,7 +81,7 @@ exports[`disable kube object detail items when cluster is not relevant given ext
|
||||
class="value"
|
||||
>
|
||||
<a
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapis%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapi%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
>
|
||||
some-namespace
|
||||
</a>
|
||||
@ -702,7 +702,7 @@ exports[`disable kube object detail items when cluster is not relevant given ext
|
||||
class="value"
|
||||
>
|
||||
<a
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapis%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapi%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
>
|
||||
some-namespace
|
||||
</a>
|
||||
@ -1318,7 +1318,7 @@ exports[`disable kube object detail items when cluster is not relevant given not
|
||||
class="value"
|
||||
>
|
||||
<a
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapis%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapi%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
>
|
||||
some-namespace
|
||||
</a>
|
||||
|
||||
@ -81,7 +81,7 @@ exports[`reactively hide kube object detail item renders 1`] = `
|
||||
class="value"
|
||||
>
|
||||
<a
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapis%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapi%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
>
|
||||
some-namespace
|
||||
</a>
|
||||
@ -697,7 +697,7 @@ exports[`reactively hide kube object detail item when the item is shown renders
|
||||
class="value"
|
||||
>
|
||||
<a
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapis%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
href="/workloads?kube-details=%2Fapi%2Fv1%2Fnamespaces%2Fsome-namespace&kube-selected=%2Fapi%2Fsome-api-version%2Fnamespaces%2Fsome-namespace%2Fsome-kind%2Fsome-name"
|
||||
>
|
||||
some-namespace
|
||||
</a>
|
||||
|
||||
@ -26,7 +26,7 @@ describe("disable kube object detail items when cluster is not relevant", () =>
|
||||
builder.afterWindowStart(({ windowDi }) => {
|
||||
const apiManager = windowDi.inject(apiManagerInjectable);
|
||||
const api = {
|
||||
apiBase: "/apis/some-api-version/some-kind",
|
||||
apiBase: "/api/some-api-version/some-kind",
|
||||
} as Partial<KubeApi<KubeObject>> as KubeApi<KubeObject>;
|
||||
const store = {
|
||||
api,
|
||||
@ -66,7 +66,7 @@ describe("disable kube object detail items when cluster is not relevant", () =>
|
||||
const windowDi = builder.applicationWindow.only.di;
|
||||
const showDetails = windowDi.inject(showDetailsInjectable);
|
||||
|
||||
showDetails("/apis/some-api-version/namespaces/some-namespace/some-kind/some-name");
|
||||
showDetails("/api/some-api-version/namespaces/some-namespace/some-kind/some-name");
|
||||
|
||||
builder.extensions.enable(testExtension);
|
||||
});
|
||||
|
||||
@ -29,7 +29,7 @@ describe("reactively hide kube object detail item", () => {
|
||||
builder.afterWindowStart(({ windowDi }) => {
|
||||
const apiManager = windowDi.inject(apiManagerInjectable);
|
||||
const api = {
|
||||
apiBase: "/apis/some-api-version/some-kind",
|
||||
apiBase: "/api/some-api-version/some-kind",
|
||||
} as Partial<KubeApi<KubeObject>> as KubeApi<KubeObject>;
|
||||
const store = {
|
||||
api,
|
||||
@ -73,7 +73,7 @@ describe("reactively hide kube object detail item", () => {
|
||||
const windowDi = builder.applicationWindow.only.di;
|
||||
const showDetails = windowDi.inject(showDetailsInjectable);
|
||||
|
||||
showDetails("/apis/some-api-version/namespaces/some-namespace/some-kind/some-name");
|
||||
showDetails("/api/some-api-version/namespaces/some-namespace/some-kind/some-name");
|
||||
|
||||
builder.extensions.enable(testExtension);
|
||||
});
|
||||
@ -96,7 +96,7 @@ describe("reactively hide kube object detail item", () => {
|
||||
|
||||
const apiManager = builder.applicationWindow.only.di.inject(apiManagerInjectable);
|
||||
|
||||
assert(apiManager.getStore("/apis/some-api-version/some-kind"));
|
||||
assert(apiManager.getStore("/api/some-api-version/some-kind"));
|
||||
});
|
||||
|
||||
it("renders", () => {
|
||||
|
||||
@ -2101,7 +2101,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeChanged: some-changed-value
|
||||
someAddedProperty: some-new-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -3633,7 +3633,7 @@ metadata:
|
||||
uid: some-other-uid
|
||||
name: some-other-name
|
||||
resourceVersion: some-resource-version
|
||||
selfLink: /apis/some-api-version/namespaces/some-other-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-other-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -4437,7 +4437,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -5964,7 +5964,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -6716,7 +6716,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -7468,7 +7468,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
labels:
|
||||
k8slens-edit-resource-version: some-api-version
|
||||
|
||||
@ -8222,7 +8222,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -9570,7 +9570,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -10322,7 +10322,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
@ -11074,7 +11074,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
|
||||
</textarea>
|
||||
</div>
|
||||
|
||||
@ -161,7 +161,7 @@ describe("cluster/namespaces - edit namespace from new tab", () => {
|
||||
|
||||
it("calls for namespace", () => {
|
||||
expect(apiKubeGetMock).toHaveBeenCalledWith(
|
||||
"/apis/some-api-version/namespaces/some-uid",
|
||||
"/api/some-api-version/namespaces/some-uid",
|
||||
);
|
||||
});
|
||||
|
||||
@ -208,7 +208,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
`);
|
||||
});
|
||||
|
||||
@ -227,7 +227,7 @@ metadata:
|
||||
|
||||
it("calls for save with just the adding version label", () => {
|
||||
expect(apiKubePatchMock).toHaveBeenCalledWith(
|
||||
"/apis/some-api-version/namespaces/some-uid",
|
||||
"/api/some-api-version/namespaces/some-uid",
|
||||
{
|
||||
data: [{
|
||||
op: "add",
|
||||
@ -509,7 +509,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeChanged: some-changed-value
|
||||
someAddedProperty: some-new-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
`,
|
||||
},
|
||||
});
|
||||
@ -532,7 +532,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeChanged: some-changed-value
|
||||
someAddedProperty: some-new-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
`);
|
||||
});
|
||||
|
||||
@ -548,7 +548,7 @@ metadata:
|
||||
expect(
|
||||
actual.edit_resource_store["some-first-tab-id"],
|
||||
).toEqual({
|
||||
resource: "/apis/some-api-version/namespaces/some-uid",
|
||||
resource: "/api/some-api-version/namespaces/some-uid",
|
||||
firstDraft: `apiVersion: some-api-version
|
||||
kind: Namespace
|
||||
metadata:
|
||||
@ -557,7 +557,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
`,
|
||||
draft: `apiVersion: some-api-version
|
||||
kind: Namespace
|
||||
@ -567,7 +567,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeChanged: some-changed-value
|
||||
someAddedProperty: some-new-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
`,
|
||||
});
|
||||
});
|
||||
@ -583,7 +583,7 @@ metadata:
|
||||
|
||||
it("calls for save with changed configuration", () => {
|
||||
expect(apiKubePatchMock).toHaveBeenCalledWith(
|
||||
"/apis/some-api-version/namespaces/some-uid",
|
||||
"/api/some-api-version/namespaces/some-uid",
|
||||
{
|
||||
data: [
|
||||
{
|
||||
@ -637,7 +637,7 @@ metadata:
|
||||
uid: some-uid
|
||||
name: some-name
|
||||
resourceVersion: some-resource-version
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
somePropertyToBeChanged: some-changed-value
|
||||
someAddedProperty: some-new-value
|
||||
someOtherAddedProperty: some-other-new-value
|
||||
@ -655,7 +655,7 @@ metadata:
|
||||
fireEvent.click(saveButton);
|
||||
|
||||
expect(apiKubePatchMock).toHaveBeenCalledWith(
|
||||
"/apis/some-api-version/namespaces/some-uid",
|
||||
"/api/some-api-version/namespaces/some-uid",
|
||||
{
|
||||
data: [
|
||||
{
|
||||
@ -732,7 +732,7 @@ metadata:
|
||||
uid: some-uid
|
||||
name: some-name
|
||||
resourceVersion: some-resource-version
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
`,
|
||||
},
|
||||
});
|
||||
@ -794,7 +794,7 @@ metadata:
|
||||
|
||||
it("calls for second namespace", () => {
|
||||
expect(apiKubeGetMock).toHaveBeenCalledWith(
|
||||
"/apis/some-api-version/namespaces/some-other-uid",
|
||||
"/api/some-api-version/namespaces/some-other-uid",
|
||||
);
|
||||
});
|
||||
|
||||
@ -811,7 +811,7 @@ metadata:
|
||||
name: "some-other-name",
|
||||
resourceVersion: "some-resource-version",
|
||||
selfLink:
|
||||
"/apis/some-api-version/namespaces/some-other-uid",
|
||||
"/api/some-api-version/namespaces/some-other-uid",
|
||||
},
|
||||
});
|
||||
|
||||
@ -833,7 +833,7 @@ metadata:
|
||||
uid: some-other-uid
|
||||
name: some-other-name
|
||||
resourceVersion: some-resource-version
|
||||
selfLink: /apis/some-api-version/namespaces/some-other-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-other-uid
|
||||
`);
|
||||
});
|
||||
|
||||
@ -847,7 +847,7 @@ metadata:
|
||||
fireEvent.click(saveButton);
|
||||
|
||||
expect(apiKubePatchMock).toHaveBeenCalledWith(
|
||||
"/apis/some-api-version/namespaces/some-other-uid",
|
||||
"/api/some-api-version/namespaces/some-other-uid",
|
||||
{
|
||||
data: [{
|
||||
op: "add",
|
||||
@ -897,7 +897,7 @@ metadata:
|
||||
});
|
||||
|
||||
it("does not call for namespace", () => {
|
||||
expect(apiKubeGetMock).not.toHaveBeenCalledWith("/apis/some-api-version/namespaces/some-uid");
|
||||
expect(apiKubeGetMock).not.toHaveBeenCalledWith("/api/some-api-version/namespaces/some-uid");
|
||||
});
|
||||
|
||||
it("has configuration in the editor", () => {
|
||||
@ -913,7 +913,7 @@ metadata:
|
||||
resourceVersion: some-resource-version
|
||||
somePropertyToBeRemoved: some-value
|
||||
somePropertyToBeChanged: some-old-value
|
||||
selfLink: /apis/some-api-version/namespaces/some-uid
|
||||
selfLink: /api/some-api-version/namespaces/some-uid
|
||||
`);
|
||||
});
|
||||
|
||||
@ -927,7 +927,7 @@ metadata:
|
||||
fireEvent.click(saveButton);
|
||||
|
||||
expect(apiKubePatchMock).toHaveBeenCalledWith(
|
||||
"/apis/some-api-version/namespaces/some-uid",
|
||||
"/api/some-api-version/namespaces/some-uid",
|
||||
{
|
||||
data: [{
|
||||
op: "add",
|
||||
@ -987,7 +987,7 @@ const someNamespaceDataStub = {
|
||||
uid: "some-uid",
|
||||
name: "some-name",
|
||||
resourceVersion: "some-resource-version",
|
||||
selfLink: "/apis/some-api-version/namespaces/some-uid",
|
||||
selfLink: "/api/some-api-version/namespaces/some-uid",
|
||||
},
|
||||
};
|
||||
|
||||
@ -998,6 +998,6 @@ const someOtherNamespaceDataStub = {
|
||||
uid: "some-other-uid",
|
||||
name: "some-other-name",
|
||||
resourceVersion: "some-resource-version",
|
||||
selfLink: "/apis/some-api-version/namespaces/some-other-uid",
|
||||
selfLink: "/api/some-api-version/namespaces/some-other-uid",
|
||||
},
|
||||
};
|
||||
|
||||
Loading…
Reference in New Issue
Block a user