chore: bump pretty-ms (#398)

* chore: bump pretty-ms * fix: unit test * fix: Add unit test for colonNotation
apache-superset · Apr 28, 2020 · 771cf20 · 771cf20 · vercel · Apr 28, 2020
1 parent 2c3fe61
commit 771cf20
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/superset-ui-number-format/package.json b/packages/superset-ui-number-format/package.json
@@ -28,7 +28,7 @@
   "dependencies": {
     "@types/d3-format": "^1.3.0",
     "d3-format": "^1.3.2",
-    "pretty-ms": "^6.0.1"
+    "pretty-ms": "^7.0.0"
   },
   "peerDependencies": {
     "@superset-ui/core": "^0.12.0"

diff --git a/packages/superset-ui-number-format/test/factories/createDurationFormatter.test.ts b/packages/superset-ui-number-format/test/factories/createDurationFormatter.test.ts
@@ -24,8 +24,10 @@ describe('createDurationFormatter()', () => {
     expect(formatter(90)).toBe('1m 30s');
   });
   it('format milliseconds in human readable format with additional pretty-ms options', () => {
+    const colonNotationFormatter = createDurationFormatter({ colonNotation: true });
+    expect(colonNotationFormatter(10500)).toBe('0:10.5');
     const zeroDecimalFormatter = createDurationFormatter({ secondsDecimalDigits: 0 });
-    expect(zeroDecimalFormatter(10500)).toBe('11s');
+    expect(zeroDecimalFormatter(10500)).toBe('10s');
     const subMillisecondFormatter = createDurationFormatter({ formatSubMilliseconds: true });
     expect(subMillisecondFormatter(100.40008)).toBe('100ms 400µs 80ns');
   });