Replaced printTime with dateTimeFormat

SourceHorizon · Aug 19, 2024 · ca1832e · ca1832e
1 parent 5281ed0
commit ca1832e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/lib/src/printers/simple_printer.dart b/lib/src/printers/simple_printer.dart
@@ -1,6 +1,7 @@
 import 'dart:convert';
 
 import '../ansi_color.dart';
+import '../date_time_format.dart';
 import '../log_event.dart';
 import '../log_level.dart';
 import '../log_printer.dart';
@@ -28,16 +29,18 @@ class SimplePrinter extends LogPrinter {
     Level.fatal: const AnsiColor.fg(199),
   };
 
-  final bool printTime;
   final bool colors;
 
-  SimplePrinter({this.printTime = false, this.colors = true});
+  SimplePrinter({
+    super.dateTimeFormat = DateTimeFormat.none,
+    this.colors = true,
+  });
 
   @override
   List<String> log(LogEvent event) {
     var messageStr = stringifyMessage(event.message);
     var errorStr = event.error != null ? '  ERROR: ${event.error}' : '';
-    var timeStr = printTime ? 'TIME: ${event.time.toIso8601String()}' : '';
+    var timeStr = printTimestamp ? 'TIME: ${getTime(event.time)}' : '';
     return ['${_labelFor(event.level)} $timeStr $messageStr$errorStr'];
   }
 

diff --git a/test/printers/simple_printer_test.dart b/test/printers/simple_printer_test.dart
@@ -11,7 +11,8 @@ void main() {
     stackTrace: StackTrace.current,
   );
 
-  var plainPrinter = SimplePrinter(colors: false, printTime: false);
+  var plainPrinter =
+      SimplePrinter(colors: false, dateTimeFormat: DateTimeFormat.none);
 
   test('represent event on a single line (ignoring stacktrace)', () {
     var outputs = plainPrinter.log(event);
@@ -37,13 +38,13 @@ void main() {
   });
 
   test('print time', () {
-    var printer = SimplePrinter(printTime: true);
+    var printer = SimplePrinter(dateTimeFormat: DateTimeFormat.iso8601);
 
     expect(printer.log(event)[0], contains('TIME'));
   });
 
   test('does not print time', () {
-    var printer = SimplePrinter(printTime: false);
+    var printer = SimplePrinter();
 
     expect(printer.log(event)[0], isNot(contains('TIME')));
   });