bugfix Tracker.record to not discard default values (#27)

Author: bbracker-int

lib/src/tracker.dart

@@ -137,7 +137,7 @@ class Tracker<TrackableType extends Trackable> {
   void record(TrackableType trackable,
       {Map<String, String?> defaults = const {}}) {
     for (final dumper in _dumpers) {
-      dumper.record(trackable);
+      dumper.record(trackable, defaults: defaults);
     }
   }
 

test/tracker_test.dart

@@ -21,8 +21,9 @@ class FruitEvent implements Trackable {
   final LogicValue apple;
   final String banana;
   final int carrot;
+  final String? pear;
 
-  FruitEvent(this.apple, this.banana, this.carrot);
+  FruitEvent(this.apple, this.banana, this.carrot, this.pear);
 
   @override
   String? trackerString(TrackerField field) {
@@ -35,6 +36,8 @@ class FruitEvent implements Trackable {
         return carrot.toString();
       case 'Durian':
         return (carrot * 2).toRadixString(16);
+      case 'Pear':
+        return pear;
     }
     return null;
   }
@@ -48,17 +51,22 @@ void main() {
         const TrackerField('Apple', columnWidth: 10),
         const TrackerField('Banana', columnWidth: 5),
         const TrackerField('Carrot', columnWidth: 12, justify: Justify.center),
-        const TrackerField('Durian', columnWidth: 12, mapOnly: true)
+        const TrackerField('Durian', columnWidth: 12, mapOnly: true),
+        const TrackerField('Pear', columnWidth: 12)
       ],
     )
-      ..record(FruitEvent(LogicValue.ofString('1x0'), 'banana', 25))
       ..record(
-          FruitEvent(LogicValue.ofString('1x01111000011010101'), 'aaa', 5));
+          FruitEvent(LogicValue.ofString('1x0'), 'banana', 25, 'green pear'),
+          defaults: {'Pear': 'red pear'})
+      ..record(
+          FruitEvent(
+              LogicValue.ofString('1x01111000011010101'), 'aaa', 5, null),
+          defaults: {'Pear': 'red pear'});
 
     // Expect JSON log to look like:
     // {"records":[
-    //   {"Apple": "3'b1x0", "Banana": "banana", "Carrot": "25", "Durian": "32"}
-    // , {"Apple": "19'b1x01111000011010101", "Banana": "aaa", "Carrot": "4", "Durian": "8"}
+    //   {"Apple": "3'b1x0", "Banana": "banana", "Carrot": "25", "Durian": "32", "Pear": "green pear"}
+    // , {"Apple": "19'b1x01111000011010101", "Banana": "aaa", "Carrot": "4", "Durian": "8", "Pear": "red pear"}
     // ]}
 
     await tracker.terminate();
@@ -68,22 +76,24 @@ void main() {
     expect(jsonOutput['records'].length, equals(2));
     expect(jsonOutput['records'][0]['Banana'], equals('banana'));
     expect(jsonOutput['records'][1]['Durian'], equals('a'));
+    expect(jsonOutput['records'][0]['Pear'], equals('green pear'));
+    expect(jsonOutput['records'][1]['Pear'], equals('red pear'));
 
     // Expect table log to look like:
-    // ---------------------------------------
-    //  | A          | B     | C            |
-    //  | P          | A     | A            |
-    //  | P          | N     | R            |
-    //  | L          | A     | R            |
-    //  | E          | N     | O            |
-    //  |            | A     | T            |
-    // ---------------------------------------
-    //  |     3'b1x0 | bana* |      25      | {Apple: 3'b1x0, Banana: banana, Carrot: 25, Durian: 32}
-    //  | 19'b1x011* |   aaa |      5       | {Apple: 19'b1x01111000011010101, Banana: aaa, Carrot: 5, Durian: a}
+// ------------------------------------------------------
+//  | A          | B     | C            | P            |
+//  | P          | A     | A            | E            |
+//  | P          | N     | R            | A            |
+//  | L          | A     | R            | R            |
+//  | E          | N     | O            |              |
+//  |            | A     | T            |              |
+// ------------------------------------------------------
+//  |     3'b1x0 | bana* |      25      |   green pear | {Apple: 3'b1x0, Banana: banana, Carrot: 25, Durian: 32, Pear: green pear}
+//  | 19'b1x011* |   aaa |      5       |     red pear | {Apple: 19'b1x01111000011010101, Banana: aaa, Carrot: 5, Durian: a, Pear: red pear}
 
     final logOutput = File(tracker.tableFileName).readAsStringSync();
     expect(logOutput.contains('bana*'), equals(true));
-    expect(logOutput.split('\n')[1].split('|').length, equals(5));
+    expect(logOutput.split('\n')[1].split('|').length, equals(6));
 
     File(tracker.jsonFileName).deleteSync();
     File(tracker.tableFileName).deleteSync();