denshooter/portfolio
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

✨ fix: remove unnecessary test assertions and improve act usage

Browse Source
This commit is contained in:
Denshooter
2025-02-22 23:58:34 +01:00
parent 725bbe5d50
commit 5affec766f
5 changed files with 47 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
app/__tests__/components/Contact.test.tsx
View File

@@ -1,4 +1,4 @@
import { render, screen, fireEvent, waitFor } from '@testing-library/react';
import { render, screen, fireEvent, waitFor, act } from '@testing-library/react';
import Contact from '@/app/components/Contact';
import '@testing-library/jest-dom';
@@ -29,15 +29,29 @@ describe('Contact', () => {
it('submits the form', async () => {
render(<Contact />);
// Fast forward time to ensure the timestamp check passes
jest.advanceTimersByTime(3000);
// Wrap timer advancement in act
await act(async () => {
jest.advanceTimersByTime(3000);
});
fireEvent.change(screen.getByPlaceholderText('Your Name'), { target: { value: 'John Doe' } });
fireEvent.change(screen.getByPlaceholderText('you@example.com'), { target: { value: 'john@example.com' } });
fireEvent.change(screen.getByPlaceholderText('Your Message...'), { target: { value: 'Hello!' } });
fireEvent.click(screen.getByLabelText('I accept the privacy policy.'));
fireEvent.click(screen.getByText('Send Message'));
// Fire events inside act if needed
act(() => {
fireEvent.change(screen.getByPlaceholderText('Your Name'), {
target: { value: 'John Doe' },
});
fireEvent.change(screen.getByPlaceholderText('you@example.com'), {
target: { value: 'john@example.com' },
});
fireEvent.change(screen.getByPlaceholderText('Your Message...'), {
target: { value: 'Hello!' },
});
fireEvent.click(screen.getByLabelText('I accept the privacy policy.'));
fireEvent.click(screen.getByText('Send Message'));
});
await waitFor(() => expect(screen.getByText('Email sent')).toBeInTheDocument());
// Wait for the result
await waitFor(() =>
expect(screen.getByText('Email sent')).toBeInTheDocument()
);
});
});