add full test-coverage for DateRange; export definitions;

Author: mbauer83

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@mbauer83/ts-utils",
-  "version": "v0.2.3",
+  "version": "v0.2.4",
   "type": "module",
   "types": "src/main.d.ts",
   "repository": "https://github.com/mbauer83/ts-utils.git",

package.json

@@ -41,11 +41,11 @@ export class DateRange {
     }
 
     hasLowerBound(): boolean {
-        return this.fromDate !== null;
+        return this.fromDate !== undefined;
     }
 
     hasUpperBound(): boolean {
-        return this.toDate !== null;
+        return this.toDate !== undefined;
     }
 
     isInRange(dateOrMillisecondTimestamp: Date | number): boolean {
@@ -55,9 +55,9 @@ export class DateRange {
 
     toString(): string {
         const lowerBoundDelimiter = this.lowerBoundIsInclusive ? '[' : '(';
-        const lowerBoundPart = lowerBoundDelimiter + (this.hasLowerBound() ? this.fromDate!.toString() : '...') + ', ';
+        const lowerBoundPart = lowerBoundDelimiter + (this.hasLowerBound() ? this.fromDate!.toISOString() : '...') + ', ';
         const upperBoundDelimiter = this.upperBoundIsInclusive ? ']' : ')';
-        const upperBoundPart = (this.hasUpperBound() ? this.toDate!.toString() : '...') + upperBoundDelimiter;
+        const upperBoundPart = (this.hasUpperBound() ? this.toDate!.toISOString() : '...') + upperBoundDelimiter;
         return lowerBoundPart + upperBoundPart;
     }
 }

src/date/DateRange.ts

@@ -140,4 +140,37 @@ describe('test DateRange', () => {
         expect(DateRange.empty.fromDate).toBe(undefined);
         expect(DateRange.empty.toDate).toBe(undefined);
     })
+
+    test('hasLowerBound reports correctly', () => {
+        expect(new DateRange().hasLowerBound()).toBe(false);
+        expect(new DateRange(new Date()).hasLowerBound()).toBe(true);
+        expect(new DateRange(undefined, new Date()).hasLowerBound()).toBe(false);
+        expect(new DateRange(new Date(), new Date()).hasLowerBound()).toBe(true);
+    })
+
+    test('hasUpperBound reports correctly', () => {
+        expect(new DateRange().hasUpperBound()).toBe(false);
+        expect(new DateRange(undefined, new Date()).hasUpperBound()).toBe(true);
+        expect(new DateRange(new Date(), undefined).hasUpperBound()).toBe(false);
+        expect(new DateRange(new Date(), new Date()).hasUpperBound()).toBe(true);
+    })
+
+    test('toString returns correct string', () => {
+        // Should use '(' / ')' for exclusive bounds and '[' / ']' for inclusive bounds.
+        // Should use '...' for undefined bounds and ISO 8601 for defined bounds.
+        expect(new DateRange().toString()).toBe('(..., ...)');
+        expect(new DateRange(undefined, undefined, true, true).toString()).toBe('[..., ...]');
+        const currDate = new Date();
+        const expectedDateString = currDate.toISOString();
+        expect(new DateRange(currDate).toString()).toBe(`(${expectedDateString}, ...)`);
+        expect(new DateRange(undefined, currDate).toString()).toBe(`(..., ${expectedDateString})`);
+        expect(new DateRange(currDate, undefined, true, true).toString()).toBe(`[${expectedDateString}, ...]`);
+        expect(new DateRange(undefined, currDate, true, true).toString()).toBe(`[..., ${expectedDateString}]`);
+        expect(new DateRange(currDate, undefined, true, false).toString()).toBe(`[${expectedDateString}, ...)`);
+        expect(new DateRange(currDate, undefined, false, true).toString()).toBe(`(${expectedDateString}, ...]`);
+        expect(new DateRange(undefined, currDate, true, false).toString()).toBe(`[..., ${expectedDateString})`);
+        expect(new DateRange(undefined, currDate, false, true).toString()).toBe(`(..., ${expectedDateString}]`);        
+        expect(new DateRange(currDate, currDate).toString()).toBe(`(${expectedDateString}, ${expectedDateString})`);
+        expect(new DateRange(currDate, currDate, true, true).toString()).toBe(`[${expectedDateString}, ${expectedDateString}]`);
+    });
 });

tests/unit/dateRange.test.ts

@@ -6,6 +6,7 @@
         "allowJs": true,
         "checkJs": false,
         "declaration": true,
+        "declarationDir": "./dist/types",
         "declarationMap": true,
         "outDir": "./dist",
         "sourceMap": true,