Skip to content

Commit 538bd21

Browse files
committed
remove unused vars and and update test message content
1 parent 602ca3d commit 538bd21

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

src/types.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -89,14 +89,6 @@ export type AdBreak = {
8989
* Length of time of the complete ad break
9090
*/
9191
duration: number;
92-
/**
93-
* Timestamp for AdBreak start playback
94-
*/
95-
adBreakStartTimestamp?: number;
96-
/**
97-
* Timestamp for AdBreak end playback
98-
*/
99-
adBreakEndTimestamp?: number;
10092
};
10193

10294
/**

test/session.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ describe('MediaSession', () => {
8282
).to.eql(adBreak);
8383
});
8484

85-
it('should not resume after AdBreakEnd if user paused before ad break and excludeAdBreaksFromContentTime is true', async () => {
85+
it('should not accumulate mediaContentTimeSpent after AdBreakEnd if user paused before ad break and excludeAdBreaksFromContentTime is true', async () => {
8686
const customSession: MediaSession = new MediaSession(
8787
mp,
8888
song.contentId,
@@ -240,7 +240,7 @@ describe('MediaSession', () => {
240240
expect(bond.args[0][0].options.currentPlayheadPosition).to.eq(32);
241241
});
242242

243-
it('should pause mediaContentTimeSpent when excludeAdBreaksFromContentTime is true', async () => {
243+
it('should pause mediaContentTimeSpent when logging ad breaks when excludeAdBreaksFromContentTime is true', async () => {
244244
const mediaSessionAttributes = {
245245
session_name: 'amazing-current-session',
246246
session_start_time: 'right-now',
@@ -360,7 +360,7 @@ describe('MediaSession', () => {
360360
expect(mpMediaContentTimeSpent).to.lessThanOrEqual(400);
361361
});
362362

363-
it('should not resume after AdBreakEnd if user paused during ad break and excludeAdBreaksFromContentTime is true', async () => {
363+
it('should not accumulate mediaContentTimeSpent after AdBreakEnd if user paused during ad break and excludeAdBreaksFromContentTime is true', async () => {
364364
const mediaSessionAttributes = {
365365
session_name: 'amazing-current-session',
366366
session_start_time: 'right-now',

0 commit comments

Comments
 (0)