Skip to content

Commit e261d36

Browse files
[IIIF-188] fix tests
1 parent c953020 commit e261d36

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

src/app/core/services/server-hard-redirect.service.spec.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,11 @@ describe('ServerHardRedirectService', () => {
88
const mockRequest = jasmine.createSpyObj(['get']);
99
const mockResponse = jasmine.createSpyObj(['redirect', 'end']);
1010

11-
let service: ServerHardRedirectService = new ServerHardRedirectService(environment, mockRequest, mockResponse);
11+
const serverResponseService = jasmine.createSpyObj('ServerResponseService', {
12+
setHeader: jasmine.createSpy('setHeader'),
13+
});
14+
15+
let service: ServerHardRedirectService = new ServerHardRedirectService(environment, mockRequest, mockResponse, serverResponseService);
1216
const origin = 'https://test-host.com:4000';
1317

1418
beforeEach(() => {
@@ -76,7 +80,7 @@ describe('ServerHardRedirectService', () => {
7680
ssrBaseUrl: 'https://private-url:4000/server',
7781
baseUrl: 'https://public-url/server',
7882
} } };
79-
service = new ServerHardRedirectService(environmentWithSSRUrl, mockRequest, mockResponse);
83+
service = new ServerHardRedirectService(environmentWithSSRUrl, mockRequest, mockResponse, serverResponseService);
8084

8185
beforeEach(() => {
8286
service.redirect(redirect);

src/app/core/services/server-hard-redirect.service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ export class ServerHardRedirectService extends HardRedirectService {
7777
this.setCorsHeader();
7878

7979
this.res.redirect(status, redirectUrl);
80-
//this.res.end();
80+
this.res.end();
8181
// I haven't found a way to correctly stop Angular rendering.
8282
// So we just let it end its work, though we have already closed
8383
// the response.

0 commit comments

Comments
 (0)