luxolus/open-nomad
2
0
Fork 0
Code Issues 1 Pull requests Packages Releases Wiki Activity

Account for the 0 case in format-duration

Browse source
This commit is contained in:
Michael Lange 2018-05-30 14:30:45 -07:00
parent 5eac0a53f6
commit e472d918cf
2 changed files with 24 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
ui/app/utils/format-duration.js
View file

@ -15,6 +15,9 @@ const allUnits = [
export default function formatDuration(duration = 0, units = 'ns') { export default function formatDuration(duration = 0, units = 'ns') {
const durationParts = {}; const durationParts = {};
// Moment only handles up to millisecond precision.
// Microseconds and nanoseconds need to be handled first,
// then Moment can take over for all larger units.
if (units === 'ns') { if (units === 'ns') {
durationParts.nanoseconds = duration % 1000; durationParts.nanoseconds = duration % 1000;
durationParts.microseconds = Math.floor((duration % 1000000) / 1000); durationParts.microseconds = Math.floor((duration % 1000000) / 1000);
@ -24,8 +27,13 @@ export default function formatDuration(duration = 0, units = 'ns') {
duration = Math.floor(duration / 1000); duration = Math.floor(duration / 1000);
} }
const momentDuration = moment.duration(duration, ['ns', 'mms'].includes(units) ? 'ms' : units); let momentUnits = units;
if (units === 'ns' || units === 'mms') {
momentUnits = 'ms';
}
const momentDuration = moment.duration(duration, momentUnits);
// Get the count of each time unit that Moment handles
allUnits allUnits
.filterBy('inMoment') .filterBy('inMoment')
.mapBy('name') .mapBy('name')
@ -33,6 +41,8 @@ export default function formatDuration(duration = 0, units = 'ns') {
durationParts[unit] = momentDuration[unit](); durationParts[unit] = momentDuration[unit]();
}); });
// Format each time time bucket as a string
// e.g., { years: 5, seconds: 30 } -> [ '5 years', '30s' ]
const displayParts = allUnits.reduce((parts, unitType) => { const displayParts = allUnits.reduce((parts, unitType) => {
if (durationParts[unitType.name]) { if (durationParts[unitType.name]) {
const count = durationParts[unitType.name]; const count = durationParts[unitType.name];
@ -43,5 +53,12 @@ export default function formatDuration(duration = 0, units = 'ns') {
return parts; return parts;
}, []); }, []);
return displayParts.join(' '); if (displayParts.length) {
return displayParts.join(' ');
}
// When the duration is 0, show 0 in terms of `units`
const unitTypeForUnits = allUnits.findBy('suffix', units);
const suffix = unitTypeForUnits.pluralizable ? units.pluralize() : units;
return `0${unitTypeForUnits.pluralizable ? ' ' : ''}${suffix}`;
} }

5
ui/tests/unit/utils/format-duration-test.js
View file

@ -21,3 +21,8 @@ test('The units option allows for units coarser than nanoseconds', function(asse
assert.equal(formatDuration(1200, 's'), expectation2, expectation2); assert.equal(formatDuration(1200, 's'), expectation2, expectation2);
assert.equal(formatDuration(32, 'd'), expectation3, expectation3); assert.equal(formatDuration(32, 'd'), expectation3, expectation3);
}); });
test('When duration is 0, 0 is shown in terms of the units provided to the function', function(assert) {
assert.equal(formatDuration(0), '0ns', 'formatDuration(0) -> 0ns');
assert.equal(formatDuration(0, 'year'), '0 years', 'formatDuration(0, "year") -> 0 years');
});