EmailJSResponseStatus refactoring

Author: Sergey Khomushin

lib/emailjs.dart

@@ -6,7 +6,10 @@ import 'package:http/http.dart' as http;
 import 'src/models/emailjs_response_status.dart';
 import 'src/utils/validate_params.dart';
 import 'src/api/send_json.dart';
-import 'src/types/options.dart';
+import 'src/models/options.dart';
+
+export 'src/models/emailjs_response_status.dart';
+export 'src/models/options.dart';
 
 class EmailJS {
   /// Public Key specified in the [init] method

lib/src/api/send_json.dart

@@ -14,9 +14,15 @@ Future<EmailJSResponseStatus> sendJSON(
     );
 
     if (response.statusCode == 200) {
-      return EmailJSResponseStatus(response.statusCode, response.body);
+      return EmailJSResponseStatus(
+        status: response.statusCode,
+        text: response.body,
+      );
     } else {
-      throw EmailJSResponseStatus(response.statusCode, response.body);
+      throw EmailJSResponseStatus(
+        status: response.statusCode,
+        text: response.body,
+      );
     }
   } finally {
     client.close();

lib/src/models/emailjs_response_status.dart

@@ -1,10 +1,13 @@
 /// Response status from EmailJS API
 class EmailJSResponseStatus {
-  int status = 0;
-  String text = 'Network Error';
+  /// Response status code
+  final int status;
 
-  EmailJSResponseStatus(int? status, String? text) {
-    this.status = status ?? 0;
-    this.text = text ?? 'Network Error';
-  }
+  /// Response text
+  final String text;
+
+  const EmailJSResponseStatus({
+    this.status = 0,
+    this.text = 'Network Error',
+  });
 }

lib/src/types/options.dart renamed to lib/src/models/options.dart

@@ -2,8 +2,6 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:mocktail/mocktail.dart';
 import 'package:http/http.dart' as http;
 
-import 'package:emailjs/src/models/emailjs_response_status.dart';
-import 'package:emailjs/src/types/options.dart';
 import 'package:emailjs/emailjs.dart';
 
 class MockClient extends Mock implements http.Client {}