Skip to content

Commit 8c50155

Browse files
committed
jwt is returned as json in body
1 parent 7b673c1 commit 8c50155

File tree

5 files changed

+12
-12
lines changed

5 files changed

+12
-12
lines changed

src/app/core/services/login.service.spec.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import {inject, TestBed} from "@angular/core/testing";
22
import {
3-
Headers,
43
ResponseOptions,
54
Response,
65
RequestMethod,
@@ -34,7 +33,7 @@ describe('LoginService', () => {
3433
it('can login', (done) => {
3534
backend.connections.subscribe(conn => {
3635
conn.mockRespond(new Response(new ResponseOptions({
37-
headers: new Headers({'x-auth-token': 'my jwt'}),
36+
body: JSON.stringify({token: 'my jwt'}),
3837
})));
3938
expect(conn.request.method).toEqual(RequestMethod.Post);
4039
expect(conn.request.url).toEqual('/api/login');

src/app/core/services/login.service.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ export class LoginService {
1818
email: email,
1919
password: password,
2020
};
21-
return this.http.post('/api/login', body).do(resp => {
22-
localStorage.setItem('jwt', resp.headers.get('x-auth-token'));
21+
return this.http.post('/api/login', body).do((resp: Response) => {
22+
localStorage.setItem('jwt', resp.json().token);
2323
this.authEvents.next(new DidLogin());
2424
});
2525
}

src/app/core/services/user.service.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,7 @@ export class UserService {
3232
}
3333

3434
updateMe(userParam: UserParams): Observable<Response> {
35-
return this.http.patch(`${url}/me`, userParam)
36-
.do(resp => {
37-
localStorage.setItem('jwt', resp.headers.get('x-auth-token'));
38-
});
35+
return this.http.patch(`${url}/me`, userParam);
3936
}
4037

4138
listFollowings(userId: string,

src/app/pages/+signup/signup.component.spec.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import {
77
TestBed,
88
ComponentFixture
99
} from "@angular/core/testing";
10-
import {BaseResponseOptions, Response} from "@angular/http";
10+
import {BaseResponseOptions, Response, ResponseOptions} from "@angular/http";
1111
import {MockBackend} from "@angular/http/testing";
1212
import {Router} from "@angular/router";
1313
import {SignupComponent} from "./signup.component";
@@ -93,7 +93,9 @@ describe('SignupComponent', () => {
9393
const page: SignupComponent = cmpDebugElement.componentInstance;
9494
spyOn(loginService, 'login').and.callThrough();
9595
backend.connections.subscribe(conn => {
96-
conn.mockRespond(new Response(new BaseResponseOptions()));
96+
conn.mockRespond(new Response(new ResponseOptions({
97+
body: JSON.stringify({token: 'my jwt'}),
98+
})));
9799
});
98100
page.onSubmit({
99101
email: 'test@test.com',

src/app/pages/login/login.component.spec.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import {
77
ComponentFixture,
88
inject
99
} from "@angular/core/testing";
10-
import {BaseResponseOptions, Response} from "@angular/http";
10+
import {BaseResponseOptions, Response, ResponseOptions} from "@angular/http";
1111
import {Location} from "@angular/common";
1212
import {MockBackend} from "@angular/http/testing";
1313
import {Router} from "@angular/router";
@@ -77,7 +77,9 @@ describe('LoginComponent', () => {
7777
const cmp: LoginComponent = cmpDebugElement.componentInstance;
7878
spyOn(loginService, 'login').and.callThrough();
7979
backend.connections.subscribe(conn => {
80-
conn.mockRespond(new Response(new BaseResponseOptions()));
80+
conn.mockRespond(new Response(new ResponseOptions({
81+
body: JSON.stringify({token: 'my jwt'}),
82+
})));
8183
});
8284
cmp.login('test@test.com', 'secret');
8385
expect(loginService.login).toHaveBeenCalledWith('test@test.com', 'secret');

0 commit comments

Comments
 (0)