Skip to content

Commit f67adba

Browse files
committed
[#11] Update creating ApiToken from LoginResponse
1 parent 29d75ba commit f67adba

File tree

4 files changed

+9
-11
lines changed

4 files changed

+9
-11
lines changed

lib/model/api_token.dart

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import 'package:json_annotation/json_annotation.dart';
2-
import 'package:survey_flutter/model/response/login_response.dart';
32
import 'package:survey_flutter/storage/secure_storage.dart';
43

54
part 'api_token.g.dart';
@@ -24,12 +23,6 @@ class ApiToken extends SecureStorageModel {
2423

2524
Map<String, dynamic> toJson() => _$ApiTokenToJson(this);
2625

27-
static ApiToken from(LoginResponse loginResponse) => ApiToken(
28-
accessToken: loginResponse.accessToken,
29-
refreshToken: loginResponse.refreshToken,
30-
tokenType: loginResponse.tokenType,
31-
);
32-
3326
@override
3427
bool operator ==(Object other) =>
3528
other is ApiToken &&

lib/model/response/login_response.dart

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import 'package:json_annotation/json_annotation.dart';
22
import 'package:survey_flutter/api/response_decoder.dart';
3+
import 'package:survey_flutter/model/api_token.dart';
34
import 'package:survey_flutter/model/login_model.dart';
45

56
part 'login_response.g.dart';
@@ -32,6 +33,12 @@ class LoginResponse {
3233
refreshToken: refreshToken,
3334
);
3435

36+
ApiToken toApiToken() => ApiToken(
37+
accessToken: accessToken,
38+
refreshToken: refreshToken,
39+
tokenType: tokenType,
40+
);
41+
3542
static LoginResponse dummy() {
3643
return LoginResponse(
3744
id: "",

lib/repositories/authentication_repository.dart

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import 'package:survey_flutter/api/authentication_api_service.dart';
33
import 'package:survey_flutter/api/exception/network_exceptions.dart';
44
import 'package:survey_flutter/di/provider/dio_provider.dart';
55
import 'package:survey_flutter/env.dart';
6-
import 'package:survey_flutter/model/api_token.dart';
76
import 'package:survey_flutter/model/login_model.dart';
87
import 'package:survey_flutter/model/request/login_request.dart';
98
import 'package:survey_flutter/storage/secure_storage.dart';
@@ -49,7 +48,7 @@ class AuthenticationRepositoryImpl extends AuthenticationRepository {
4948
grantType: _grantType,
5049
));
5150
await _secureStorage.save(
52-
value: ApiToken.from(response),
51+
value: response.toApiToken(),
5352
key: SecureStorageKey.apiToken,
5453
);
5554
return response.toLoginModel();

test/api/repositories/authentication_repository_test.dart

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ import 'package:flutter_config/flutter_config.dart';
22
import 'package:flutter_test/flutter_test.dart';
33
import 'package:mockito/mockito.dart';
44
import 'package:survey_flutter/api/exception/network_exceptions.dart';
5-
import 'package:survey_flutter/model/api_token.dart';
65
import 'package:survey_flutter/model/response/login_response.dart';
76
import 'package:survey_flutter/repositories/authentication_repository.dart';
87
import 'package:survey_flutter/storage/secure_storage.dart';
@@ -46,7 +45,7 @@ void main() {
4645
expect(result, loginResponse.toLoginModel());
4746
verify(
4847
mockSecureStorage.save(
49-
value: ApiToken.from(loginResponse),
48+
value: loginResponse.toApiToken(),
5049
key: SecureStorageKey.apiToken,
5150
),
5251
).called(1);

0 commit comments

Comments
 (0)