Skip to content

Commit

Permalink
fix: added test case
Browse files Browse the repository at this point in the history
  • Loading branch information
riddhybansal committed May 7, 2024
1 parent 1306f1d commit 455529f
Showing 1 changed file with 30 additions and 1 deletion.
31 changes: 30 additions & 1 deletion packages/react/src/components/DatePicker/DatePicker-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
import React, { useState } from 'react';
import DatePicker from './DatePicker';
import DatePickerInput from '../DatePickerInput';
import { render, screen } from '@testing-library/react';
import { render, screen, fireEvent } from '@testing-library/react';
import userEvent from '@testing-library/user-event';
import { Slug } from '../Slug';

Expand Down Expand Up @@ -100,6 +100,35 @@ describe('DatePicker', () => {
).toBeInTheDocument();
});

it('should not fire onChange handler when clicking outside the datepicker in range mode', () => {
const handleChange = jest.fn();
const { getByLabelText, getByText } = render(
<DatePicker
onChange={handleChange}
dateFormat="m/d/Y"
datePickerType="range">
<DatePickerInput
id="date-picker-input-id-start"
placeholder="mm/dd/yyyy"
labelText="Start date"
/>
<DatePickerInput
id="date-picker-input-id-finish"
placeholder="mm/dd/yyyy"
labelText="End date"
/>
</DatePicker>
);
const startDateInput = getByLabelText('Start date');
const endDateInput = getByLabelText('End date');
// Change the dates
fireEvent.change(startDateInput, { target: { value: '01/01/2023' } });
fireEvent.change(endDateInput, { target: { value: '01/07/2023' } });
// Simulate a click event outside the datepicker
fireEvent.click(document.body);
expect(handleChange).not.toHaveBeenCalled();
});

it('should render the children as expected', () => {
render(
<DatePicker onChange={() => {}} dateFormat="m/d/Y" datePickerType="range">
Expand Down

0 comments on commit 455529f

Please sign in to comment.