fixed sublistenable parameters

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 1.0.2
+
+- Fixed SubListenanble requiring a builder instead of a child
+
 ## 1.0.1
 
 - Fixed an issue with SubListenable calling its listener too much

lib/src/sub_listenable.dart

@@ -23,7 +23,8 @@ class SubListener extends StatelessWidget {
   /// - The optional [listener] is called when the listenable notifies.
   /// - If [initialize] is true, then [listener] is also called once this Widget is built for the first time.
   const SubListener({
-    required this.builder,
+    @required this.child,
+    @Deprecated('Use the child Parameter instead') this.builder,
     required this.listenable,
     this.listener,
     this.initialize = false,
@@ -38,11 +39,20 @@ class SubListener extends StatelessWidget {
   /// Whether to call [listener] when the [SubListener] is created initially.
   final bool initialize;
 
+  /// The widget below this one.
+  final Widget? child;
+
   /// Builds the widget below this one.
-  final WidgetBuilder builder;
+  @Deprecated('Use the child Parameter instead')
+  final WidgetBuilder? builder;
 
   @override
   Widget build(BuildContext context) {
+    assert(
+      // ignore: deprecated_member_use_from_same_package
+      builder != null || child != null,
+      '$runtimeType must have a child widget!',
+    );
     return SubValue<_SubListenerState>(
       create: () => _SubListenerState(
         listenable: listenable,
@@ -51,7 +61,8 @@ class SubListener extends StatelessWidget {
       ),
       keys: [listenable],
       dispose: (value) => value.dispose(),
-      builder: (context, _) => builder(context),
+      // ignore: deprecated_member_use_from_same_package
+      builder: (context, _) => builder?.call(context) ?? child!,
     );
   }
 }

lib/src/sub_value_listenable.dart

@@ -38,7 +38,7 @@ class SubValueListener<T> extends SubListener {
   }) : super(
           listenable: listenable,
           listener: listener != null ? () => listener(listenable.value) : null,
-          builder: (context) => ValueListenableBuilder<T>(
+          child: ValueListenableBuilder<T>(
             valueListenable: listenable,
             builder: (context, value, child) => builder(context, value),
           ),

pubspec.yaml

@@ -2,7 +2,7 @@ name: flutter_sub
 description: Idiomatic State Lifecycles for Flutter. Part of the State, part of the Tree.
 repository: https://github.com/clragon/flutter_sub/
 
-version: 1.0.1
+version: 1.0.2
 
 environment:
   sdk: ">=2.17.0 <3.0.0"

test/sub_listenable_test.dart

@@ -6,8 +6,6 @@ import 'mock.dart';
 
 void main() {
   group('SubListener', () {
-    Container mockBuilder(BuildContext context) => Container();
-
     testWidgets('calls its listener', (tester) async {
       final notifier = ValueNotifier(0);
       int counter = 0;
@@ -16,7 +14,7 @@ void main() {
         SubListener(
           listenable: notifier,
           listener: () => counter++,
-          builder: mockBuilder,
+          child: const SizedBox(),
         ),
       );
 
@@ -40,12 +38,28 @@ void main() {
           initialize: true,
           listenable: notifier,
           listener: () => counter++,
-          builder: mockBuilder,
+          child: const SizedBox(),
         ),
       );
 
       expect(counter, 1);
     });
+
+    testWidgets('throws if neither child nor builder are used', (tester) async {
+      final notifier = ValueNotifier(0);
+      int counter = 0;
+
+      await tester.pumpWidget(
+        // ignore: missing_required_param
+        SubListener(
+          initialize: true,
+          listenable: notifier,
+          listener: () => counter++,
+        ),
+      );
+
+      expect(tester.takeException(), isAssertionError);
+    });
   });
 
   group('SubDisposableListenable', () {