Skip to content

Commit 0b018ce

Browse files
committed
remove noop tests [ci]
Signed-off-by: Paweł Perek <pawel.perek@digitalasset.com>
1 parent c209574 commit 0b018ce

File tree

6 files changed

+14
-48
lines changed

6 files changed

+14
-48
lines changed

apps/sv/frontend/src/__tests__/governance/forms/create-unallocated-unclaimed-activity-form.test.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -432,11 +432,6 @@ describe('Create Unallocated Unclaimed Activity Record Form', () => {
432432
await user.click(submitButton); //review proposal
433433
await user.click(submitButton); //submit proposal
434434

435-
waitFor(() => {
436-
expect(screen.getByText('Action Required')).toBeInTheDocument();
437-
expect(screen.getByText('Inflight Votes')).toBeInTheDocument();
438-
expect(screen.getByText('Vote History')).toBeInTheDocument();
439-
expect(screen.getByText('Successfully submitted the proposal')).toBeInTheDocument();
440-
});
435+
await screen.findByText('Successfully submitted the proposal');
441436
});
442437
});

apps/sv/frontend/src/__tests__/governance/forms/grant-revoke-featured-app-form.test.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -458,11 +458,6 @@ describe('Revoke Featured App Form', () => {
458458
await user.click(submitButton); //review proposal
459459
await user.click(submitButton); //submit proposal
460460

461-
waitFor(() => {
462-
expect(screen.getByText('Action Required')).toBeInTheDocument();
463-
expect(screen.getByText('Inflight Votes')).toBeInTheDocument();
464-
expect(screen.getByText('Vote History')).toBeInTheDocument();
465-
expect(screen.getByText('Successfully submitted the proposal')).toBeInTheDocument();
466-
});
461+
await screen.findByText('Successfully submitted the proposal');
467462
});
468463
});

apps/sv/frontend/src/__tests__/governance/forms/offboard-sv-form.test.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -327,11 +327,6 @@ describe('Offboard SV Form', () => {
327327
await user.click(submitButton); //review proposal
328328
await user.click(submitButton); //submit proposal
329329

330-
waitFor(() => {
331-
expect(screen.getByText('Action Required')).toBeInTheDocument();
332-
expect(screen.getByText('Inflight Votes')).toBeInTheDocument();
333-
expect(screen.getByText('Vote History')).toBeInTheDocument();
334-
expect(screen.getByText('Successfully submitted the proposal')).toBeInTheDocument();
335-
});
330+
await screen.findByText('Successfully submitted the proposal');
336331
});
337332
});

apps/sv/frontend/src/__tests__/governance/forms/set-amulet-rules-form.test.tsx

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -312,22 +312,15 @@ describe('Set Amulet Config Rules Form', () => {
312312
const c2Input = screen.getByTestId('config-field-transferConfigTransferFeeInitialRate');
313313
await user.type(c2Input, '9.99');
314314

315-
const reviewButton = screen.getByTestId('submit-button');
315+
const submitButton = screen.getByTestId('submit-button');
316316
await waitFor(async () => {
317-
expect(reviewButton.getAttribute('disabled')).toBeNull();
317+
expect(submitButton.getAttribute('disabled')).toBeNull();
318318
});
319-
await user.click(reviewButton);
320319

321-
await screen.findByText('Proposal Summary');
322-
const submitButton = screen.getByTestId('submit-button');
323-
await user.click(submitButton);
320+
await user.click(submitButton); //review proposal
321+
await user.click(submitButton); //submit proposal
324322

325-
// Verify the success toast appears
326-
// Note: Governance page content (Action Required, Inflight Votes, etc.) can't be tested
327-
// without full routing and the Wrapper doesn't include routes for the governance page
328-
await waitFor(() => {
329-
expect(screen.queryByText('Successfully submitted the proposal')).toBeInTheDocument();
330-
});
323+
await screen.findByText('Successfully submitted the proposal');
331324
});
332325

333326
test('should render diffs if changes to config values were made', async () => {

apps/sv/frontend/src/__tests__/governance/forms/set-dso-rules-form.test.tsx

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -308,22 +308,15 @@ describe('Set DSO Config Rules Form', () => {
308308

309309
await user.click(actionInput); // using this to trigger the onBlur event which triggers the validation
310310

311-
const reviewButton = screen.getByTestId('submit-button');
311+
const submitButton = screen.getByTestId('submit-button');
312312
await waitFor(async () => {
313-
expect(reviewButton.getAttribute('disabled')).toBeNull();
313+
expect(submitButton.getAttribute('disabled')).toBeNull();
314314
});
315-
await user.click(reviewButton);
316315

317-
await screen.findByText('Proposal Summary');
318-
const submitButton = screen.getByTestId('submit-button');
319-
await user.click(submitButton);
316+
await user.click(submitButton); //review proposal
317+
await user.click(submitButton); //submit proposal
320318

321-
// Verify the success toast appears
322-
// Note: Governance page content (Action Required, Inflight Votes, etc.) can't be tested
323-
// without full routing and the Wrapper doesn't include routes for the governance page
324-
await waitFor(() => {
325-
expect(screen.queryByText('Successfully submitted the proposal')).toBeInTheDocument();
326-
});
319+
await screen.findByText('Successfully submitted the proposal');
327320
});
328321

329322
test('should render diffs if changes to config values were made', async () => {

apps/sv/frontend/src/__tests__/governance/forms/update-sv-reward-weight-form-test.test.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -445,11 +445,6 @@ describe('Update SV Reward Weight Form', () => {
445445
await user.click(submitButton); //review proposal
446446
await user.click(submitButton); //submit proposal
447447

448-
waitFor(() => {
449-
expect(screen.getByText('Action Required')).toBeInTheDocument();
450-
expect(screen.getByText('Inflight Votes')).toBeInTheDocument();
451-
expect(screen.getByText('Vote History')).toBeInTheDocument();
452-
expect(screen.getByText('Successfully submitted the proposal')).toBeInTheDocument();
453-
});
448+
await screen.findByText('Successfully submitted the proposal');
454449
});
455450
});

0 commit comments

Comments
 (0)