🎨 Format file and fix lins

Author: Guldem

lib/logger/bolt_logger.dart

@@ -21,6 +21,8 @@ class BoltLogger {
 
   static final _instance = BoltLogger._();
 
+  // Is cancelled in discharge
+  //ignore: cancel_subscriptions
   StreamSubscription<LogRecord>? _subscription;
 
   final Map<String, BoltCharge> _outputs = {};

lib/logger/charges/bolt_charge.dart

@@ -6,9 +6,9 @@ export 'package:dcc_toolkit/logger/charges/memory_charge.dart';
 
 /// Interface for creating [BoltCharge]s to power the [BoltLogger]
 abstract interface class BoltCharge {
-
   /// The name of the charge.
   abstract final String name;
+
   /// Log the output of the [ZapEvent].
   void logOutput(ZapEvent event);
 

lib/logger/charges/debug_console_charge.dart

@@ -27,7 +27,9 @@ class DebugConsoleCharge implements BoltCharge {
             event.origin.stackTrace != null ||
             event.origin.error != null);
 
-    return shouldPaint ? event.lines.map((line) => '$_red$line$_reset').toList() : event.lines;
+    return shouldPaint
+        ? event.lines.map((line) => '$_red$line$_reset').toList()
+        : event.lines;
   }
 
   @override

lib/logger/charges/file_charge.dart

@@ -13,21 +13,23 @@ import 'package:intl/intl.dart';
 ///
 /// {@endtemplate}
 class FileCharge implements BoltCharge {
-
   /// {@macro file_charge}
-  FileCharge(this.path, {this.bufferSize = 1000, this.writeDelay = const Duration(seconds: 5)}) {
+  FileCharge(this.path,
+      {this.bufferSize = 1000, this.writeDelay = const Duration(seconds: 5),}) {
     final fileName = '${DateFormat('yyyy-MM-dd').format(DateTime.now())}.log';
     _file = File('$path/$fileName');
 
     Timer.periodic(writeDelay, (_) => _flush());
   }
   @override
-   String get name => 'FileCharge';
+  String get name => 'FileCharge';
 
   /// The size of the buffer (in lines) before writing to the file.
   final int bufferSize;
+
   /// The path to the directory where the log files will be written.
   final String path;
+
   /// The delay between writing to the file.
   final Duration writeDelay;
   File? _file;

lib/logger/charges/memory_charge.dart

@@ -16,7 +16,7 @@ class MemoryCharge implements BoltCharge {
   MemoryCharge({this.maxItems = 1000});
 
   @override
-   String get name => 'MemoryCharge';
+  String get name => 'MemoryCharge';
 
   /// The maximum amount of logs to store in memory.
   final int maxItems;
@@ -41,7 +41,7 @@ class MemoryCharge implements BoltCharge {
 
   @override
   void discharge() {
-   _controller.close();
-   _items.clear();
+    _controller.close();
+    _items.clear();
   }
 }

lib/logger/extensions/stacktrace.dart

@@ -1,10 +1,11 @@
 /// StackTrace extension methods.
 extension ZapStackTraceExtension on StackTrace {
-
   /// Strike the [StackTrace] which will format it so that it's easier to read.
   String get strike => toString()
       .split('\n')
-      .map((e) => e.isNotEmpty && e.startsWith('#') ? e.replaceAll(RegExp(r'\s\s+'), ' ') : null)
+      .map((e) => e.isNotEmpty && e.startsWith('#')
+          ? e.replaceAll(RegExp(r'\s\s+'), ' ')
+          : null,)
       .whereType<String>()
       .join('\n');
 }

lib/logger/extensions/zap_extension.dart

@@ -15,10 +15,10 @@ extension ZapExtension on Object {
 
   /// {@macro shock}
   void shock(
-      Object? message, {
-        String? tag,
-        Level level = Level.SEVERE,
-      }) {
+    Object? message, {
+    String? tag,
+    Level level = Level.SEVERE,
+  }) {
     //ignore: no_runtimeType_toString
     BoltLogger.shock(message, tag: tag ?? runtimeType.toString(), level: level);
   }

lib/logger/log_record_formatter.dart

@@ -25,7 +25,7 @@ class LogRecordFormatter {
   }
 
   List<String> _formatMessage(LogRecord record) {
-    if (record.message == 'null'|| record.message.trim().isEmpty) return [];
+    if (record.message == 'null' || record.message.trim().isEmpty) return [];
 
     return record.message.split('\n');
   }
@@ -36,7 +36,8 @@ class LogRecordFormatter {
     return record.error.toString().split('\n').map((e) => e.trim()).toList();
   }
 
-  List<String> _formatStacktrace(LogRecord record) => record.stackTrace?.strike.split('\n') ?? [];
+  List<String> _formatStacktrace(LogRecord record) =>
+      record.stackTrace?.strike.split('\n') ?? [];
 
   String _prefix(LogRecord record) {
     final tag = record.loggerName;

lib/logger/zap_event.dart

@@ -1,5 +1,6 @@
 import 'package:dcc_toolkit/logger/log_record_formatter.dart';
 import 'package:logging/logging.dart';
+
 /// {@template zap_event}
 /// A zap event that contains the [lines] of a log record and the [origin] [LogRecord].
 /// {@endtemplate}

test/logger/bolt_logger_test.dart

@@ -1,3 +1,5 @@
+// Mocktail doesn't work properly with tear-offs
+//ignore_for_file: unnecessary_lambdas
 import 'package:dcc_toolkit/dcc_toolkit.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:logging/logging.dart';
@@ -10,7 +12,8 @@ void main() {
   tearDown(() => BoltLogger.discharge());
 
   group('charge tests', () {
-    test('BoltLogger without a charge only redirect log event to root Logger', () {
+    test('BoltLogger without a charge only redirect log event to root Logger',
+        () {
       final records = <LogRecord>[];
       Logger.root.level = Level.ALL;
       final sub = Logger.root.onRecord.listen(records.add);
@@ -71,7 +74,8 @@ void main() {
       BoltLogger.charge([memoryCharge]);
     });
 
-    parameterizedGroup1('BoltLogger/extension function tests', [true, false], (bool useBoltLogger) {
+    parameterizedGroup1('BoltLogger/extension function tests', [true, false],
+        (bool useBoltLogger) {
       test('zap/shock sends a message via a ZapEvent to a charge', () {
         if (useBoltLogger) {
           BoltLogger.zap('zap');
@@ -83,9 +87,11 @@ void main() {
 
         expect(memoryCharge.items.length, 2);
         expect(memoryCharge.items[0].origin.message, 'zap');
-        expect(memoryCharge.items[0].origin.loggerName, useBoltLogger ? 'BoltLogger' : 'TestReferenceClass');
+        expect(memoryCharge.items[0].origin.loggerName,
+            useBoltLogger ? 'BoltLogger' : 'TestReferenceClass',);
         expect(memoryCharge.items[1].origin.message, 'shock');
-        expect(memoryCharge.items[1].origin.loggerName, useBoltLogger ? 'BoltLogger' : 'TestReferenceClass');
+        expect(memoryCharge.items[1].origin.loggerName,
+            useBoltLogger ? 'BoltLogger' : 'TestReferenceClass',);
         expect(memoryCharge.items[1].origin.level, Level.SEVERE);
       });
 
@@ -104,7 +110,8 @@ void main() {
         expect(memoryCharge.items[1].origin.level, Level.SEVERE);
       });
 
-      parameterizedTest1('zap/shock sends a LogLevel via a ZapEvent to a Charge', [
+      parameterizedTest1(
+          'zap/shock sends a LogLevel via a ZapEvent to a Charge', [
         Level.ALL,
         Level.CONFIG,
         Level.FINE,
@@ -223,25 +230,38 @@ void main() {
         expect(memoryCharge.items[1].origin.stackTrace, stackTrace);
       });
 
-      test('zap/shock sends a List with more than 3 elements via a ZapEvent to a Charge', () {
+      test(
+          'zap/shock sends a List with more than 3 elements via a ZapEvent to a Charge',
+          () {
         final exception = Exception('exception');
         final stackTrace = StackTrace.current;
         final error = Error();
         if (useBoltLogger) {
-          expect(() => BoltLogger.zap(['zap', exception, stackTrace, error]), throwsAssertionError);
-          expect(() => BoltLogger.shock(['shock', exception, stackTrace, error]), throwsAssertionError);
+          expect(() => BoltLogger.zap(['zap', exception, stackTrace, error]),
+              throwsAssertionError,);
+          expect(
+              () => BoltLogger.shock(['shock', exception, stackTrace, error]),
+              throwsAssertionError,);
         } else {
-          expect(() => TestReferenceClass().zapExtension(['zap', exception, stackTrace, error]), throwsAssertionError);
-          expect(() => TestReferenceClass().shockExtension(['shock', exception, stackTrace, error]), throwsAssertionError);
+          expect(
+              () => TestReferenceClass()
+                  .zapExtension(['zap', exception, stackTrace, error]),
+              throwsAssertionError,);
+          expect(
+              () => TestReferenceClass()
+                  .shockExtension(['shock', exception, stackTrace, error]),
+              throwsAssertionError,);
         }
       });
     });
   });
 }
 
 class TestReferenceClass {
-  void zapExtension(Object? message, {String? tag, Level level = Level.INFO}) => zap(message, tag: tag, level: level);
+  void zapExtension(Object? message, {String? tag, Level level = Level.INFO}) =>
+      zap(message, tag: tag, level: level);
 
-  void shockExtension(Object? message, {String? tag, Level level = Level.SEVERE}) =>
+  void shockExtension(Object? message,
+          {String? tag, Level level = Level.SEVERE,}) =>
       shock(message, tag: tag, level: level);
 }