Skip to content

Commit 895d05a

Browse files
ppaternostroppaternostro
committed
Use provideHttpClientTesting provider for spec files
1 parent ef7e9be commit 895d05a

5 files changed

Lines changed: 35 additions & 36 deletions

File tree

‎src/app/app.component.spec.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { HttpHandler, provideHttpClient } from '@angular/common/http';
1+
import { provideHttpClientTesting } from '@angular/common/http/testing';
22
import { TestBed } from '@angular/core/testing';
33
import { AppComponent } from './app.component';
44
import { PostService } from './service/rest/post/post.service';
@@ -7,7 +7,7 @@ describe('AppComponent', () => {
77
beforeEach(async () => {
88
await TestBed.configureTestingModule({
99
imports: [AppComponent],
10-
providers: [provideHttpClient(), HttpHandler, PostService],
10+
providers: [provideHttpClientTesting(), PostService],
1111
}).compileComponents();
1212
});
1313

‎src/app/component/content/content.component.spec.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { HttpHandler, provideHttpClient } from '@angular/common/http';
1+
import { provideHttpClientTesting } from '@angular/common/http/testing';
22
import { ComponentFixture, TestBed } from '@angular/core/testing';
33
import { PostService } from '../../service/rest/post/post.service';
44
import { ContentComponent } from './content.component';
@@ -10,7 +10,7 @@ describe('ContentComponent', () => {
1010
beforeEach(async () => {
1111
await TestBed.configureTestingModule({
1212
imports: [ContentComponent],
13-
providers: [provideHttpClient(), HttpHandler, PostService],
13+
providers: [provideHttpClientTesting(), PostService],
1414
}).compileComponents();
1515
});
1616

‎src/app/component/result/result.component.spec.ts‎

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { ComponentFixture, TestBed } from '@angular/core/testing';
22

3-
import { HttpHandler, provideHttpClient } from '@angular/common/http';
3+
import { provideHttpClientTesting } from '@angular/common/http/testing';
44
import { By } from '@angular/platform-browser';
55
import { of } from 'rxjs';
66
import { Post } from '../../model/post';
@@ -15,7 +15,7 @@ describe('ResultComponent', () => {
1515
beforeEach(async () => {
1616
await TestBed.configureTestingModule({
1717
imports: [ResultComponent],
18-
providers: [provideHttpClient(), HttpHandler, PostService],
18+
providers: [provideHttpClientTesting(), PostService],
1919
}).compileComponents();
2020

2121
fixture = TestBed.createComponent(ResultComponent);
@@ -30,7 +30,7 @@ describe('ResultComponent', () => {
3030
it('should contain zero rows and the CREATE button should be disabled on initial state', () => {
3131
const rows = fixture.debugElement.queryAll(By.css('table tbody tr')).length;
3232
const createButton = fixture.debugElement.query(
33-
By.css('#result-create')
33+
By.css('#result-create'),
3434
).nativeElement;
3535

3636
expect(rows).toBe(0);
@@ -44,7 +44,7 @@ describe('ResultComponent', () => {
4444

4545
const rows = fixture.debugElement.queryAll(By.css('table tbody tr')).length;
4646
const createButton = fixture.debugElement.query(
47-
By.css('#result-create')
47+
By.css('#result-create'),
4848
).nativeElement;
4949

5050
expect(rows).toBe(1);
@@ -64,7 +64,7 @@ describe('ResultComponent', () => {
6464

6565
let rows = fixture.debugElement.queryAll(By.css('table tbody tr')).length;
6666
const createButton = fixture.debugElement.query(
67-
By.css('#result-create')
67+
By.css('#result-create'),
6868
).nativeElement;
6969

7070
expect(rows).toBe(1);
@@ -75,7 +75,7 @@ describe('ResultComponent', () => {
7575
});
7676

7777
const deleteIcon = fixture.debugElement.query(
78-
By.css('#result-delete')
78+
By.css('#result-delete'),
7979
).nativeElement;
8080

8181
deleteIcon.click();
@@ -94,34 +94,34 @@ describe('ResultComponent', () => {
9494

9595
let rows = fixture.debugElement.queryAll(By.css('table tbody tr')).length;
9696
const createButton = fixture.debugElement.query(
97-
By.css('#result-create')
97+
By.css('#result-create'),
9898
).nativeElement;
9999

100100
expect(rows).toBe(1);
101101
expect(createButton.disabled).toBeTruthy();
102102

103103
const editIcon = fixture.debugElement.query(
104-
By.css('#result-edit')
104+
By.css('#result-edit'),
105105
).nativeElement;
106106

107107
editIcon.click();
108108

109109
fixture.detectChanges();
110110

111111
let titleInput = fixture.debugElement.query(
112-
By.css('#result-title-input')
112+
By.css('#result-title-input'),
113113
).nativeElement;
114114

115115
let bodyInput = fixture.debugElement.query(
116-
By.css('#result-body-input')
116+
By.css('#result-body-input'),
117117
).nativeElement;
118118

119119
let saveIcon = fixture.debugElement.query(
120-
By.css('#result-save')
120+
By.css('#result-save'),
121121
).nativeElement;
122122

123123
let cancelIcon = fixture.debugElement.query(
124-
By.css('#result-cancel')
124+
By.css('#result-cancel'),
125125
).nativeElement;
126126

127127
expect(titleInput).toBeTruthy();
@@ -151,30 +151,30 @@ describe('ResultComponent', () => {
151151

152152
let rows = fixture.debugElement.queryAll(By.css('table tbody tr')).length;
153153
const createButton = fixture.debugElement.query(
154-
By.css('#result-create')
154+
By.css('#result-create'),
155155
).nativeElement;
156156

157157
expect(rows).toBe(1);
158158
expect(createButton.disabled).toBeTruthy();
159159

160160
const editIcon = fixture.debugElement.query(
161-
By.css('#result-edit')
161+
By.css('#result-edit'),
162162
).nativeElement;
163163

164164
editIcon.click();
165165

166166
fixture.detectChanges();
167167

168168
let titleInput = fixture.debugElement.query(
169-
By.css('#result-title-input')
169+
By.css('#result-title-input'),
170170
).nativeElement;
171171

172172
let bodyInput = fixture.debugElement.query(
173-
By.css('#result-body-input')
173+
By.css('#result-body-input'),
174174
).nativeElement;
175175

176176
let saveIcon = fixture.debugElement.query(
177-
By.css('#result-save')
177+
By.css('#result-save'),
178178
).nativeElement;
179179

180180
expect(titleInput).toBeTruthy();
@@ -204,10 +204,10 @@ describe('ResultComponent', () => {
204204
expect(saveIcon).toBeFalsy();
205205

206206
const title = fixture.debugElement.query(
207-
By.css('#result-title')
207+
By.css('#result-title'),
208208
).nativeElement;
209209
const body = fixture.debugElement.query(
210-
By.css('#result-body')
210+
By.css('#result-body'),
211211
).nativeElement;
212212

213213
expect(title.textContent.trim()).toBe('Title Change');

‎src/app/component/search/search.component.spec.ts‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { ComponentFixture, TestBed } from '@angular/core/testing';
22

33
import { HarnessLoader } from '@angular/cdk/testing';
44
import { TestbedHarnessEnvironment } from '@angular/cdk/testing/testbed';
5-
import { HttpHandler, provideHttpClient } from '@angular/common/http';
5+
import { provideHttpClientTesting } from '@angular/common/http/testing';
66
import { MatSelectHarness } from '@angular/material/select/testing';
77
import { By } from '@angular/platform-browser';
88
import { PostService } from '../../service/rest/post/post.service';
@@ -16,7 +16,7 @@ describe('SearchComponent', () => {
1616
beforeEach(async () => {
1717
await TestBed.configureTestingModule({
1818
imports: [SearchComponent],
19-
providers: [provideHttpClient(), HttpHandler, PostService],
19+
providers: [provideHttpClientTesting(), PostService],
2020
}).compileComponents();
2121

2222
fixture = TestBed.createComponent(SearchComponent);
@@ -31,10 +31,10 @@ describe('SearchComponent', () => {
3131

3232
it('should ensure default state on creation', async () => {
3333
const searchText = fixture.debugElement.query(
34-
By.css('#search-text')
34+
By.css('#search-text'),
3535
).nativeElement;
3636
const searchButton = fixture.debugElement.query(
37-
By.css('#search')
37+
By.css('#search'),
3838
).nativeElement;
3939

4040
const selectHarness = await loader.getHarness(MatSelectHarness);
@@ -53,10 +53,10 @@ describe('SearchComponent', () => {
5353
fixture.detectChanges();
5454

5555
const searchText = fixture.debugElement.query(
56-
By.css('#search-text')
56+
By.css('#search-text'),
5757
).nativeElement;
5858
const searchButton = fixture.debugElement.query(
59-
By.css('#search')
59+
By.css('#search'),
6060
).nativeElement;
6161

6262
const selectHarness = await loader.getHarness(MatSelectHarness);

‎src/app/service/rest/post/post.service.spec.ts‎

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import {
66
TestRequest,
77
} from '@angular/common/http/testing';
88

9-
import { HttpHandler, provideHttpClient } from '@angular/common/http';
109
import { Subscription } from 'rxjs';
1110
import { mockPost } from '../../../../mock/model/post/mock-post';
1211
import { Post } from '../../../model/post';
@@ -20,7 +19,7 @@ describe('PostService', () => {
2019

2120
beforeEach(() => {
2221
TestBed.configureTestingModule({
23-
providers: [HttpHandler, provideHttpClient(), provideHttpClientTesting()],
22+
providers: [provideHttpClientTesting()],
2423
});
2524
service = TestBed.inject(PostService);
2625
backEnd = TestBed.inject(HttpTestingController);
@@ -42,7 +41,7 @@ describe('PostService', () => {
4241
tracker.add(
4342
service.getPost(mockPost.id).subscribe((result: Post) => {
4443
expect(result).toEqual(mockPost);
45-
})
44+
}),
4645
);
4746

4847
const call: TestRequest = backEnd.expectOne({
@@ -57,7 +56,7 @@ describe('PostService', () => {
5756
tracker.add(
5857
service.getPosts().subscribe((result: Post[]) => {
5958
expect(result).toEqual([mockPost]);
60-
})
59+
}),
6160
);
6261

6362
const call: TestRequest = backEnd.expectOne({
@@ -72,7 +71,7 @@ describe('PostService', () => {
7271
tracker.add(
7372
service.savePost(mockPost).subscribe((result: Post) => {
7473
expect(result).toEqual(mockPost);
75-
})
74+
}),
7675
);
7776

7877
const call: TestRequest = backEnd.expectOne({
@@ -87,7 +86,7 @@ describe('PostService', () => {
8786
tracker.add(
8887
service.deletePost(mockPost).subscribe((result: Post) => {
8988
expect(result).toEqual(mockPost);
90-
})
89+
}),
9190
);
9291

9392
const call: TestRequest = backEnd.expectOne({
@@ -102,7 +101,7 @@ describe('PostService', () => {
102101
tracker.add(
103102
service.createPost(mockPost).subscribe((result: Post) => {
104103
expect(result).toEqual(mockPost);
105-
})
104+
}),
106105
);
107106

108107
const call: TestRequest = backEnd.expectOne({

0 commit comments

Comments
 (0)