Skip to content

Commit 7a347a9

Browse files
committed
Added little tests for date time picker string value
1 parent 43a0857 commit 7a347a9

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

packages/pf3-component-mapper/src/form-fields/date-time-picker/date-time-picker.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export class DateTimePicker extends React.Component {
3131
super(props);
3232
this.state = {
3333
positionLeft: 0,
34-
selectedDay: typeof props.value === 'string' ? new Date(props.value) : props.value,
34+
selectedDay: props.value ? typeof props.value === 'string' ? new Date(props.value) : props.value : undefined,
3535
selectingMonth: false,
3636
selectingYear: false,
3737
isOpen: false,
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
import React from 'react';
2+
import { mount } from 'enzyme';
3+
4+
import { DateTimePicker } from '../../../form-fields/date-time-picker/date-time-picker';
5+
6+
describe('<DateTimePicker />', () => {
7+
it('should use value of type Date', () => {
8+
const wrapper = mount(<DateTimePicker value={ new Date() } />);
9+
expect(wrapper.state().selectedDay).toBeInstanceOf(Date);
10+
});
11+
12+
it('should convert string value into Date object', () => {
13+
const wrapper = mount(<DateTimePicker value='2019-11-01T12:31:00.000Z' />);
14+
expect(wrapper.state().selectedDay).toBeInstanceOf(Date);
15+
});
16+
17+
it('should not set state for undefined value', () => {
18+
const wrapper = mount(<DateTimePicker />);
19+
expect(wrapper.state().selectedDay).toBeUndefined();
20+
});
21+
});
22+

0 commit comments

Comments
 (0)