Skip to content

Commit ee54c8d

Browse files
committed
make it prettier!
1 parent a6a1aad commit ee54c8d

File tree

3 files changed

+23
-13
lines changed

3 files changed

+23
-13
lines changed

spec/unit/webrtc/stats/statsReportBuilder.spec.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ describe("StatsReportBuilder", () => {
112112
remoteAudioTrack.setLoss({ packetsTotal: 20, packetsLost: 0, isDownloadStream: true });
113113
remoteAudioTrack.setBitrate({ download: 4000, upload: 0 });
114114
remoteAudioTrack.setJitter(0.1);
115-
remoteAudioTrack.setAudioConcealment(3000,3000 * 20);
115+
remoteAudioTrack.setAudioConcealment(3000, 3000 * 20);
116116

117117
localVideoTrack.setCodec("v8");
118118
localVideoTrack.setLoss({ packetsTotal: 30, packetsLost: 6, isDownloadStream: false });

src/webrtc/stats/statsReportBuilder.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -66,14 +66,14 @@ export class StatsReportBuilder {
6666
const audioConcealmentForTrack = trackStats.getAudioConcealment();
6767
totalConcealedAudio += audioConcealmentForTrack.concealedAudio;
6868
totalAudioDuration += audioConcealmentForTrack.totalAudioDuration;
69-
69+
7070
audioBitrateDownload += trackStats.getBitrate().download;
7171
audioBitrateUpload += trackStats.getBitrate().upload;
7272
} else {
7373
videoBitrateDownload += trackStats.getBitrate().download;
7474
videoBitrateUpload += trackStats.getBitrate().upload;
7575
}
76-
76+
7777
resolutions[trackStats.getType()].set(trackId, trackStats.getResolution());
7878
framerates[trackStats.getType()].set(trackId, trackStats.getFramerate());
7979
codecs[trackStats.getType()].set(trackId, trackStats.getCodec());

src/webrtc/stats/summaryStatsReporter.ts

+20-10
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,12 @@ export class SummaryStatsReporter {
2929
if (summaryTotalCount === 0) {
3030
return;
3131
}
32-
const summaryCounter: SummaryCounter = { receivedAudio: 0, receivedVideo: 0, receivedMedia: 0, concealedAudio:0 };
32+
const summaryCounter: SummaryCounter = {
33+
receivedAudio: 0,
34+
receivedVideo: 0,
35+
receivedMedia: 0,
36+
concealedAudio: 0,
37+
};
3338
let maxJitter = 0;
3439
let maxPacketLoss = 0;
3540
summary.forEach((stats) => {
@@ -38,14 +43,22 @@ export class SummaryStatsReporter {
3843
maxJitter = this.buildMaxJitter(maxJitter, stats);
3944
maxPacketLoss = this.buildMaxPacketLoss(maxPacketLoss, stats);
4045
});
41-
const digitsForPercentages = 5;
46+
const decimalPlaces = 5;
4247
const report = {
43-
percentageReceivedMedia: Number((summaryCounter.receivedMedia / summaryTotalCount).toFixed(digitsForPercentages)),
44-
percentageReceivedVideoMedia: Number((summaryCounter.receivedVideo / summaryTotalCount).toFixed(digitsForPercentages)),
45-
percentageReceivedAudioMedia: Number((summaryCounter.receivedAudio / summaryTotalCount).toFixed(digitsForPercentages)),
48+
percentageReceivedMedia: Number(
49+
(summaryCounter.receivedMedia / summaryTotalCount).toFixed(decimalPlaces),
50+
),
51+
percentageReceivedVideoMedia: Number(
52+
(summaryCounter.receivedVideo / summaryTotalCount).toFixed(decimalPlaces),
53+
),
54+
percentageReceivedAudioMedia: Number(
55+
(summaryCounter.receivedAudio / summaryTotalCount).toFixed(decimalPlaces),
56+
),
4657
maxJitter,
4758
maxPacketLoss,
48-
percentageConcealedAudio: Number((summaryCounter.concealedAudio / summaryTotalCount).toFixed(digitsForPercentages)),
59+
percentageConcealedAudio: Number(
60+
(summaryCounter.concealedAudio / summaryTotalCount).toFixed(decimalPlaces),
61+
),
4962
} as SummaryStatsReport;
5063
this.emitter.emitSummaryStatsReport(report);
5164
}
@@ -94,10 +107,7 @@ export class SummaryStatsReporter {
94107
return maxPacketLoss;
95108
}
96109

97-
private countPercentageConcealedAudio(
98-
summaryCounter: SummaryCounter,
99-
stats: SummaryStats,
100-
): void {
110+
private countPercentageConcealedAudio(summaryCounter: SummaryCounter, stats: SummaryStats): void {
101111
summaryCounter.concealedAudio += stats.audioTrackSummary.concealedAudioRatio;
102112
}
103113
}

0 commit comments

Comments
 (0)