MM-58772 Fix counter metrics not using Date.now (#27702)
Этот коммит содержится в:
коммит произвёл
GitHub
родитель
68737af37d
Коммит
a773e6828c
@@ -94,7 +94,7 @@ describe('PerformanceReporter', () => {
|
|||||||
|
|
||||||
expect(reporter.handleObservations).toHaveBeenCalled();
|
expect(reporter.handleObservations).toHaveBeenCalled();
|
||||||
|
|
||||||
const timestamp = performance.timeOrigin + performance.now();
|
const timestamp = Date.now();
|
||||||
|
|
||||||
await waitForReport();
|
await waitForReport();
|
||||||
|
|
||||||
@@ -113,8 +113,8 @@ describe('PerformanceReporter', () => {
|
|||||||
},
|
},
|
||||||
],
|
],
|
||||||
});
|
});
|
||||||
expect(report.start).toBeGreaterThan(timestamp);
|
expect(report.start).toBeGreaterThanOrEqual(timestamp);
|
||||||
expect(report.end).toBeGreaterThan(timestamp);
|
expect(report.end).toBeGreaterThanOrEqual(timestamp);
|
||||||
expect(report.start).toEqual(report.end);
|
expect(report.start).toEqual(report.end);
|
||||||
|
|
||||||
reporter.disconnect();
|
reporter.disconnect();
|
||||||
|
|||||||
@@ -268,7 +268,7 @@ export default class PerformanceReporter {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private generateReport(histogramMeasures: PerformanceReportMeasure[], counters: Map<string, number>): PerformanceReport {
|
private generateReport(histogramMeasures: PerformanceReportMeasure[], counters: Map<string, number>): PerformanceReport {
|
||||||
const now = performance.timeOrigin + performance.now();
|
const now = Date.now();
|
||||||
|
|
||||||
const counterMeasures = this.countersToMeasures(now, counters);
|
const counterMeasures = this.countersToMeasures(now, counters);
|
||||||
|
|
||||||
@@ -289,7 +289,7 @@ export default class PerformanceReporter {
|
|||||||
|
|
||||||
private getReportStartEnd(now: number, histogramMeasures: PerformanceReportMeasure[], counterMeasures: PerformanceReportMeasure[]): {start: number; end: number} {
|
private getReportStartEnd(now: number, histogramMeasures: PerformanceReportMeasure[], counterMeasures: PerformanceReportMeasure[]): {start: number; end: number} {
|
||||||
let start = now;
|
let start = now;
|
||||||
let end = performance.timeOrigin;
|
let end = 0;
|
||||||
|
|
||||||
for (const measure of histogramMeasures) {
|
for (const measure of histogramMeasures) {
|
||||||
start = Math.min(start, measure.timestamp);
|
start = Math.min(start, measure.timestamp);
|
||||||
|
|||||||
Ссылка в новой задаче
Block a user