Skip to content

Commit 924baa0

Browse files
committed
fix(tests): updated tests
1 parent 269a864 commit 924baa0

4 files changed

Lines changed: 12 additions & 6 deletions

File tree

src/app/app.component.spec.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,10 @@ import { ComponentFixture, TestBed } from '@angular/core/testing';
88
import { By } from '@angular/platform-browser';
99

1010
import { GetCurrentUser, UserState } from '@core/store/user';
11+
import { UserEmailsState } from '@core/store/user-emails';
1112

1213
import { FullScreenLoaderComponent, ToastComponent } from './shared/components';
14+
import { TranslateServiceMock } from './shared/mocks';
1315
import { AppComponent } from './app.component';
1416

1517
describe('AppComponent', () => {
@@ -19,7 +21,12 @@ describe('AppComponent', () => {
1921
beforeEach(async () => {
2022
await TestBed.configureTestingModule({
2123
imports: [AppComponent, ...MockComponents(ToastComponent, FullScreenLoaderComponent)],
22-
providers: [provideStore([UserState]), provideHttpClient(), provideHttpClientTesting()],
24+
providers: [
25+
provideStore([UserState, UserEmailsState]),
26+
provideHttpClient(),
27+
provideHttpClientTesting(),
28+
TranslateServiceMock,
29+
],
2330
}).compileComponents();
2431

2532
fixture = TestBed.createComponent(AppComponent);

src/app/features/settings/account-settings/account-settings.component.spec.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,6 @@ describe('AccountSettingsComponent', () => {
3737
case UserSelectors.getCurrentUser:
3838
return () => MOCK_USER;
3939

40-
case AccountSettingsSelectors.getEmails:
41-
return () => [];
42-
4340
case AccountSettingsSelectors.getAccountSettings:
4441
return () => null;
4542

src/app/features/settings/account-settings/components/add-email/add-email.component.spec.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import { provideHttpClient } from '@angular/common/http';
99
import { provideHttpClientTesting } from '@angular/common/http/testing';
1010
import { ComponentFixture, TestBed } from '@angular/core/testing';
1111

12+
import { UserEmailsState } from '@core/store/user-emails';
1213
import { TranslateServiceMock } from '@shared/mocks';
1314
import { ToastService } from '@shared/services';
1415

@@ -25,7 +26,7 @@ describe('AddEmailComponent', () => {
2526
await TestBed.configureTestingModule({
2627
imports: [AddEmailComponent, MockPipe(TranslatePipe)],
2728
providers: [
28-
provideStore([AccountSettingsState]),
29+
provideStore([AccountSettingsState, UserEmailsState]),
2930
MockProviders(DynamicDialogRef, ToastService),
3031
TranslateServiceMock,
3132
provideHttpClient(),

src/app/features/settings/account-settings/components/connected-emails/connected-emails.component.spec.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import { provideHttpClientTesting } from '@angular/common/http/testing';
1212
import { DestroyRef } from '@angular/core';
1313
import { ComponentFixture, TestBed } from '@angular/core/testing';
1414

15+
import { UserEmailsState } from '@core/store/user-emails';
1516
import { SetCurrentUser, UserState } from '@osf/core/store/user';
1617
import { AddEmailComponent, ConfirmationSentDialogComponent } from '@osf/features/settings/account-settings/components';
1718
import { MOCK_USER, MockCustomConfirmationServiceProvider } from '@shared/mocks';
@@ -42,7 +43,7 @@ describe('ConnectedEmailsComponent', () => {
4243
await TestBed.configureTestingModule({
4344
imports: [ConnectedEmailsComponent, MockPipe(TranslatePipe)],
4445
providers: [
45-
provideStore([AccountSettingsState, UserState]),
46+
provideStore([AccountSettingsState, UserState, UserEmailsState]),
4647
provideHttpClient(),
4748
provideHttpClientTesting(),
4849
MockProviders(DialogService, TranslateService, DestroyRef, LoaderService, ToastService),

0 commit comments

Comments
 (0)