diff --git a/frontend/app/model/network/autehentication.service.spec.ts b/frontend/app/model/network/autehentication.service.spec.ts index c2497d9..b0a3bf6 100644 --- a/frontend/app/model/network/autehentication.service.spec.ts +++ b/frontend/app/model/network/autehentication.service.spec.ts @@ -1,14 +1,13 @@ import {inject, TestBed} from "@angular/core/testing"; import {UserService} from "./user.service"; import {UserDTO} from "../../../../common/entities/UserDTO"; -import {Message} from "../../../../common/entities/Message"; import "rxjs/Rx"; import {LoginCredential} from "../../../../common/entities/LoginCredential"; import {AuthenticationService} from "./authentication.service"; class MockUserService { - public login(credential: LoginCredential) { - return Promise.resolve(new Message(null, {name: "testUserName"})) + public login(credential: LoginCredential): Promise { + return Promise.resolve({name: "testUserName"}) } public async getSessionUser() { @@ -34,20 +33,23 @@ describe('AuthenticationService', () => { expect(userService.login).toHaveBeenCalled(); })); - it('should have NO Authenticated use', inject([AuthenticationService], (authService) => { - expect(authService.getUser()).toBe(null); + it('should have NO Authenticated use', inject([AuthenticationService], (authService: AuthenticationService) => { + expect(authService.user.value).toBe(null); expect(authService.isAuthenticated()).toBe(false); })); - it('should have Authenticated use', inject([AuthenticationService], (authService) => { - spyOn(authService.OnUserChanged, "trigger").and.callThrough(); - authService.login(); - authService.OnUserChanged.on(() => { - expect(authService.OnUserChanged.trigger).toHaveBeenCalled(); - expect(authService.getUser()).not.toBe(null); + it('should have Authenticated use', inject([AuthenticationService], (authService: AuthenticationService) => { + spyOn(authService.user, "next").and.callThrough(); + authService.user.subscribe((user) => { + if (user == null) { + return; + } + expect(authService.user.next).toHaveBeenCalled(); + expect(authService.user.value).not.toBe(null); expect(authService.isAuthenticated()).toBe(true); }); + authService.login({}); })); diff --git a/frontend/app/model/network/network.service.spec.ts b/frontend/app/model/network/network.service.spec.ts index d6a5215..38ee3c2 100644 --- a/frontend/app/model/network/network.service.spec.ts +++ b/frontend/app/model/network/network.service.spec.ts @@ -53,37 +53,37 @@ describe('NetworkService Success tests', () => { it('should call GET', inject([NetworkService], (networkService) => { - networkService.getJson(testUrl).then((res: Message) => { - expect(res.result).toBe(testResponse); + networkService.getJson(testUrl).then((res: any) => { + expect(res).toBe(testResponse); }); })); it('should call POST', inject([NetworkService, MockBackend], (networkService) => { - networkService.postJson(testUrl, testData).then((res: Message) => { - expect(res.result).toBe(testResponse); + networkService.postJson(testUrl, testData).then((res: any) => { + expect(res).toBe(testResponse); }); expect(connection.request.text()).toBe(JSON.stringify(testData)); - networkService.postJson(testUrl).then((res: Message) => { - expect(res.result).toBe(testResponse); + networkService.postJson(testUrl).then((res: any) => { + expect(res).toBe(testResponse); }); expect(connection.request.text()).toBe(JSON.stringify({})); })); it('should call PUT', inject([NetworkService, MockBackend], (networkService) => { - networkService.putJson(testUrl, testData).then((res: Message) => { - expect(res.result).toBe(testResponse); + networkService.putJson(testUrl, testData).then((res: any) => { + expect(res).toBe(testResponse); }); expect(connection.request.text()).toBe(JSON.stringify(testData)); - networkService.putJson(testUrl).then((res: Message) => { - expect(res.result).toBe(testResponse); + networkService.putJson(testUrl).then((res: any) => { + expect(res).toBe(testResponse); }); expect(connection.request.text()).toBe(JSON.stringify({})); @@ -91,8 +91,8 @@ describe('NetworkService Success tests', () => { it('should call DELETE', inject([NetworkService, MockBackend], (networkService) => { - networkService.deleteJson(testUrl).then((res: Message) => { - expect(res.result).toBe(testResponse); + networkService.deleteJson(testUrl).then((res: any) => { + expect(res).toBe(testResponse); }); })); });