feat!: improve error system

Change string by ApklisErrorModel with
statusCode: int and statusMessage: String

Author: leynier

packages/apklis_dio_web_api/lib/apklis_dio_web_api.dart

@@ -2,6 +2,7 @@ library apklis_dio_web_api;
 
 import 'package:apklis_web_api/apklis_api_result.dart';
 import 'package:apklis_web_api/apklis_web_api.dart';
+import 'package:apklis_web_api/models/apklis_error_model.dart';
 import 'package:apklis_web_api/models/models.dart';
 import 'package:dio/dio.dart';
 
@@ -20,10 +21,16 @@ class ApklisDioWebApi extends ApklisWebApi {
         final model = ApklisModel.fromJson(json);
         return ApklisApiResult.success(model);
       } else {
-        return ApklisApiResult.failure(response.statusMessage!);
+        return ApklisApiResult.failure(ApklisErrorModel(
+          statusCode: response.statusCode!,
+          statusMessage: response.statusMessage!,
+        ));
       }
     } catch (e) {
-      return ApklisApiResult.failure(e.toString());
+      return ApklisApiResult.failure(ApklisErrorModel(
+        statusCode: -1,
+        statusMessage: e.toString(),
+      ));
     }
   }
 }

packages/apklis_dio_web_api/test/apklis_dio_web_api_test.dart

@@ -68,7 +68,8 @@ void main() {
         throw Exception('Result should be failure.');
       },
       failure: (error) {
-        expect(error.isNotEmpty, true);
+        expect(error.statusCode, isNot(200));
+        expect(error.statusMessage.isNotEmpty, true);
       },
     );
   });
@@ -97,7 +98,8 @@ void main() {
         throw Exception('Result should be failure.');
       },
       failure: (error) {
-        expect(error.isNotEmpty, true);
+        expect(error.statusCode, isNot(200));
+        expect(error.statusMessage.isNotEmpty, true);
       },
     );
   });

packages/apklis_http_web_api/lib/apklis_http_web_api.dart

@@ -4,6 +4,7 @@ import 'dart:convert';
 
 import 'package:apklis_web_api/apklis_api_result.dart';
 import 'package:apklis_web_api/apklis_web_api.dart';
+import 'package:apklis_web_api/models/apklis_error_model.dart';
 import 'package:apklis_web_api/models/models.dart';
 import 'package:http/http.dart';
 
@@ -28,10 +29,16 @@ class ApklisHttpWebApi extends ApklisWebApi {
         final model = ApklisModel.fromJson(json);
         return ApklisApiResult.success(model);
       } else {
-        return ApklisApiResult.failure(response.reasonPhrase!);
+        return ApklisApiResult.failure(ApklisErrorModel(
+          statusCode: response.statusCode,
+          statusMessage: response.reasonPhrase!,
+        ));
       }
     } catch (e) {
-      return ApklisApiResult.failure(e.toString());
+      return ApklisApiResult.failure(ApklisErrorModel(
+        statusCode: -1,
+        statusMessage: e.toString(),
+      ));
     }
   }
 }

packages/apklis_http_web_api/test/apklis_http_web_api_test.dart

@@ -45,7 +45,8 @@ void main() {
         throw Exception('Result should be failure.');
       },
       failure: (error) {
-        expect(error.isNotEmpty, true);
+        expect(error.statusCode, isNot(200));
+        expect(error.statusMessage.isNotEmpty, true);
       },
     );
   });
@@ -64,7 +65,8 @@ void main() {
         throw Exception('Result should be failure.');
       },
       failure: (error) {
-        expect(error.isNotEmpty, true);
+        expect(error.statusCode, isNot(200));
+        expect(error.statusMessage.isNotEmpty, true);
       },
     );
   });

packages/apklis_web_api/lib/apklis_api_result.dart

@@ -1,3 +1,4 @@
+import 'package:apklis_web_api/models/apklis_error_model.dart';
 import 'package:apklis_web_api/models/models.dart';
 import 'package:freezed_annotation/freezed_annotation.dart';
 
@@ -7,5 +8,6 @@ part 'apklis_api_result.freezed.dart';
 class ApklisApiResult with _$ApklisApiResult {
   const factory ApklisApiResult.success(ApklisModel result) =
       _SuccessApklisApiResult;
-  const factory ApklisApiResult.failure(String error) = _FailureApklisApiResult;
+  const factory ApklisApiResult.failure(ApklisErrorModel error) =
+      _FailureApklisApiResult;
 }

packages/apklis_web_api/lib/apklis_api_result.freezed.dart

@@ -0,0 +1,21 @@
+import 'package:json_annotation/json_annotation.dart';
+
+part 'apklis_error_model.g.dart';
+
+@JsonSerializable(explicitToJson: true)
+class ApklisErrorModel {
+  const ApklisErrorModel({
+    required this.statusCode,
+    required this.statusMessage,
+  });
+
+  @JsonKey(name: 'status_code')
+  final int statusCode;
+  @JsonKey(name: 'status_message')
+  final String statusMessage;
+
+  static ApklisErrorModel fromJson(Map<String, dynamic> json) =>
+      _$ApklisErrorModelFromJson(json);
+
+  Map<String, dynamic> toJson() => _$ApklisErrorModelToJson(this);
+}