Skip to content

Commit ce682cc

Browse files
committed
fixup! Update existing tests to handle dialog elements remaining in the DOM
1 parent c3a783e commit ce682cc

File tree

1 file changed

+16
-8
lines changed

1 file changed

+16
-8
lines changed

packages/compass-collection/src/components/mock-data-generator-modal/mock-data-generator-modal.spec.tsx

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -134,23 +134,27 @@ describe('MockDataGeneratorModal', () => {
134134
it('renders the modal when isOpen is true', async () => {
135135
await renderModal();
136136

137-
expect(screen.getByTestId('generate-mock-data-modal')).to.exist;
137+
expect(screen.getByTestId('generate-mock-data-modal').firstChild).to
138+
.exist;
138139
});
139140

140141
it('does not render the modal when isOpen is false', async () => {
141142
await renderModal({ isOpen: false });
142143

143-
expect(screen.queryByTestId('generate-mock-data-modal')).to.not.exist;
144+
expect(screen.queryByTestId('generate-mock-data-modal')?.firstChild).to
145+
.not.exist;
144146
});
145147

146148
it('closes the modal when the close button is clicked', async () => {
147149
await renderModal();
148150

149-
expect(screen.getByTestId('generate-mock-data-modal')).to.exist;
151+
expect(screen.getByTestId('generate-mock-data-modal').firstChild).to
152+
.exist;
150153
userEvent.click(screen.getByLabelText('Close modal'));
151154
await waitFor(
152155
() =>
153-
expect(screen.queryByTestId('generate-mock-data-modal')).to.not.exist
156+
expect(screen.getByTestId('generate-mock-data-modal').firstChild).to
157+
.not.exist
154158
);
155159
});
156160

@@ -172,11 +176,13 @@ describe('MockDataGeneratorModal', () => {
172176
it('closes the modal when the cancel button is clicked', async () => {
173177
await renderModal();
174178

175-
expect(screen.getByTestId('generate-mock-data-modal')).to.exist;
179+
expect(screen.getByTestId('generate-mock-data-modal').firstChild).to
180+
.exist;
176181
userEvent.click(screen.getByText('Cancel'));
177182
await waitFor(
178183
() =>
179-
expect(screen.queryByTestId('generate-mock-data-modal')).to.not.exist
184+
expect(screen.getByTestId('generate-mock-data-modal').firstChild).to
185+
.not.exist
180186
);
181187
});
182188

@@ -1062,11 +1068,13 @@ describe('MockDataGeneratorModal', () => {
10621068
},
10631069
});
10641070

1065-
expect(screen.getByTestId('generate-mock-data-modal')).to.exist;
1071+
expect(screen.getByTestId('generate-mock-data-modal').firstChild).to
1072+
.exist;
10661073
userEvent.click(screen.getByText('Done'));
10671074
await waitFor(
10681075
() =>
1069-
expect(screen.queryByTestId('generate-mock-data-modal')).to.not.exist
1076+
expect(screen.getByTestId('generate-mock-data-modal').firstChild).to
1077+
.not.exist
10701078
);
10711079
});
10721080

0 commit comments

Comments
 (0)