feat(isBefore): allow usage of options object

README.md

@@ -98,7 +98,7 @@ Validator                               | Description
 **isBase32(str [, options])**           | check if a string is base32 encoded. `options` is optional and defaults to `{crockford: false}`.<br/> When `crockford` is true it tests the given base32 encoded string using [Crockford's base32 alternative](http://www.crockford.com/base32.html).
 **isBase58(str)**                       | check if a string is base58 encoded.
 **isBase64(str [, options])**          | check if a string is base64 encoded. options is optional and defaults to `{urlSafe: false}`<br/> when `urlSafe` is true it tests the given base64 encoded string is [url safe](https://base64.guru/standards/base64url)
-**isBefore(str [, date])**              | check if the string is a date that's before the specified date.
+**isBefore(str [, options])**              | check if the string is a date that's before the specified date.<br/><br/>`options` is an object that defaults to `{ comparisonDate: Date().toString() }`.<br/><br/>**Options:**<br/>`comparisonDate`: Date to compare to. Defaults to the current date `Date().toString()` (now).
 **isBIC(str)**                          | check if a string is a BIC (Bank Identification Code) or SWIFT code.
 **isBoolean(str [, options])**          | check if a string is a boolean.<br/>`options` is an object which defaults to `{ loose: false }`. If loose is is set to false, the validator will strictly match ['true', 'false', '0', '1']. If loose is set to true, the validator will also match 'yes', 'no', and will match a valid boolean string of any case. (eg: ['true', 'True', 'TRUE']).
 **isBtcAddress(str)**            | check if the string is a valid BTC address.

src/lib/isBefore.js

@@ -1,9 +1,11 @@
-import assertString from './util/assertString';
 import toDate from './toDate';
 
-export default function isBefore(str, date = String(new Date())) {
-  assertString(str);
-  const comparison = toDate(date);
-  const original = toDate(str);
-  return !!(original && comparison && original < comparison);
+export default function isBefore(date, options) {
+  // accessing 'arguments' for backwards compatibility: isBefore(date [, comparisonDate])
+  // eslint-disable-next-line prefer-rest-params
+  const comparisonDate = (typeof options === 'object' ? options.comparisonDate : arguments[1]) || Date().toString();
+  const comparison = toDate(comparisonDate);
+  const original = toDate(date);
+
+  return !!(original < comparison);
 }

src/lib/toDate.js

@@ -2,6 +2,6 @@ import assertString from './util/assertString';
 
 export default function toDate(date) {
   assertString(date);
-  date = Date.parse(date);
-  return !isNaN(date) ? new Date(date) : null;
+
+  return new Date(date);
 }

test/validators.test.js

@@ -4993,41 +4993,6 @@ describe('Validators', () => {
     });
   });
 
-  it('should validate dates against an end date', () => {
-    test({
-      validator: 'isBefore',
-      args: ['08/04/2011'],
-      valid: ['2010-07-02', '2010-08-04', new Date(0).toString()],
-      invalid: ['08/04/2011', new Date(2011, 9, 10).toString()],
-    });
-    test({
-      validator: 'isBefore',
-      args: [new Date(2011, 7, 4).toString()],
-      valid: ['2010-07-02', '2010-08-04', new Date(0).toString()],
-      invalid: ['08/04/2011', new Date(2011, 9, 10).toString()],
-    });
-    test({
-      validator: 'isBefore',
-      valid: [
-        '2000-08-04',
-        new Date(0).toString(),
-        new Date(Date.now() - 86400000).toString(),
-      ],
-      invalid: ['2100-07-02', new Date(2217, 10, 10).toString()],
-    });
-    test({
-      validator: 'isBefore',
-      args: ['2011-08-03'],
-      valid: ['1999-12-31'],
-      invalid: ['invalid date'],
-    });
-    test({
-      validator: 'isBefore',
-      args: ['invalid date'],
-      invalid: ['invalid date', '1999-12-31'],
-    });
-  });
-
   it('should validate IBAN', () => {
     test({
       validator: 'isIBAN',

test/validators/isBefore.test.js

@@ -0,0 +1,75 @@
+import { test } from '../testFunctions';
+
+describe('isBefore', () => {
+  it('should validate dates against an end date', () => {
+    test({
+      validator: 'isBefore',
+      args: [{ comparisonDate: '08/04/2011' }],
+      valid: ['2010-07-02', '2010-08-04', new Date(0).toString()],
+      invalid: ['08/04/2011', new Date(2011, 9, 10).toString()],
+    });
+    test({
+      validator: 'isBefore',
+      args: [{ comparisonDate: new Date(2011, 7, 4).toString() }],
+      valid: ['2010-07-02', '2010-08-04', new Date(0).toString()],
+      invalid: ['08/04/2011', new Date(2011, 9, 10).toString()],
+    });
+    test({
+      validator: 'isBefore',
+      valid: [
+        '2000-08-04',
+        new Date(0).toString(),
+        new Date(Date.now() - 86400000).toString(),
+      ],
+      invalid: ['2100-07-02', new Date(2217, 10, 10).toString()],
+    });
+    test({
+      validator: 'isBefore',
+      args: [{ comparisonDate: '2011-08-03' }],
+      valid: ['1999-12-31'],
+      invalid: ['invalid date'],
+    });
+    test({
+      validator: 'isBefore',
+      args: [{ comparisonDate: 'invalid date' }],
+      invalid: ['invalid date', '1999-12-31'],
+    });
+  });
+
+  describe('(legacy syntax)', () => {
+    it('should validate dates against an end date', () => {
+      test({
+        validator: 'isBefore',
+        args: ['08/04/2011'],
+        valid: ['2010-07-02', '2010-08-04', new Date(0).toString()],
+        invalid: ['08/04/2011', new Date(2011, 9, 10).toString()],
+      });
+      test({
+        validator: 'isBefore',
+        args: [new Date(2011, 7, 4).toString()],
+        valid: ['2010-07-02', '2010-08-04', new Date(0).toString()],
+        invalid: ['08/04/2011', new Date(2011, 9, 10).toString()],
+      });
+      test({
+        validator: 'isBefore',
+        valid: [
+          '2000-08-04',
+          new Date(0).toString(),
+          new Date(Date.now() - 86400000).toString(),
+        ],
+        invalid: ['2100-07-02', new Date(2217, 10, 10).toString()],
+      });
+      test({
+        validator: 'isBefore',
+        args: ['2011-08-03'],
+        valid: ['1999-12-31'],
+        invalid: ['invalid date'],
+      });
+      test({
+        validator: 'isBefore',
+        args: ['invalid date'],
+        invalid: ['invalid date', '1999-12-31'],
+      });
+    });
+  });
+});