🎨 Fixed minor issues

Author: Guldem

lib/chopper/json_serializable_converter.dart

@@ -21,18 +21,23 @@ class JsonSerializableConverter extends JsonConverter {
   ///  User: User.fromJson,
   ///  NoContent: NoContent.fromJson,
   ///  });
+  ///  ```
   final Map<Type, JsonFactory<dynamic>> factories;
 
   T _decodeMap<T>(Map<String, dynamic> values) {
     final jsonFactory = factories[T];
     if (jsonFactory == null) {
-      throw JsonUnsupportedObjectError(T,
-          cause:
-              'No fromJson was registered for JsonSerializableConverter for $T');
+      throw JsonUnsupportedObjectError(
+        T,
+        cause:
+            'No fromJson was registered for JsonSerializableConverter for $T',
+      );
     }
     if (jsonFactory is! JsonFactory<T>) {
-      throw JsonUnsupportedObjectError(T,
-          cause: 'fromJson type does not match $T');
+      throw JsonUnsupportedObjectError(
+        T,
+        cause: 'fromJson type does not match $T',
+      );
     }
 
     return jsonFactory(values);

lib/common/dcc_logger.dart

@@ -32,8 +32,11 @@ class DCCLogger {
         : record.message;
 
     debugPrint(_formatLogPrint(record, message));
-    if (error != null)
+
+    if (error != null) {
       debugPrint(_formatLogPrint(record, '${error.runtimeType}: $error'));
+    }
+
     if (stacktrace != null) {
       final lines =
           stacktrace.toString().split('\n').where((e) => e.isNotEmpty);

lib/common/result/result.dart

@@ -1,4 +1,4 @@
-import 'package:flutter/cupertino.dart';
+import 'package:flutter/foundation.dart';
 
 /// Result class to handle success, failure and loading states.
 @immutable

pubspec.yaml

@@ -21,8 +21,8 @@ dependencies:
 dev_dependencies:
   flutter_test:
     sdk: flutter
-  mocktail: ^1.0.2
   http: ^1.1.2
+  mocktail: ^1.0.2
   parameterized_test: ^1.1.1
   very_good_analysis: ^5.1.0
 

test/chopper/json_serializable_converter_test.dart

@@ -13,8 +13,10 @@ void main() {
 
     const converter = JsonSerializableConverter({});
 
-    expect(() => converter.convertResponse<TestModel, TestModel>(response),
-        throwsA(isA<JsonUnsupportedObjectError>()));
+    expect(
+      () => converter.convertResponse<TestModel, TestModel>(response),
+      throwsA(isA<JsonUnsupportedObjectError>()),
+    );
   });
 
   test('Error is thrown when wrong factory is added', () {
@@ -23,8 +25,10 @@ void main() {
     const converter =
         JsonSerializableConverter({TestModel: TestModel2.fromJson});
 
-    expect(() => converter.convertResponse<TestModel, TestModel>(response),
-        throwsA(isA<JsonUnsupportedObjectError>()));
+    expect(
+      () => converter.convertResponse<TestModel, TestModel>(response),
+      throwsA(isA<JsonUnsupportedObjectError>()),
+    );
   });
 
   test('Uses model to convert json into model', () async {

test/dcc_logger_test.dart

@@ -9,9 +9,13 @@ void main() {
   group('uninitialised tests', () {
     test('throws assertion error when logger is not initialised', () {
       expect(
-          () => DCCLogger.info('test message'), throwsA(isA<AssertionError>()));
-      expect(() => DCCLogger.severe('test message'),
-          throwsA(isA<AssertionError>()));
+        () => DCCLogger.info('test message'),
+        throwsA(isA<AssertionError>()),
+      );
+      expect(
+        () => DCCLogger.severe('test message'),
+        throwsA(isA<AssertionError>()),
+      );
     });
   });