Skip to content

Commit 291c805

Browse files
committed
Tighten supervisor intro structure test coverage
1 parent 249280b commit 291c805

2 files changed

Lines changed: 39 additions & 3 deletions

File tree

packages/web/src/features/supervisor/components/objective-dialog.test.tsx

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,21 @@ describe("ObjectiveDialog", () => {
198198
</Provider>
199199
);
200200

201-
expect(document.querySelector(".supervisor-dialog-intro")).toBeTruthy();
201+
const dialog = screen.getByRole("dialog");
202+
const intro = dialog.querySelector(".supervisor-dialog-intro");
203+
const introIcon = dialog.querySelector(".supervisor-dialog-intro__icon");
204+
const introCopy = dialog.querySelector(".supervisor-dialog-intro__copy");
205+
const firstFormGroup = dialog.querySelector(".form-group");
206+
207+
expect(intro).toBeTruthy();
208+
expect(introIcon).toBeTruthy();
209+
expect(introCopy).toBeTruthy();
210+
expect(intro?.querySelector(".supervisor-dialog-intro__icon")).toBe(introIcon);
211+
expect(intro?.querySelector(".supervisor-dialog-intro__copy")).toBe(introCopy);
212+
expect(firstFormGroup).toBeTruthy();
213+
expect(intro?.compareDocumentPosition(firstFormGroup as Node)).toBe(
214+
Node.DOCUMENT_POSITION_FOLLOWING
215+
);
202216
});
203217

204218
it("renders disable confirmation mode", () => {

packages/web/src/features/supervisor/views/shared/objective-dialog-content.test.tsx

Lines changed: 24 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,19 @@ describe("ObjectiveDialogContent", () => {
4646
);
4747

4848
let intro = document.querySelector(".supervisor-dialog-intro");
49+
let introIcon = document.querySelector(".supervisor-dialog-intro__icon");
50+
let introCopy = document.querySelector(".supervisor-dialog-intro__copy");
51+
let firstFormGroup = document.querySelector(".form-group");
4952
expect(intro).toBeTruthy();
50-
expect(intro?.querySelector('[data-icon-semantic="supervisor.mode.enable"]')).toBeTruthy();
53+
expect(introIcon).toBeTruthy();
54+
expect(introCopy).toBeTruthy();
55+
expect(intro?.querySelector(".supervisor-dialog-intro__icon")).toBe(introIcon);
56+
expect(intro?.querySelector(".supervisor-dialog-intro__copy")).toBe(introCopy);
57+
expect(introIcon?.querySelector('[data-icon-semantic="supervisor.mode.enable"]')).toBeTruthy();
58+
expect(firstFormGroup).toBeTruthy();
59+
expect(intro?.compareDocumentPosition(firstFormGroup as Node)).toBe(
60+
Node.DOCUMENT_POSITION_FOLLOWING
61+
);
5162
expect(intro?.querySelector(".supervisor-dialog-intro__title")).toHaveTextContent(
5263
"supervisor.dialog.enable.title"
5364
);
@@ -73,8 +84,19 @@ describe("ObjectiveDialogContent", () => {
7384
);
7485

7586
intro = document.querySelector(".supervisor-dialog-intro");
87+
introIcon = document.querySelector(".supervisor-dialog-intro__icon");
88+
introCopy = document.querySelector(".supervisor-dialog-intro__copy");
89+
firstFormGroup = document.querySelector(".form-group");
7690
expect(intro).toBeTruthy();
77-
expect(intro?.querySelector('[data-icon-semantic="supervisor.mode.edit"]')).toBeTruthy();
91+
expect(introIcon).toBeTruthy();
92+
expect(introCopy).toBeTruthy();
93+
expect(intro?.querySelector(".supervisor-dialog-intro__icon")).toBe(introIcon);
94+
expect(intro?.querySelector(".supervisor-dialog-intro__copy")).toBe(introCopy);
95+
expect(introIcon?.querySelector('[data-icon-semantic="supervisor.mode.edit"]')).toBeTruthy();
96+
expect(firstFormGroup).toBeTruthy();
97+
expect(intro?.compareDocumentPosition(firstFormGroup as Node)).toBe(
98+
Node.DOCUMENT_POSITION_FOLLOWING
99+
);
78100
expect(intro?.querySelector(".supervisor-dialog-intro__title")).toHaveTextContent(
79101
"supervisor.dialog.edit.title"
80102
);

0 commit comments

Comments
 (0)