Skip to content

Commit 7bab731

Browse files
committed
fixup! build: update all non-major dependencies
1 parent 6b1bae5 commit 7bab731

3 files changed

Lines changed: 17 additions & 6 deletions

File tree

src/youtube-player/fake-youtube-player.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
* found in the LICENSE file at https://angular.dev/license
77
*/
88

9+
import type * as YT from 'youtube';
10+
911
// A re-creation of YT.PlayerState since enum values cannot be bound to the window object.
1012
const playerState = {
1113
UNSTARTED: -1,

src/youtube-player/youtube-player.spec.ts

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import {
1515
YouTubePlayer,
1616
} from './youtube-player';
1717
import {PlaceholderImageQuality} from './youtube-player-placeholder';
18+
import type * as YT from 'youtube';
1819

1920
declare var window: Window;
2021

@@ -200,7 +201,7 @@ describe('YoutubePlayer', () => {
200201
fixture.detectChanges();
201202
events.onReady({target: playerSpy});
202203

203-
const playerVars: YT.PlayerVars = {modestbranding: YT.ModestBranding.Modest};
204+
const playerVars: YT.PlayerVars = {modestbranding: (window as any).YT.ModestBranding.Modest};
204205
fixture.componentInstance.playerVars = playerVars;
205206
fixture.changeDetectorRef.markForCheck();
206207
fixture.detectChanges();
@@ -389,7 +390,9 @@ describe('YoutubePlayer', () => {
389390
fixture.detectChanges();
390391

391392
testComponent.youtubePlayer.playVideo();
392-
expect(testComponent.youtubePlayer.getPlayerState()).toBe(YT.PlayerState.PLAYING);
393+
expect(testComponent.youtubePlayer.getPlayerState()).toBe(
394+
(window as any).YT.PlayerState.PLAYING,
395+
);
393396

394397
events.onReady({target: playerSpy});
395398

@@ -401,7 +404,9 @@ describe('YoutubePlayer', () => {
401404
fixture.detectChanges();
402405

403406
testComponent.youtubePlayer.pauseVideo();
404-
expect(testComponent.youtubePlayer.getPlayerState()).toBe(YT.PlayerState.PAUSED);
407+
expect(testComponent.youtubePlayer.getPlayerState()).toBe(
408+
(window as any).YT.PlayerState.PAUSED,
409+
);
405410

406411
events.onReady({target: playerSpy});
407412

@@ -413,7 +418,9 @@ describe('YoutubePlayer', () => {
413418
fixture.detectChanges();
414419

415420
testComponent.youtubePlayer.stopVideo();
416-
expect(testComponent.youtubePlayer.getPlayerState()).toBe(YT.PlayerState.CUED);
421+
expect(testComponent.youtubePlayer.getPlayerState()).toBe(
422+
(window as any).YT.PlayerState.CUED,
423+
);
417424

418425
events.onReady({target: playerSpy});
419426

src/youtube-player/youtube-player.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
// Workaround for: https://github.com/bazelbuild/rules_nodejs/issues/1265
1010
/// <reference types="youtube" preserve="true" />
1111

12+
import type * as YT from 'youtube';
13+
1214
import {
1315
ChangeDetectionStrategy,
1416
Component,
@@ -576,7 +578,7 @@ export class YouTubePlayer implements AfterViewInit, OnChanges, OnDestroy {
576578

577579
// A player can't be created if the API isn't loaded,
578580
// or there isn't a video or playlist to be played.
579-
if (typeof YT === 'undefined' || (!this.videoId && !this.playerVars?.list)) {
581+
if (typeof (window as any).YT === 'undefined' || (!this.videoId && !this.playerVars?.list)) {
580582
return;
581583
}
582584

@@ -597,7 +599,7 @@ export class YouTubePlayer implements AfterViewInit, OnChanges, OnDestroy {
597599
params.videoId = this.videoId;
598600
}
599601
const player = this._ngZone.runOutsideAngular(
600-
() => new YT.Player(this.youtubeContainer.nativeElement, params),
602+
() => new (window as any).YT.Player(this.youtubeContainer.nativeElement, params),
601603
);
602604

603605
const whenReady = (event: YT.PlayerEvent) => {

0 commit comments

Comments
 (0)