mirror of
https://github.com/lensapp/lens.git
synced 2025-05-20 05:10:56 +00:00
Add unit test for missing namespace when removing a kube object
We almost forgot to test this, because we started without TDD. Co-authored-by: Janne Savolainen <janne.savolainen@live.fi> Signed-off-by: Mikko Aspiala <mikko.aspiala@houston-inc.com>
This commit is contained in:
parent
0c53a16971
commit
bc3a8794cc
@ -108,6 +108,210 @@ exports[`kube-object-menu given kube object when removing kube object renders 1`
|
|||||||
</span>
|
</span>
|
||||||
</i>
|
</i>
|
||||||
|
|
||||||
|
<div>
|
||||||
|
<p>
|
||||||
|
Remove
|
||||||
|
some-kind
|
||||||
|
|
||||||
|
<b>
|
||||||
|
some-namespace/some-name
|
||||||
|
</b>
|
||||||
|
from
|
||||||
|
<b>
|
||||||
|
Some cluster name
|
||||||
|
</b>
|
||||||
|
?
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
<div
|
||||||
|
class="confirm-buttons"
|
||||||
|
>
|
||||||
|
<button
|
||||||
|
class="Button cancel plain"
|
||||||
|
type="button"
|
||||||
|
>
|
||||||
|
Cancel
|
||||||
|
</button>
|
||||||
|
<button
|
||||||
|
class="Button ok primary"
|
||||||
|
data-testid="confirm"
|
||||||
|
type="button"
|
||||||
|
>
|
||||||
|
Remove
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
<div
|
||||||
|
class="Tooltip narrow invisible formatter"
|
||||||
|
>
|
||||||
|
Delete
|
||||||
|
</div>
|
||||||
|
</body>
|
||||||
|
`;
|
||||||
|
|
||||||
|
exports[`kube-object-menu given kube object with namespace when removing kube object, renders confirmation dialog with namespace 1`] = `
|
||||||
|
<body>
|
||||||
|
<div>
|
||||||
|
<div>
|
||||||
|
<ul
|
||||||
|
class="Animate opacity Menu MenuActions flex KubeObjectMenu toolbar gaps right bottom"
|
||||||
|
>
|
||||||
|
<div>
|
||||||
|
Some menu item
|
||||||
|
</div>
|
||||||
|
<li
|
||||||
|
class="MenuItem"
|
||||||
|
data-menu-action-remove-test="true"
|
||||||
|
data-testid="menu-action-remove"
|
||||||
|
tabindex="0"
|
||||||
|
>
|
||||||
|
<i
|
||||||
|
class="Icon material interactive focusable"
|
||||||
|
id="tooltip_target_32"
|
||||||
|
tabindex="0"
|
||||||
|
>
|
||||||
|
<span
|
||||||
|
class="icon"
|
||||||
|
data-icon-name="delete"
|
||||||
|
>
|
||||||
|
delete
|
||||||
|
</span>
|
||||||
|
<div />
|
||||||
|
</i>
|
||||||
|
<span
|
||||||
|
class="title"
|
||||||
|
>
|
||||||
|
Remove
|
||||||
|
</span>
|
||||||
|
</li>
|
||||||
|
</ul>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
<div
|
||||||
|
class="Animate opacity-scale Dialog flex center ConfirmDialog modal"
|
||||||
|
data-testid="confirmation-dialog"
|
||||||
|
>
|
||||||
|
<div
|
||||||
|
class="box"
|
||||||
|
>
|
||||||
|
<div
|
||||||
|
class="confirm-content"
|
||||||
|
>
|
||||||
|
<i
|
||||||
|
class="Icon material focusable big"
|
||||||
|
>
|
||||||
|
<span
|
||||||
|
class="icon"
|
||||||
|
data-icon-name="warning"
|
||||||
|
>
|
||||||
|
warning
|
||||||
|
</span>
|
||||||
|
</i>
|
||||||
|
|
||||||
|
<div>
|
||||||
|
<p>
|
||||||
|
Remove
|
||||||
|
some-kind
|
||||||
|
|
||||||
|
<b>
|
||||||
|
some-namespace/some-name
|
||||||
|
</b>
|
||||||
|
from
|
||||||
|
<b>
|
||||||
|
Some cluster name
|
||||||
|
</b>
|
||||||
|
?
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
<div
|
||||||
|
class="confirm-buttons"
|
||||||
|
>
|
||||||
|
<button
|
||||||
|
class="Button cancel plain"
|
||||||
|
type="button"
|
||||||
|
>
|
||||||
|
Cancel
|
||||||
|
</button>
|
||||||
|
<button
|
||||||
|
class="Button ok primary"
|
||||||
|
data-testid="confirm"
|
||||||
|
type="button"
|
||||||
|
>
|
||||||
|
Remove
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
<div
|
||||||
|
class="Tooltip narrow invisible formatter"
|
||||||
|
>
|
||||||
|
Delete
|
||||||
|
</div>
|
||||||
|
</body>
|
||||||
|
`;
|
||||||
|
|
||||||
|
exports[`kube-object-menu given kube object without namespace when removing kube object, renders confirmation dialog without namespace 1`] = `
|
||||||
|
<body>
|
||||||
|
<div>
|
||||||
|
<div>
|
||||||
|
<ul
|
||||||
|
class="Animate opacity Menu MenuActions flex KubeObjectMenu toolbar gaps right bottom"
|
||||||
|
>
|
||||||
|
<div>
|
||||||
|
Some menu item
|
||||||
|
</div>
|
||||||
|
<li
|
||||||
|
class="MenuItem"
|
||||||
|
data-menu-action-remove-test="true"
|
||||||
|
data-testid="menu-action-remove"
|
||||||
|
tabindex="0"
|
||||||
|
>
|
||||||
|
<i
|
||||||
|
class="Icon material interactive focusable"
|
||||||
|
id="tooltip_target_40"
|
||||||
|
tabindex="0"
|
||||||
|
>
|
||||||
|
<span
|
||||||
|
class="icon"
|
||||||
|
data-icon-name="delete"
|
||||||
|
>
|
||||||
|
delete
|
||||||
|
</span>
|
||||||
|
<div />
|
||||||
|
</i>
|
||||||
|
<span
|
||||||
|
class="title"
|
||||||
|
>
|
||||||
|
Remove
|
||||||
|
</span>
|
||||||
|
</li>
|
||||||
|
</ul>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
<div
|
||||||
|
class="Animate opacity-scale Dialog flex center ConfirmDialog modal"
|
||||||
|
data-testid="confirmation-dialog"
|
||||||
|
>
|
||||||
|
<div
|
||||||
|
class="box"
|
||||||
|
>
|
||||||
|
<div
|
||||||
|
class="confirm-content"
|
||||||
|
>
|
||||||
|
<i
|
||||||
|
class="Icon material focusable big"
|
||||||
|
>
|
||||||
|
<span
|
||||||
|
class="icon"
|
||||||
|
data-icon-name="warning"
|
||||||
|
>
|
||||||
|
warning
|
||||||
|
</span>
|
||||||
|
</i>
|
||||||
|
|
||||||
<div>
|
<div>
|
||||||
<p>
|
<p>
|
||||||
Remove
|
Remove
|
||||||
|
|||||||
@ -117,6 +117,7 @@ describe("kube-object-menu", () => {
|
|||||||
uid: "some-uid",
|
uid: "some-uid",
|
||||||
name: "some-name",
|
name: "some-name",
|
||||||
resourceVersion: "some-resource-version",
|
resourceVersion: "some-resource-version",
|
||||||
|
namespace: "some-namespace",
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
|
|
||||||
@ -181,4 +182,82 @@ describe("kube-object-menu", () => {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe("given kube object with namespace", () => {
|
||||||
|
let baseElement: Element;
|
||||||
|
let getByTestId: (arg0: string) => any;
|
||||||
|
|
||||||
|
beforeEach(async () => {
|
||||||
|
objectStub = KubeObject.create({
|
||||||
|
apiVersion: "some-api-version",
|
||||||
|
kind: "some-kind",
|
||||||
|
metadata: {
|
||||||
|
uid: "some-uid",
|
||||||
|
name: "some-name",
|
||||||
|
resourceVersion: "some-resource-version",
|
||||||
|
namespace: "some-namespace",
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
({ baseElement, getByTestId } = render(
|
||||||
|
<div>
|
||||||
|
<ConfirmDialog/>
|
||||||
|
|
||||||
|
<KubeObjectMenu
|
||||||
|
object={objectStub}
|
||||||
|
dependencies={dependencies}
|
||||||
|
toolbar={true}
|
||||||
|
removeAction={() => {}}
|
||||||
|
/>
|
||||||
|
</div>,
|
||||||
|
));
|
||||||
|
});
|
||||||
|
|
||||||
|
it("when removing kube object, renders confirmation dialog with namespace", () => {
|
||||||
|
const menuItem = getByTestId("menu-action-remove");
|
||||||
|
|
||||||
|
userEvent.click(menuItem);
|
||||||
|
|
||||||
|
expect(baseElement).toMatchSnapshot();
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe("given kube object without namespace", () => {
|
||||||
|
let baseElement: Element;
|
||||||
|
let getByTestId: (arg0: string) => any;
|
||||||
|
|
||||||
|
beforeEach(async () => {
|
||||||
|
objectStub = KubeObject.create({
|
||||||
|
apiVersion: "some-api-version",
|
||||||
|
kind: "some-kind",
|
||||||
|
metadata: {
|
||||||
|
uid: "some-uid",
|
||||||
|
name: "some-name",
|
||||||
|
resourceVersion: "some-resource-version",
|
||||||
|
namespace: undefined,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
({ baseElement, getByTestId } = render(
|
||||||
|
<div>
|
||||||
|
<ConfirmDialog/>
|
||||||
|
|
||||||
|
<KubeObjectMenu
|
||||||
|
object={objectStub}
|
||||||
|
dependencies={dependencies}
|
||||||
|
toolbar={true}
|
||||||
|
removeAction={() => {}}
|
||||||
|
/>
|
||||||
|
</div>,
|
||||||
|
));
|
||||||
|
});
|
||||||
|
|
||||||
|
it("when removing kube object, renders confirmation dialog without namespace", () => {
|
||||||
|
const menuItem = getByTestId("menu-action-remove");
|
||||||
|
|
||||||
|
userEvent.click(menuItem);
|
||||||
|
|
||||||
|
expect(baseElement).toMatchSnapshot();
|
||||||
|
});
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user