refactor: 프로젝트 구조 개선 및 테스트 시스템 강화
주요 변경사항: - CLAUDE.md: 프로젝트 규칙 v2.0으로 업데이트, 아키텍처 명확화 - 불필요한 문서 제거: NEXT_TASKS.md, TEST_PROGRESS.md, test_results 파일들 - 테스트 시스템 개선: 실제 API 테스트 스위트 추가 (15개 새 테스트 파일) - License 관리: DTO 모델 개선, API 응답 처리 최적화 - 에러 처리: Interceptor 로직 강화, 상세 로깅 추가 - Company/User/Warehouse 테스트: 자동화 테스트 안정성 향상 - Phone Utils: 전화번호 포맷팅 로직 개선 - Overview Controller: 대시보드 데이터 로딩 최적화 - Analysis Options: Flutter 린트 규칙 추가 테스트 개선: - company_real_api_test.dart: 실제 API 회사 관리 테스트 - equipment_in/out_real_api_test.dart: 장비 입출고 API 테스트 - license_real_api_test.dart: 라이선스 관리 API 테스트 - user_real_api_test.dart: 사용자 관리 API 테스트 - warehouse_location_real_api_test.dart: 창고 위치 API 테스트 - filter_sort_test.dart: 필터링/정렬 기능 테스트 - pagination_test.dart: 페이지네이션 테스트 - interactive_search_test.dart: 검색 기능 테스트 - overview_dashboard_test.dart: 대시보드 통합 테스트 코드 품질: - 모든 서비스에 에러 처리 강화 - DTO 모델 null safety 개선 - 테스트 커버리지 확대 - 불필요한 로그 파일 제거로 리포지토리 정리 Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
@@ -1,4 +1,5 @@
|
||||
import 'package:dio/dio.dart';
|
||||
import 'package:flutter/foundation.dart';
|
||||
import 'package:injectable/injectable.dart';
|
||||
import 'package:superport/core/constants/api_endpoints.dart';
|
||||
import 'package:superport/core/errors/exceptions.dart';
|
||||
@@ -116,26 +117,42 @@ class CompanyRemoteDataSourceImpl implements CompanyRemoteDataSource {
|
||||
@override
|
||||
Future<CompanyResponse> createCompany(CreateCompanyRequest request) async {
|
||||
try {
|
||||
debugPrint('[CompanyRemoteDataSource] Sending POST request to ${ApiEndpoints.companies}');
|
||||
debugPrint('[CompanyRemoteDataSource] Request data: ${request.toJson()}');
|
||||
|
||||
final response = await _apiClient.post(
|
||||
ApiEndpoints.companies,
|
||||
data: request.toJson(),
|
||||
);
|
||||
|
||||
if (response.statusCode == 201) {
|
||||
final apiResponse = ApiResponse<CompanyResponse>.fromJson(
|
||||
response.data,
|
||||
(json) => CompanyResponse.fromJson(json as Map<String, dynamic>),
|
||||
);
|
||||
return apiResponse.data!;
|
||||
debugPrint('[CompanyRemoteDataSource] Response status: ${response.statusCode}');
|
||||
debugPrint('[CompanyRemoteDataSource] Response data: ${response.data}');
|
||||
|
||||
if (response.statusCode == 201 || response.statusCode == 200) {
|
||||
// API 응답 구조 확인
|
||||
final responseData = response.data;
|
||||
if (responseData != null && responseData['success'] == true && responseData['data'] != null) {
|
||||
// 직접 파싱
|
||||
return CompanyResponse.fromJson(responseData['data'] as Map<String, dynamic>);
|
||||
} else {
|
||||
// ApiResponse 형식으로 파싱 시도
|
||||
final apiResponse = ApiResponse<CompanyResponse>.fromJson(
|
||||
response.data,
|
||||
(json) => CompanyResponse.fromJson(json as Map<String, dynamic>),
|
||||
);
|
||||
return apiResponse.data!;
|
||||
}
|
||||
} else {
|
||||
throw ApiException(
|
||||
message: 'Failed to create company',
|
||||
message: 'Failed to create company - Status: ${response.statusCode}',
|
||||
statusCode: response.statusCode,
|
||||
);
|
||||
}
|
||||
} catch (e) {
|
||||
} catch (e, stackTrace) {
|
||||
debugPrint('[CompanyRemoteDataSource] Error creating company: $e');
|
||||
debugPrint('[CompanyRemoteDataSource] Stack trace: $stackTrace');
|
||||
if (e is ApiException) rethrow;
|
||||
throw ApiException(message: e.toString());
|
||||
throw ApiException(message: 'Error creating company: $e');
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -18,6 +18,7 @@ abstract class EquipmentRemoteDataSource {
|
||||
String? status,
|
||||
int? companyId,
|
||||
int? warehouseLocationId,
|
||||
String? search,
|
||||
});
|
||||
|
||||
Future<EquipmentResponse> createEquipment(CreateEquipmentRequest request);
|
||||
@@ -49,6 +50,7 @@ class EquipmentRemoteDataSourceImpl implements EquipmentRemoteDataSource {
|
||||
String? status,
|
||||
int? companyId,
|
||||
int? warehouseLocationId,
|
||||
String? search,
|
||||
}) async {
|
||||
try {
|
||||
final queryParams = {
|
||||
@@ -57,6 +59,7 @@ class EquipmentRemoteDataSourceImpl implements EquipmentRemoteDataSource {
|
||||
if (status != null) 'status': status,
|
||||
if (companyId != null) 'company_id': companyId,
|
||||
if (warehouseLocationId != null) 'warehouse_location_id': warehouseLocationId,
|
||||
if (search != null && search.isNotEmpty) 'search': search,
|
||||
};
|
||||
|
||||
final response = await _apiClient.get(
|
||||
|
||||
@@ -1,5 +1,7 @@
|
||||
import 'dart:io';
|
||||
|
||||
import 'package:dio/dio.dart';
|
||||
import 'package:flutter/foundation.dart';
|
||||
import '../../../../core/errors/exceptions.dart';
|
||||
import '../../../../core/constants/app_constants.dart';
|
||||
|
||||
@@ -69,9 +71,9 @@ class ErrorInterceptor extends Interceptor {
|
||||
|
||||
if (errorMessage.contains('cors') || errorString.contains('cors') ||
|
||||
errorMessage.contains('xmlhttprequest') || errorString.contains('xmlhttprequest')) {
|
||||
print('[ErrorInterceptor] CORS 에러 감지됨');
|
||||
print('[ErrorInterceptor] 요청 URL: ${err.requestOptions.uri}');
|
||||
print('[ErrorInterceptor] 에러 메시지: ${err.message}');
|
||||
debugPrint('[ErrorInterceptor] CORS 에러 감지됨');
|
||||
debugPrint('[ErrorInterceptor] 요청 URL: ${err.requestOptions.uri}');
|
||||
debugPrint('[ErrorInterceptor] 에러 메시지: ${err.message}');
|
||||
|
||||
handler.reject(
|
||||
DioException(
|
||||
@@ -84,10 +86,10 @@ class ErrorInterceptor extends Interceptor {
|
||||
),
|
||||
);
|
||||
} else {
|
||||
print('[ErrorInterceptor] 알 수 없는 에러');
|
||||
print('[ErrorInterceptor] 에러 타입: ${err.error?.runtimeType}');
|
||||
print('[ErrorInterceptor] 에러 메시지: ${err.message}');
|
||||
print('[ErrorInterceptor] 에러 내용: ${err.error}');
|
||||
debugPrint('[ErrorInterceptor] 알 수 없는 에러');
|
||||
debugPrint('[ErrorInterceptor] 에러 타입: ${err.error?.runtimeType}');
|
||||
debugPrint('[ErrorInterceptor] 에러 메시지: ${err.message}');
|
||||
debugPrint('[ErrorInterceptor] 에러 내용: ${err.error}');
|
||||
|
||||
handler.reject(
|
||||
DioException(
|
||||
|
||||
@@ -1,4 +1,5 @@
|
||||
import 'package:dio/dio.dart';
|
||||
import 'package:flutter/foundation.dart';
|
||||
|
||||
/// API 응답을 정규화하는 인터셉터
|
||||
///
|
||||
@@ -6,16 +7,16 @@ import 'package:dio/dio.dart';
|
||||
class ResponseInterceptor extends Interceptor {
|
||||
@override
|
||||
void onResponse(Response response, ResponseInterceptorHandler handler) {
|
||||
print('[ResponseInterceptor] 응답 수신: ${response.requestOptions.path}');
|
||||
print('[ResponseInterceptor] 상태 코드: ${response.statusCode}');
|
||||
print('[ResponseInterceptor] 응답 데이터 타입: ${response.data.runtimeType}');
|
||||
debugPrint('[ResponseInterceptor] 응답 수신: ${response.requestOptions.path}');
|
||||
debugPrint('[ResponseInterceptor] 상태 코드: ${response.statusCode}');
|
||||
debugPrint('[ResponseInterceptor] 응답 데이터 타입: ${response.data.runtimeType}');
|
||||
|
||||
// 장비 관련 API 응답 상세 로깅
|
||||
if (response.requestOptions.path.contains('equipment')) {
|
||||
print('[ResponseInterceptor] 장비 API 응답 전체: ${response.data}');
|
||||
debugPrint('[ResponseInterceptor] 장비 API 응답 전체: ${response.data}');
|
||||
if (response.data is List && (response.data as List).isNotEmpty) {
|
||||
final firstItem = (response.data as List).first;
|
||||
print('[ResponseInterceptor] 첫 번째 장비 상태: ${firstItem['status']}');
|
||||
debugPrint('[ResponseInterceptor] 첫 번째 장비 상태: ${firstItem['status']}');
|
||||
}
|
||||
}
|
||||
|
||||
@@ -27,7 +28,7 @@ class ResponseInterceptor extends Interceptor {
|
||||
|
||||
// 이미 정규화된 형식인지 확인
|
||||
if (data.containsKey('success') && data.containsKey('data')) {
|
||||
print('[ResponseInterceptor] 이미 정규화된 응답 형식');
|
||||
debugPrint('[ResponseInterceptor] 이미 정규화된 응답 형식');
|
||||
handler.next(response);
|
||||
return;
|
||||
}
|
||||
@@ -35,7 +36,7 @@ class ResponseInterceptor extends Interceptor {
|
||||
// API 응답이 직접 데이터를 반환하는 경우
|
||||
// (예: {accessToken: "...", refreshToken: "...", user: {...}})
|
||||
if (_isDirectDataResponse(data)) {
|
||||
print('[ResponseInterceptor] 직접 데이터 응답을 정규화된 형식으로 변환');
|
||||
debugPrint('[ResponseInterceptor] 직접 데이터 응답을 정규화된 형식으로 변환');
|
||||
|
||||
// 정규화된 응답으로 변환
|
||||
response.data = {
|
||||
|
||||
@@ -1,3 +1,4 @@
|
||||
import 'package:flutter/foundation.dart';
|
||||
import 'package:injectable/injectable.dart';
|
||||
import 'package:superport/core/constants/api_endpoints.dart';
|
||||
import 'package:superport/core/errors/exceptions.dart';
|
||||
@@ -63,7 +64,53 @@ class LicenseRemoteDataSourceImpl implements LicenseRemoteDataSource {
|
||||
);
|
||||
|
||||
if (response.data != null && response.data['success'] == true && response.data['data'] != null) {
|
||||
return LicenseListResponseDto.fromJson(response.data['data']);
|
||||
// API 응답이 배열인 경우와 객체인 경우를 모두 처리
|
||||
final data = response.data['data'];
|
||||
if (data is List) {
|
||||
// 배열 응답을 LicenseListResponseDto 형식으로 변환
|
||||
final List<LicenseDto> licenses = [];
|
||||
|
||||
for (int i = 0; i < data.length; i++) {
|
||||
try {
|
||||
final item = data[i];
|
||||
debugPrint('📑 Parsing license item $i: ${item['license_key']}');
|
||||
|
||||
// null 검사 및 기본값 설정
|
||||
final licenseDto = LicenseDto.fromJson({
|
||||
...item,
|
||||
// 필수 필드 보장
|
||||
'license_key': item['license_key'] ?? '',
|
||||
'is_active': item['is_active'] ?? true,
|
||||
'created_at': item['created_at'] ?? DateTime.now().toIso8601String(),
|
||||
'updated_at': item['updated_at'] ?? DateTime.now().toIso8601String(),
|
||||
});
|
||||
licenses.add(licenseDto);
|
||||
} catch (e, stackTrace) {
|
||||
debugPrint('❌ Error parsing license item $i: $e');
|
||||
debugPrint('Item data: ${data[i]}');
|
||||
debugPrint('Stack trace: $stackTrace');
|
||||
// 파싱 실패한 항목은 건너뛰고 계속
|
||||
continue;
|
||||
}
|
||||
}
|
||||
|
||||
final pagination = response.data['pagination'] ?? {};
|
||||
return LicenseListResponseDto(
|
||||
items: licenses,
|
||||
total: pagination['total'] ?? licenses.length,
|
||||
page: pagination['page'] ?? page,
|
||||
perPage: pagination['per_page'] ?? perPage,
|
||||
totalPages: pagination['total_pages'] ?? 1,
|
||||
);
|
||||
} else if (data['items'] != null) {
|
||||
// 이미 LicenseListResponseDto 형식인 경우
|
||||
return LicenseListResponseDto.fromJson(data);
|
||||
} else {
|
||||
// 예상치 못한 형식인 경우
|
||||
throw ApiException(
|
||||
message: 'Unexpected response format for license list',
|
||||
);
|
||||
}
|
||||
} else {
|
||||
throw ApiException(
|
||||
message: response.data?['error']?['message'] ?? 'Failed to fetch licenses',
|
||||
@@ -202,7 +249,35 @@ class LicenseRemoteDataSourceImpl implements LicenseRemoteDataSource {
|
||||
);
|
||||
|
||||
if (response.data != null && response.data['success'] == true && response.data['data'] != null) {
|
||||
return ExpiringLicenseListDto.fromJson(response.data['data']);
|
||||
// API 응답이 배열 형태인 경우 처리
|
||||
final data = response.data['data'];
|
||||
final pagination = response.data['pagination'] ?? {};
|
||||
|
||||
if (data is List) {
|
||||
// 배열 응답을 ExpiringLicenseListDto 형식으로 변환
|
||||
final List<ExpiringLicenseDto> licenses = [];
|
||||
|
||||
for (var item in data) {
|
||||
try {
|
||||
licenses.add(ExpiringLicenseDto.fromJson(item));
|
||||
} catch (e) {
|
||||
debugPrint('❌ Error parsing expiring license: $e');
|
||||
debugPrint('Item data: $item');
|
||||
continue;
|
||||
}
|
||||
}
|
||||
|
||||
return ExpiringLicenseListDto(
|
||||
items: licenses,
|
||||
total: pagination['total'] ?? licenses.length,
|
||||
page: pagination['page'] ?? page,
|
||||
perPage: pagination['per_page'] ?? perPage,
|
||||
totalPages: pagination['total_pages'] ?? 1,
|
||||
);
|
||||
} else {
|
||||
// 이미 올바른 형식인 경우
|
||||
return ExpiringLicenseListDto.fromJson(data);
|
||||
}
|
||||
} else {
|
||||
throw ApiException(
|
||||
message: response.data?['error']?['message'] ?? 'Failed to fetch expiring licenses',
|
||||
|
||||
@@ -33,7 +33,34 @@ class UserRemoteDataSource {
|
||||
);
|
||||
|
||||
if (response.data != null && response.data['success'] == true && response.data['data'] != null) {
|
||||
return UserListDto.fromJson(response.data['data']);
|
||||
// API 응답이 배열인 경우와 객체인 경우를 모두 처리
|
||||
final data = response.data['data'];
|
||||
if (data is List) {
|
||||
// 배열 응답을 UserListDto 형식으로 변환
|
||||
// role이 null인 경우 기본값 설정
|
||||
final users = data.map((json) {
|
||||
if (json['role'] == null) {
|
||||
json['role'] = 'staff'; // 기본값
|
||||
}
|
||||
return UserDto.fromJson(json);
|
||||
}).toList();
|
||||
final pagination = response.data['pagination'] ?? {};
|
||||
return UserListDto(
|
||||
users: users,
|
||||
total: pagination['total'] ?? users.length,
|
||||
page: pagination['page'] ?? page,
|
||||
perPage: pagination['per_page'] ?? perPage,
|
||||
totalPages: pagination['total_pages'] ?? 1,
|
||||
);
|
||||
} else if (data['users'] != null) {
|
||||
// 이미 UserListDto 형식인 경우
|
||||
return UserListDto.fromJson(data);
|
||||
} else {
|
||||
// 예상치 못한 형식인 경우
|
||||
throw ApiException(
|
||||
message: 'Unexpected response format for user list',
|
||||
);
|
||||
}
|
||||
} else {
|
||||
throw ApiException(
|
||||
message: response.data?['error']?['message'] ?? '사용자 목록을 불러오는데 실패했습니다',
|
||||
|
||||
Reference in New Issue
Block a user