Frezyx · Frezyx · Jun 13, 2025 · May 31, 2025 · May 31, 2025 · May 31, 2025
diff --git a/packages/talker_riverpod_logger/lib/riverpod_logs.dart b/packages/talker_riverpod_logger/lib/riverpod_logs.dart
@@ -6,11 +6,7 @@ String _defaultMessage({
   required ProviderBase<Object?> provider,
   required String suffix,
 }) {
-  if (provider.name == null) {
-    return '${provider.runtimeType} $suffix';
-  }
-
-  return '${provider.name} | ${provider.runtimeType} $suffix';
+  return "$provider $suffix";
 }
 
 /// [Riverpod] add provider log model

diff --git a/packages/talker_riverpod_logger/test/observer_test.dart b/packages/talker_riverpod_logger/test/observer_test.dart
@@ -1,5 +1,7 @@
 // ignore_for_file: invalid_use_of_protected_member, override_on_non_overriding_member
 
+import 'dart:async';
+
 import 'package:riverpod/riverpod.dart';
 import 'package:talker/talker.dart';
 import 'package:talker_riverpod_logger/talker_riverpod_logger.dart';
@@ -13,6 +15,25 @@ class TestNotifier extends StateNotifier<String> {
   }
 }
 
+class FamilyTestNotifier extends FamilyNotifier<int, String> {
+  @override
+  int build(String arg) {
+    return arg.length;
+  }
+
+  void changeState(String newState) {
+    state = newState.length;
+  }
+}
+
+final familyProvider = NotifierProvider.family<FamilyTestNotifier, int, String>(
+  FamilyTestNotifier.new,
+);
+
+final familyErrorProvider = FutureProvider.family<String, String>(
+  (ref, arg) => throw ("Error"),
+);
+
 ProviderContainer createContainer({
   ProviderContainer? parent,
   List<Override> overrides = const [],
@@ -90,28 +111,71 @@ void main() {
       expect(log.generateTextMessage(), contains('failed'));
     });
 
-    test('providerDidFail with filter', () async {
-      talkerRiverpodObserver = TalkerRiverpodObserver(
-        talker: talker,
-        settings: TalkerRiverpodLoggerSettings(
-          enabled: true,
-          printProviderDisposed: true,
-          didFailFilter: (error) {
-            if (error is Exception) return true;
-            return false;
-          },
-        ),
-      );
-      container = createContainer(
-        observers: [talkerRiverpodObserver],
-        overrides: [
-          provider.overrideWith((ref) => TestNotifier()),
-        ],
-      );
-      container.read(errorProvider);
-      await Future.delayed(const Duration(milliseconds: 10));
-      final log = talker.history;
-      expect(log.whereType<RiverpodFailLog>(), isEmpty);
+    group('with arguments', () {
+      test('didAddProvider', () {
+        const arg = "99";
+        const initial = 2;
+        container.read(familyProvider(arg));
+        final generateTextMessage = talker.history.first.generateTextMessage();
+        expect(generateTextMessage, contains("($arg)"));
+        expect(generateTextMessage, contains("$initial"));
+      });
+      test('didUpdateProvider', () async {
+        const arg = "99";
+        const input = "999";
+        const expectedChange = 3;
+        container.read(familyProvider(arg));
+        await container.pump();
+        container.read(familyProvider(arg).notifier).changeState(input);
+        await container.pump();
+        final generateTextMessage = talker.history.last.generateTextMessage();
+        expect(generateTextMessage, contains("($arg)"));
+        expect(generateTextMessage, contains("$expectedChange"));
+      });
+      test('didDisposeProvider', () async {
+        const arg = "99";
+        container.read(familyProvider(arg));
+        await container.pump();
+        container.dispose();
+        await container.pump();
+        final generateTextMessage = talker.history.last.generateTextMessage();
+        expect(generateTextMessage, contains("($arg)"));
+        expect(generateTextMessage, contains('disposed'));
+      });
+
+      test('providerDidFail', () async {
+        const arg = "99";
+        try {
+          final _ = container.read(familyErrorProvider(arg));
+        } catch (_) {}
+        await container.pump();
+        final log = talker.history.first.generateTextMessage();
+        expect(log, contains("($arg)"));
+        expect(log, contains('failed'));
+      });
+      test('providerDidFail with filter', () async {
+        talkerRiverpodObserver = TalkerRiverpodObserver(
+          talker: talker,
+          settings: TalkerRiverpodLoggerSettings(
+            enabled: true,
+            printProviderDisposed: true,
+            didFailFilter: (error) {
+              if (error is Exception) return true;
+              return false;
+            },
+          ),
+        );
+        container = createContainer(
+          observers: [talkerRiverpodObserver],
+          overrides: [
+            provider.overrideWith((ref) => TestNotifier()),
+          ],
+        );
+        container.read(errorProvider);
+        await Future.delayed(const Duration(milliseconds: 10));
+        final log = talker.history;
+        expect(log.whereType<RiverpodFailLog>(), isEmpty);
+      });
     });
   });
 }