From a82e1c1db860d9c0ebbf00c8cdc36e56d79717ef Mon Sep 17 00:00:00 2001
From: Remi Rousselet <darky12s@gmail.com>
Date: Thu, 14 Mar 2024 20:14:04 +0100
Subject: [PATCH] Unskip stuff

---
 packages/riverpod/test/new/core/ref_test.dart | 38 +++++++------------
 .../test/new/providers/notifier_test.dart     |  6 ---
 .../old/framework/provider_element_test.dart  |  2 -
 .../test/old/framework/ref_watch_test.dart    |  2 -
 4 files changed, 14 insertions(+), 34 deletions(-)

diff --git a/packages/riverpod/test/new/core/ref_test.dart b/packages/riverpod/test/new/core/ref_test.dart
index 3e83558fc..f0e375838 100644
--- a/packages/riverpod/test/new/core/ref_test.dart
+++ b/packages/riverpod/test/new/core/ref_test.dart
@@ -2606,26 +2606,22 @@ void main() {
         verifyZeroInteractions(listener);
       });
 
-      test(
-        'is not called when using container.read (autoDispose)',
-        skip: true,
-        () async {
-          final container = ProviderContainer.test();
-          final listener = OnCancelMock();
-          final dispose = OnDisposeMock();
-          final provider = StateProvider.autoDispose((ref) {
-            ref.keepAlive();
-            ref.onCancel(listener.call);
-            ref.onDispose(dispose.call);
-          });
+      test('is not called when using container.read (autoDispose)', () async {
+        final container = ProviderContainer.test();
+        final listener = OnCancelMock();
+        final dispose = OnDisposeMock();
+        final provider = StateProvider.autoDispose((ref) {
+          ref.keepAlive();
+          ref.onCancel(listener.call);
+          ref.onDispose(dispose.call);
+        });
 
-          container.read(provider);
-          await container.pump();
+        container.read(provider);
+        await container.pump();
 
-          verifyZeroInteractions(listener);
-          verifyZeroInteractions(dispose);
-        },
-      );
+        verifyZeroInteractions(listener);
+        verifyZeroInteractions(dispose);
+      });
 
       test('listeners are cleared on rebuild', () {
         final container = ProviderContainer.test();
@@ -2804,12 +2800,6 @@ void main() {
         expect(buildCount, 1);
         verifyNoMoreInteractions(onDispose);
       });
-
-      test(
-        'once a provider was disposed, cannot add more listeners until it is rebuilt',
-        () {},
-        skip: 'TODO',
-      );
     });
 
     group('mounted', () {
diff --git a/packages/riverpod/test/new/providers/notifier_test.dart b/packages/riverpod/test/new/providers/notifier_test.dart
index e48d7e852..35ae2a421 100644
--- a/packages/riverpod/test/new/providers/notifier_test.dart
+++ b/packages/riverpod/test/new/providers/notifier_test.dart
@@ -498,12 +498,6 @@ void main() {
       verifyOnly(listener, listener(Equal(42), Equal(21)));
     });
 
-    test(
-      'can override the Notifier with a matching custom implementation',
-      () {},
-      skip: 'TODO',
-    );
-
     test('can override Notifier.build', () {});
 
     if (factory.isAutoDispose) {
diff --git a/packages/riverpod/test/old/framework/provider_element_test.dart b/packages/riverpod/test/old/framework/provider_element_test.dart
index 9ece4a450..00c6720fb 100644
--- a/packages/riverpod/test/old/framework/provider_element_test.dart
+++ b/packages/riverpod/test/old/framework/provider_element_test.dart
@@ -158,8 +158,6 @@ void main() {
         ]),
       );
     });
-
-    test('include ref.read dependents', () {}, skip: true);
   });
 
   group('hasListeners', () {
diff --git a/packages/riverpod/test/old/framework/ref_watch_test.dart b/packages/riverpod/test/old/framework/ref_watch_test.dart
index da696092c..9ba5d089e 100644
--- a/packages/riverpod/test/old/framework/ref_watch_test.dart
+++ b/packages/riverpod/test/old/framework/ref_watch_test.dart
@@ -107,8 +107,6 @@ void main() {
     expect(buildCount, 2);
   });
 
-  test('when selector throws, rebuild providers', () {}, skip: true);
-
   test('on provider that threw, exceptions bypass the selector', () {
     final container = ProviderContainer.test();
     final dep = Provider<int>((ref) {