Use @wojtekmaj/date-utils package (#61)

Author: wojtekmaj

package.json

@@ -40,6 +40,7 @@
   },
   "license": "MIT",
   "dependencies": {
+    "@wojtekmaj/date-utils": "^1.0.0",
     "get-user-locale": "^1.2.0",
     "make-event-props": "^1.1.0",
     "merge-class-names": "^1.1.1",

src/TimeInput.jsx

@@ -1,6 +1,13 @@
 import React, { PureComponent } from 'react';
 import PropTypes from 'prop-types';
 import { polyfill } from 'react-lifecycles-compat';
+import {
+  getHours,
+  getMinutes,
+  getSeconds,
+  getHoursMinutes,
+  getHoursMinutesSeconds,
+} from '@wojtekmaj/date-utils';
 
 import Divider from './Divider';
 import Hour12Input from './TimeInput/Hour12Input';
@@ -11,15 +18,7 @@ import NativeInput from './TimeInput/NativeInput';
 import AmPm from './TimeInput/AmPm';
 
 import { getFormatter } from './shared/dateFormatter';
-import {
-  getHours,
-  getMinutes,
-  getSeconds,
-  getHoursMinutes,
-  getHoursMinutesSeconds,
-  convert12to24,
-  convert24to12,
-} from './shared/dates';
+import { convert12to24, convert24to12 } from './shared/dates';
 import { isTime } from './shared/propTypes';
 import { getAmPmLabels } from './shared/utils';
 

src/TimeInput/AmPm.jsx

@@ -1,11 +1,9 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import mergeClassNames from 'merge-class-names';
+import { getHours } from '@wojtekmaj/date-utils';
 
-import {
-  getHours,
-  convert24to12,
-} from '../shared/dates';
+import { convert24to12 } from '../shared/dates';
 import { isTime } from '../shared/propTypes';
 import { getAmPmLabels } from '../shared/utils';
 

src/TimeInput/Hour12Input.jsx

@@ -1,10 +1,10 @@
 import React from 'react';
 import PropTypes from 'prop-types';
+import { getHours } from '@wojtekmaj/date-utils';
 
 import Input from './Input';
 
 import {
-  getHours,
   convert24to12,
 } from '../shared/dates';
 import { isTime } from '../shared/propTypes';

src/TimeInput/Hour24Input.jsx

@@ -1,9 +1,9 @@
 import React from 'react';
 import PropTypes from 'prop-types';
+import { getHours } from '@wojtekmaj/date-utils';
 
 import Input from './Input';
 
-import { getHours } from '../shared/dates';
 import { isTime } from '../shared/propTypes';
 import { min, max } from '../shared/utils';
 

src/TimeInput/MinuteInput.jsx

@@ -1,9 +1,9 @@
 import React from 'react';
 import PropTypes from 'prop-types';
+import { getHours, getMinutes } from '@wojtekmaj/date-utils';
 
 import Input from './Input';
 
-import { getHours, getMinutes } from '../shared/dates';
 import { isTime } from '../shared/propTypes';
 import { min, max } from '../shared/utils';
 

src/TimeInput/NativeInput.jsx

@@ -1,11 +1,11 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-
 import {
   getHours,
   getHoursMinutes,
   getHoursMinutesSeconds,
-} from '../shared/dates';
+} from '@wojtekmaj/date-utils';
+
 import { isTime, isValueType } from '../shared/propTypes';
 
 export default function NativeInput({

src/TimeInput/SecondInput.jsx

@@ -1,13 +1,9 @@
 import React from 'react';
 import PropTypes from 'prop-types';
+import { getHours, getMinutes, getSeconds } from '@wojtekmaj/date-utils';
 
 import Input from './Input';
 
-import {
-  getHours,
-  getMinutes,
-  getSeconds,
-} from '../shared/dates';
 import { isTime } from '../shared/propTypes';
 import { min, max } from '../shared/utils';
 

src/shared/dates.js

@@ -1,38 +1,3 @@
-import {
-  getHours,
-  getMinutes,
-  getSeconds,
-} from 'react-clock/dist/shared/dates';
-
-export {
-  getHours,
-  getMinutes,
-  getSeconds,
-};
-
-export function getHoursMinutes(date) {
-  if (!date) {
-    return date;
-  }
-
-  const hours = `0${getHours(date)}`.slice(-2);
-  const minutes = `0${getMinutes(date)}`.slice(-2);
-
-  return `${hours}:${minutes}`;
-}
-
-export function getHoursMinutesSeconds(date) {
-  if (!date) {
-    return date;
-  }
-
-  const hours = `0${getHours(date)}`.slice(-2);
-  const minutes = `0${getMinutes(date)}`.slice(-2);
-  const seconds = `0${getSeconds(date)}`.slice(-2);
-
-  return `${hours}:${minutes}:${seconds}`;
-}
-
 export function convert12to24(hour12, amPm) {
   let hour24 = parseInt(hour12, 10);
 

src/shared/dates.spec.js

@@ -1,78 +1,8 @@
 import {
-  getHoursMinutes,
-  getHoursMinutesSeconds,
   convert12to24,
   convert24to12,
 } from './dates';
 
-describe('getHoursMinutes', () => {
-  it('returns proper hour and minute for a given date', () => {
-    const date = new Date(2017, 0, 1, 16, 4);
-
-    const hoursMinutes = getHoursMinutes(date);
-
-    expect(hoursMinutes).toBe('16:04');
-  });
-
-  it('returns proper hour and minute for a given string of hour and minute', () => {
-    const date = '16:04';
-
-    const hoursMinutes = getHoursMinutes(date);
-
-    expect(hoursMinutes).toBe('16:04');
-  });
-
-  it('returns proper hour and minute for a given string of hour, minute and second', () => {
-    const date = '16:04:08';
-
-    const hoursMinutes = getHoursMinutes(date);
-
-    expect(hoursMinutes).toBe('16:04');
-  });
-
-  it('throws an error when given nonsense data', () => {
-    const text = 'wololo';
-    const flag = true;
-
-    expect(() => getHoursMinutes(text)).toThrow();
-    expect(() => getHoursMinutes(flag)).toThrow();
-  });
-});
-
-describe('getHoursMinutesSeconds', () => {
-  it('returns proper hour, minute and second for a given date', () => {
-    const date = new Date(2017, 0, 1, 16, 4, 41);
-
-    const hoursMinutesSeconds = getHoursMinutesSeconds(date);
-
-    expect(hoursMinutesSeconds).toBe('16:04:41');
-  });
-
-  it('returns proper hour, minute and second for a given string of hour and minute', () => {
-    const date = '16:04';
-
-    const hoursMinutesSeconds = getHoursMinutesSeconds(date);
-
-    expect(hoursMinutesSeconds).toBe('16:04:00');
-  });
-
-  it('returns proper hour, minute and second for a given string of hour, minute and second', () => {
-    const date = '16:04:08';
-
-    const hoursMinutesSeconds = getHoursMinutesSeconds(date);
-
-    expect(hoursMinutesSeconds).toBe('16:04:08');
-  });
-
-  it('throws an error when given nonsense data', () => {
-    const text = 'wololo';
-    const flag = true;
-
-    expect(() => getHoursMinutesSeconds(text)).toThrow();
-    expect(() => getHoursMinutesSeconds(flag)).toThrow();
-  });
-});
-
 describe('convert12to24', () => {
   it.each`
     hour12 | amPm    | hour24