Skip to content

Commit

Permalink
Fixes unit tests
Browse files Browse the repository at this point in the history
  • Loading branch information
digimezzo committed Sep 1, 2024
1 parent b59cd8d commit df86f74
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 3 deletions.
4 changes: 4 additions & 0 deletions src/app/app.component.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,10 @@ import { DesktopBase } from './common/io/desktop.base';
import { AudioVisualizer } from './services/playback/audio-visualizer';
import { PlaybackServiceBase } from './services/playback/playback.service.base';
import { LifetimeService } from './services/lifetime/lifetime.service';
import { SwitchPlayerService } from './services/player-switcher/switch-player.service';

describe('AppComponent', () => {
let playerSwitcherServiceMock: IMock<SwitchPlayerService>;
let playbackServiceMock: IMock<PlaybackServiceBase>;
let navigationServiceMock: IMock<NavigationServiceBase>;
let appearanceServiceMock: IMock<AppearanceServiceBase>;
Expand All @@ -43,6 +45,7 @@ describe('AppComponent', () => {

function createComponent(): AppComponent {
return new AppComponent(
playerSwitcherServiceMock.object,
playbackServiceMock.object,
navigationServiceMock.object,
appearanceServiceMock.object,
Expand All @@ -62,6 +65,7 @@ describe('AppComponent', () => {
}

beforeEach(() => {
playerSwitcherServiceMock = Mock.ofType<SwitchPlayerService>();
playbackServiceMock = Mock.ofType<PlaybackServiceBase>();
navigationServiceMock = Mock.ofType<NavigationServiceBase>();
appearanceServiceMock = Mock.ofType<AppearanceServiceBase>();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,20 @@ import { CoverPlayerComponent } from './cover-player.component';
import { AppearanceServiceBase } from '../../../../services/appearance/appearance.service.base';
import { IMock, Mock } from 'typemoq';
import { MatBottomSheet } from '@angular/material/bottom-sheet';
import { NavigationServiceBase } from '../../../../services/navigation/navigation.service.base';

describe('CoverPlayerComponent', () => {
let appearanceServiceMock: IMock<AppearanceServiceBase>;
let navigationServiceMock: IMock<NavigationServiceBase>;
let matBottomSheetMock: IMock<MatBottomSheet>;

let component: CoverPlayerComponent;

beforeEach(() => {
appearanceServiceMock = Mock.ofType<AppearanceServiceBase>();
navigationServiceMock = Mock.ofType<NavigationServiceBase>();
matBottomSheetMock = Mock.ofType<MatBottomSheet>();
component = new CoverPlayerComponent(appearanceServiceMock.object, matBottomSheetMock.object);
component = new CoverPlayerComponent(appearanceServiceMock.object, navigationServiceMock.object, matBottomSheetMock.object);
});

describe('constructor', () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ describe('PlaybackQueueComponent', () => {
let playbackQueue: TrackModels;
let playbackServicePlaybackStarted: Subject<PlaybackStarted>;
let navigationServiceShowPlaybackQueueRequested: Subject<void>;
let refreshPlaybackQueueListRequested: Subject<void>;
let settingsMock: any;

function createComponent(): PlaybackQueueComponent {
Expand Down Expand Up @@ -59,6 +60,10 @@ describe('PlaybackQueueComponent', () => {
navigationServiceShowPlaybackQueueRequested = new Subject();
const navigationServiceShowPlaybackQueueRequested$: Observable<void> = navigationServiceShowPlaybackQueueRequested.asObservable();
navigationServiceMock.setup((x) => x.showPlaybackQueueRequested$).returns(() => navigationServiceShowPlaybackQueueRequested$);

refreshPlaybackQueueListRequested = new Subject();
const refreshPlaybackQueueListRequested$: Observable<void> = refreshPlaybackQueueListRequested.asObservable();
navigationServiceMock.setup((x) => x.refreshPlaybackQueueListRequested$).returns(() => refreshPlaybackQueueListRequested$);
});

describe('constructor', () => {
Expand Down Expand Up @@ -149,7 +154,7 @@ describe('PlaybackQueueComponent', () => {

// Act
component.ngOnInit();
navigationServiceShowPlaybackQueueRequested.next();
refreshPlaybackQueueListRequested.next();

// Assert
mouseSelectionWatcherMock.verify((x) => x.initialize(playbackQueue.tracks), Times.once());
Expand All @@ -163,7 +168,7 @@ describe('PlaybackQueueComponent', () => {
component.ngOnInit();
const shouldShowListBefore: boolean = component.shouldShowList;
jest.useFakeTimers();
navigationServiceShowPlaybackQueueRequested.next();
refreshPlaybackQueueListRequested.next();
jest.runAllTimers();
const shouldShowListAfter: boolean = component.shouldShowList;

Expand Down

0 comments on commit df86f74

Please sign in to comment.