Skip to content

Commit

Permalink
refactor: remove wait time since it is the same as assigned time
Browse files Browse the repository at this point in the history
  • Loading branch information
BenMaruchu committed Oct 7, 2019
1 parent 8876806 commit b7d6b45
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 107 deletions.
3 changes: 0 additions & 3 deletions src/base/facets.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,6 @@ const METRIC_TIMES = {
maximumCallTime: { $max: '$call.duration.milliseconds' },
minimumCallTime: { $min: '$call.duration.milliseconds' },
averageCallTime: { $avg: '$call.duration.milliseconds' },
maximumWaitTime: { $max: '$waitTime' },
minimumWaitTime: { $min: '$waitTime' },
averageWaitTime: { $avg: '$waitTime' },
maximumWorkTime: { $max: '$workTime' },
minimumWorkTime: { $min: '$workTime' },
averageWorkTime: { $avg: '$workTime' },
Expand Down
7 changes: 0 additions & 7 deletions src/base/servicerequest.base.js
Original file line number Diff line number Diff line change
Expand Up @@ -290,13 +290,6 @@ const METRIC_TIMES_FIELDS = {
* marked as complete
*/
workTime: { $subtract: ['$completedAt', '$assignedAt'] },

/**
* This is the time elapsed between when service request have been confirmed
* (reported by an operator) and when is have been assigned to someone to work
* on it
*/
waitTime: { $subtract: ['$assignedAt', '$confirmedAt'] },
};

/* fields that can be added service request base aggregation */
Expand Down
4 changes: 0 additions & 4 deletions src/util.js
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@ export const normalizeMetricTimes = data => {
'resolveTime',
'lateTime',
'callTime',
'waitTime',
'workTime',
];

Expand Down Expand Up @@ -118,9 +117,6 @@ export const normalizeMetricTimes = data => {
'maximumCallTime',
'minimumCallTime',
'averageCallTime',
'maximumWaitTime',
'minimumWaitTime',
'averageWaitTime',
'maximumWorkTime',
'minimumWorkTime',
'averageWorkTime',
Expand Down
93 changes: 0 additions & 93 deletions test/unit/util.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -123,9 +123,6 @@ describe('Utils', () => {
maximumWorkTime: 0,
minimumWorkTime: 0,
averageWorkTime: 0,
maximumWaitTime: 0,
minimumWaitTime: 0,
averageWaitTime: 0,
};

const expectedOutput = {
Expand Down Expand Up @@ -421,35 +418,6 @@ describe('Utils', () => {
seconds: 0,
},
},
waitTime: {
average: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
maximum: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
minimum: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
},
resolved: 2,
};

Expand Down Expand Up @@ -493,9 +461,6 @@ describe('Utils', () => {
maximumCallTime: 0,
minimumCallTime: 0,
averageCallTime: 0,
maximumWaitTime: null,
minimumWaitTime: null,
averageWaitTime: null,
maximumWorkTime: null,
minimumWorkTime: null,
averageWorkTime: null,
Expand Down Expand Up @@ -804,35 +769,6 @@ describe('Utils', () => {
seconds: 0,
},
},
waitTime: {
average: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
maximum: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
minimum: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
},
workTime: {
average: {
days: 0,
Expand Down Expand Up @@ -1130,35 +1066,6 @@ describe('Utils', () => {
seconds: 0,
},
},
waitTime: {
average: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
maximum: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
minimum: {
days: 0,
hours: 0,
microseconds: 0,
milliseconds: 0,
minutes: 0,
nanoseconds: 0,
seconds: 0,
},
},
workTime: {
average: {
days: 0,
Expand Down

0 comments on commit b7d6b45

Please sign in to comment.