refactor: Clean Architecture 적용 및 코드베이스 전면 리팩토링
## 주요 변경사항 ### 아키텍처 개선 - Clean Architecture 패턴 적용 (Domain, Data, Presentation 레이어 분리) - Use Case 패턴 도입으로 비즈니스 로직 캡슐화 - Repository 패턴으로 데이터 접근 추상화 - 의존성 주입 구조 개선 ### 상태 관리 최적화 - 모든 Controller에서 불필요한 상태 관리 로직 제거 - 페이지네이션 로직 통일 및 간소화 - 에러 처리 로직 개선 (에러 메시지 한글화) - 로딩 상태 관리 최적화 ### Mock 서비스 제거 - MockDataService 완전 제거 - 모든 화면을 실제 API 전용으로 전환 - 불필요한 Mock 관련 코드 정리 ### UI/UX 개선 - Overview 화면 대시보드 기능 강화 - 라이선스 만료 알림 위젯 추가 - 사이드바 네비게이션 개선 - 일관된 UI 컴포넌트 사용 ### 코드 품질 - 중복 코드 제거 및 함수 추출 - 파일별 책임 분리 명확화 - 테스트 코드 업데이트 ## 영향 범위 - 모든 화면의 Controller 리팩토링 - API 통신 레이어 구조 개선 - 에러 처리 및 로깅 시스템 개선 ## 향후 계획 - 단위 테스트 커버리지 확대 - 통합 테스트 시나리오 추가 - 성능 모니터링 도구 통합
This commit is contained in:
JiWoong Sul
@@ -0,0 +1,231 @@
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:mockito/mockito.dart';
|
||||
import 'package:mockito/annotations.dart';
|
||||
import 'package:dartz/dartz.dart';
|
||||
import 'package:superport/data/models/warehouse_location/warehouse_location.dart';
|
||||
import 'package:superport/data/repositories/warehouse_location_repository.dart';
|
||||
import 'package:superport/domain/usecases/base_usecase.dart';
|
||||
import 'package:superport/domain/usecases/warehouse_location/create_warehouse_location_usecase.dart';
|
||||
|
||||
import 'create_warehouse_location_usecase_test.mocks.dart';
|
||||
|
||||
@GenerateMocks([WarehouseLocationRepository])
|
||||
void main() {
|
||||
late CreateWarehouseLocationUseCase useCase;
|
||||
late MockWarehouseLocationRepository mockRepository;
|
||||
|
||||
setUp(() {
|
||||
mockRepository = MockWarehouseLocationRepository();
|
||||
useCase = CreateWarehouseLocationUseCase(mockRepository);
|
||||
});
|
||||
|
||||
group('CreateWarehouseLocationUseCase', () {
|
||||
final validParams = CreateWarehouseLocationParams(
|
||||
name: 'Main Warehouse',
|
||||
address: '123 Storage Street',
|
||||
description: 'Primary storage location',
|
||||
contactNumber: '010-1234-5678',
|
||||
manager: 'John Doe',
|
||||
latitude: 37.5665,
|
||||
longitude: 126.9780,
|
||||
);
|
||||
|
||||
final mockLocation = WarehouseLocation(
|
||||
id: 1,
|
||||
name: 'Main Warehouse',
|
||||
address: '123 Storage Street',
|
||||
description: 'Primary storage location',
|
||||
contactNumber: '010-1234-5678',
|
||||
manager: 'John Doe',
|
||||
latitude: 37.5665,
|
||||
longitude: 126.9780,
|
||||
isActive: true,
|
||||
createdAt: DateTime.now(),
|
||||
updatedAt: DateTime.now(),
|
||||
);
|
||||
|
||||
test('창고 위치 생성 성공', () async {
|
||||
// arrange
|
||||
when(mockRepository.createWarehouseLocation(any))
|
||||
.thenAnswer((_) async => mockLocation);
|
||||
|
||||
// act
|
||||
final result = await useCase(validParams);
|
||||
|
||||
// assert
|
||||
expect(result.isRight(), true);
|
||||
result.fold(
|
||||
(failure) => fail('Should not return failure'),
|
||||
(location) => expect(location, equals(mockLocation)),
|
||||
);
|
||||
verify(mockRepository.createWarehouseLocation(validParams.toMap())).called(1);
|
||||
});
|
||||
|
||||
test('창고 이름이 비어있는 경우 검증 실패', () async {
|
||||
// arrange
|
||||
final invalidParams = CreateWarehouseLocationParams(
|
||||
name: '', // 빈 이름
|
||||
address: '123 Storage Street',
|
||||
);
|
||||
|
||||
// act
|
||||
final result = await useCase(invalidParams);
|
||||
|
||||
// assert
|
||||
expect(result.isLeft(), true);
|
||||
result.fold(
|
||||
(failure) {
|
||||
expect(failure, isA<ValidationFailure>());
|
||||
expect(failure.message, contains('창고 위치 이름은 필수입니다'));
|
||||
},
|
||||
(location) => fail('Should not return location'),
|
||||
);
|
||||
verifyNever(mockRepository.createWarehouseLocation(any));
|
||||
});
|
||||
|
||||
test('창고 주소가 비어있는 경우 검증 실패', () async {
|
||||
// arrange
|
||||
final invalidParams = CreateWarehouseLocationParams(
|
||||
name: 'Main Warehouse',
|
||||
address: '', // 빈 주소
|
||||
);
|
||||
|
||||
// act
|
||||
final result = await useCase(invalidParams);
|
||||
|
||||
// assert
|
||||
expect(result.isLeft(), true);
|
||||
result.fold(
|
||||
(failure) {
|
||||
expect(failure, isA<ValidationFailure>());
|
||||
expect(failure.message, contains('창고 주소는 필수입니다'));
|
||||
},
|
||||
(location) => fail('Should not return location'),
|
||||
);
|
||||
verifyNever(mockRepository.createWarehouseLocation(any));
|
||||
});
|
||||
|
||||
test('잘못된 연락처 형식인 경우 검증 실패', () async {
|
||||
// arrange
|
||||
final invalidParams = CreateWarehouseLocationParams(
|
||||
name: 'Main Warehouse',
|
||||
address: '123 Storage Street',
|
||||
contactNumber: 'invalid-phone!@#', // 잘못된 형식
|
||||
);
|
||||
|
||||
// act
|
||||
final result = await useCase(invalidParams);
|
||||
|
||||
// assert
|
||||
expect(result.isLeft(), true);
|
||||
result.fold(
|
||||
(failure) {
|
||||
expect(failure, isA<ValidationFailure>());
|
||||
expect(failure.message, contains('올바른 연락처 형식이 아닙니다'));
|
||||
},
|
||||
(location) => fail('Should not return location'),
|
||||
);
|
||||
verifyNever(mockRepository.createWarehouseLocation(any));
|
||||
});
|
||||
|
||||
test('올바른 연락처 형식들 허용', () async {
|
||||
// arrange
|
||||
final validPhoneNumbers = [
|
||||
'010-1234-5678',
|
||||
'02-123-4567',
|
||||
'+82-10-1234-5678',
|
||||
'(02) 123-4567',
|
||||
'010 1234 5678',
|
||||
];
|
||||
|
||||
when(mockRepository.createWarehouseLocation(any))
|
||||
.thenAnswer((_) async => mockLocation);
|
||||
|
||||
// act & assert
|
||||
for (final phone in validPhoneNumbers) {
|
||||
final params = CreateWarehouseLocationParams(
|
||||
name: 'Main Warehouse',
|
||||
address: '123 Storage Street',
|
||||
contactNumber: phone,
|
||||
);
|
||||
|
||||
final result = await useCase(params);
|
||||
expect(result.isRight(), true);
|
||||
}
|
||||
});
|
||||
|
||||
test('Repository에서 예외 발생 시 ServerFailure 반환', () async {
|
||||
// arrange
|
||||
when(mockRepository.createWarehouseLocation(any))
|
||||
.thenThrow(Exception('Server error'));
|
||||
|
||||
// act
|
||||
final result = await useCase(validParams);
|
||||
|
||||
// assert
|
||||
expect(result.isLeft(), true);
|
||||
result.fold(
|
||||
(failure) {
|
||||
expect(failure, isA<ServerFailure>());
|
||||
expect(failure.message, contains('Server error'));
|
||||
},
|
||||
(location) => fail('Should not return location'),
|
||||
);
|
||||
verify(mockRepository.createWarehouseLocation(validParams.toMap())).called(1);
|
||||
});
|
||||
|
||||
test('파라미터를 올바른 Map으로 변환', () {
|
||||
// arrange
|
||||
final params = CreateWarehouseLocationParams(
|
||||
name: 'Test Warehouse',
|
||||
address: '456 Test Avenue',
|
||||
description: 'Test description',
|
||||
contactNumber: '010-9876-5432',
|
||||
manager: 'Jane Smith',
|
||||
latitude: 35.1796,
|
||||
longitude: 129.0756,
|
||||
);
|
||||
|
||||
// act
|
||||
final map = params.toMap();
|
||||
|
||||
// assert
|
||||
expect(map['name'], equals('Test Warehouse'));
|
||||
expect(map['address'], equals('456 Test Avenue'));
|
||||
expect(map['description'], equals('Test description'));
|
||||
expect(map['contact_number'], equals('010-9876-5432'));
|
||||
expect(map['manager'], equals('Jane Smith'));
|
||||
expect(map['latitude'], equals(35.1796));
|
||||
expect(map['longitude'], equals(129.0756));
|
||||
});
|
||||
|
||||
test('옵셔널 파라미터가 null인 경우에도 정상 처리', () async {
|
||||
// arrange
|
||||
final paramsWithNulls = CreateWarehouseLocationParams(
|
||||
name: 'Main Warehouse',
|
||||
address: '123 Storage Street',
|
||||
description: null,
|
||||
contactNumber: null,
|
||||
manager: null,
|
||||
latitude: null,
|
||||
longitude: null,
|
||||
);
|
||||
|
||||
when(mockRepository.createWarehouseLocation(any))
|
||||
.thenAnswer((_) async => mockLocation);
|
||||
|
||||
// act
|
||||
final result = await useCase(paramsWithNulls);
|
||||
|
||||
// assert
|
||||
expect(result.isRight(), true);
|
||||
|
||||
final map = paramsWithNulls.toMap();
|
||||
expect(map['description'], isNull);
|
||||
expect(map['contact_number'], isNull);
|
||||
expect(map['manager'], isNull);
|
||||
expect(map['latitude'], isNull);
|
||||
expect(map['longitude'], isNull);
|
||||
});
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,171 @@
|
||||
// Mocks generated by Mockito 5.4.5 from annotations
|
||||
// in superport/test/domain/usecases/warehouse_location/create_warehouse_location_usecase_test.dart.
|
||||
// Do not manually edit this file.
|
||||
|
||||
// ignore_for_file: no_leading_underscores_for_library_prefixes
|
||||
import 'dart:async' as _i4;
|
||||
|
||||
import 'package:mockito/mockito.dart' as _i1;
|
||||
import 'package:superport/data/models/warehouse/warehouse_dto.dart' as _i2;
|
||||
import 'package:superport/data/repositories/warehouse_location_repository.dart'
|
||||
as _i3;
|
||||
|
||||
// ignore_for_file: type=lint
|
||||
// ignore_for_file: avoid_redundant_argument_values
|
||||
// ignore_for_file: avoid_setters_without_getters
|
||||
// ignore_for_file: comment_references
|
||||
// ignore_for_file: deprecated_member_use
|
||||
// ignore_for_file: deprecated_member_use_from_same_package
|
||||
// ignore_for_file: implementation_imports
|
||||
// ignore_for_file: invalid_use_of_visible_for_testing_member
|
||||
// ignore_for_file: must_be_immutable
|
||||
// ignore_for_file: prefer_const_constructors
|
||||
// ignore_for_file: unnecessary_parenthesis
|
||||
// ignore_for_file: camel_case_types
|
||||
// ignore_for_file: subtype_of_sealed_class
|
||||
|
||||
class _FakeWarehouseLocationListDto_0 extends _i1.SmartFake
|
||||
implements _i2.WarehouseLocationListDto {
|
||||
_FakeWarehouseLocationListDto_0(
|
||||
Object parent,
|
||||
Invocation parentInvocation,
|
||||
) : super(
|
||||
parent,
|
||||
parentInvocation,
|
||||
);
|
||||
}
|
||||
|
||||
class _FakeWarehouseLocationDto_1 extends _i1.SmartFake
|
||||
implements _i2.WarehouseLocationDto {
|
||||
_FakeWarehouseLocationDto_1(
|
||||
Object parent,
|
||||
Invocation parentInvocation,
|
||||
) : super(
|
||||
parent,
|
||||
parentInvocation,
|
||||
);
|
||||
}
|
||||
|
||||
/// A class which mocks [WarehouseLocationRepository].
|
||||
///
|
||||
/// See the documentation for Mockito's code generation for more information.
|
||||
class MockWarehouseLocationRepository extends _i1.Mock
|
||||
implements _i3.WarehouseLocationRepository {
|
||||
MockWarehouseLocationRepository() {
|
||||
_i1.throwOnMissingStub(this);
|
||||
}
|
||||
|
||||
@override
|
||||
_i4.Future<_i2.WarehouseLocationListDto> getWarehouseLocations({
|
||||
int? page = 1,
|
||||
int? perPage = 20,
|
||||
String? search,
|
||||
Map<String, dynamic>? filters,
|
||||
}) =>
|
||||
(super.noSuchMethod(
|
||||
Invocation.method(
|
||||
#getWarehouseLocations,
|
||||
[],
|
||||
{
|
||||
#page: page,
|
||||
#perPage: perPage,
|
||||
#search: search,
|
||||
#filters: filters,
|
||||
},
|
||||
),
|
||||
returnValue: _i4.Future<_i2.WarehouseLocationListDto>.value(
|
||||
_FakeWarehouseLocationListDto_0(
|
||||
this,
|
||||
Invocation.method(
|
||||
#getWarehouseLocations,
|
||||
[],
|
||||
{
|
||||
#page: page,
|
||||
#perPage: perPage,
|
||||
#search: search,
|
||||
#filters: filters,
|
||||
},
|
||||
),
|
||||
)),
|
||||
) as _i4.Future<_i2.WarehouseLocationListDto>);
|
||||
|
||||
@override
|
||||
_i4.Future<_i2.WarehouseLocationDto> getWarehouseLocationDetail(int? id) =>
|
||||
(super.noSuchMethod(
|
||||
Invocation.method(
|
||||
#getWarehouseLocationDetail,
|
||||
[id],
|
||||
),
|
||||
returnValue: _i4.Future<_i2.WarehouseLocationDto>.value(
|
||||
_FakeWarehouseLocationDto_1(
|
||||
this,
|
||||
Invocation.method(
|
||||
#getWarehouseLocationDetail,
|
||||
[id],
|
||||
),
|
||||
)),
|
||||
) as _i4.Future<_i2.WarehouseLocationDto>);
|
||||
|
||||
@override
|
||||
_i4.Future<_i2.WarehouseLocationDto> createWarehouseLocation(
|
||||
Map<String, dynamic>? data) =>
|
||||
(super.noSuchMethod(
|
||||
Invocation.method(
|
||||
#createWarehouseLocation,
|
||||
[data],
|
||||
),
|
||||
returnValue: _i4.Future<_i2.WarehouseLocationDto>.value(
|
||||
_FakeWarehouseLocationDto_1(
|
||||
this,
|
||||
Invocation.method(
|
||||
#createWarehouseLocation,
|
||||
[data],
|
||||
),
|
||||
)),
|
||||
) as _i4.Future<_i2.WarehouseLocationDto>);
|
||||
|
||||
@override
|
||||
_i4.Future<_i2.WarehouseLocationDto> updateWarehouseLocation(
|
||||
int? id,
|
||||
Map<String, dynamic>? data,
|
||||
) =>
|
||||
(super.noSuchMethod(
|
||||
Invocation.method(
|
||||
#updateWarehouseLocation,
|
||||
[
|
||||
id,
|
||||
data,
|
||||
],
|
||||
),
|
||||
returnValue: _i4.Future<_i2.WarehouseLocationDto>.value(
|
||||
_FakeWarehouseLocationDto_1(
|
||||
this,
|
||||
Invocation.method(
|
||||
#updateWarehouseLocation,
|
||||
[
|
||||
id,
|
||||
data,
|
||||
],
|
||||
),
|
||||
)),
|
||||
) as _i4.Future<_i2.WarehouseLocationDto>);
|
||||
|
||||
@override
|
||||
_i4.Future<void> deleteWarehouseLocation(int? id) => (super.noSuchMethod(
|
||||
Invocation.method(
|
||||
#deleteWarehouseLocation,
|
||||
[id],
|
||||
),
|
||||
returnValue: _i4.Future<void>.value(),
|
||||
returnValueForMissingStub: _i4.Future<void>.value(),
|
||||
) as _i4.Future<void>);
|
||||
|
||||
@override
|
||||
_i4.Future<bool> checkWarehouseHasEquipment(int? id) => (super.noSuchMethod(
|
||||
Invocation.method(
|
||||
#checkWarehouseHasEquipment,
|
||||
[id],
|
||||
),
|
||||
returnValue: _i4.Future<bool>.value(false),
|
||||
) as _i4.Future<bool>);
|
||||
}
|
||||
Reference in New Issue
Block a user