🐛 Call AssetPickerProvider.dispose when disposing the builder delegate (#421)

Author: AlexV525

CHANGELOG.md

@@ -4,6 +4,12 @@ that can be found in the LICENSE file. -->
 
 # Changelog
 
+## 8.4.1
+
+### Fixes
+
+- Call `AssetPickerProvider.dispose` when disposing the builder delegate. (#421)
+
 ## 8.4.0
 
 ### Breaking changes

example/pubspec.yaml

@@ -1,6 +1,6 @@
 name: wechat_assets_picker_demo
 description: The demo project for the wechat_assets_picker package.
-version: 8.4.0+34
+version: 8.4.1+35
 publish_to: none
 
 environment:

lib/src/delegates/asset_picker_builder_delegate.dart

@@ -764,6 +764,7 @@ class DefaultAssetPickerBuilderDelegate
     if (keepScrollOffset) {
       return;
     }
+    provider.dispose();
     super.dispose();
   }
 

pubspec.yaml

@@ -1,5 +1,5 @@
 name: wechat_assets_picker
-version: 8.4.0
+version: 8.4.1
 description: |
   An image picker (also with videos and audio)
   for Flutter projects based on WeChat's UI,

test/delegates/sort_path_delegate_test.dart renamed to test/delegates_test.dart

@@ -0,0 +1,40 @@
+// Copyright 2019 The FlutterCandies author. All rights reserved.
+// Use of this source code is governed by an Apache license that can be found
+// in the LICENSE file.
+
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:wechat_assets_picker/wechat_assets_picker.dart';
+
+import 'test_utils.dart';
+
+void main() async {
+  PhotoManager.withPlugin(TestPhotoManagerPlugin());
+  AssetPicker.setPickerDelegate(TestAssetPickerDelegate());
+
+  group('AssetPickerProvider', () {
+    testWidgets('disposed correctly', (WidgetTester tester) async {
+      await tester.pumpWidget(
+        defaultPickerTestApp(
+          onButtonPressed: (BuildContext context) {
+            AssetPicker.pickAssets(context);
+          },
+        ),
+      );
+      await tester.tap(defaultButtonFinder);
+      await tester.pumpAndSettle();
+      final Finder pickerFinder = find.byType(
+        AssetPicker<AssetEntity, AssetPathEntity>,
+      );
+      final AssetPicker<AssetEntity, AssetPathEntity> picker = tester.widget(
+        pickerFinder,
+      );
+      final DefaultAssetPickerProvider provider =
+          (picker.builder as DefaultAssetPickerBuilderDelegate).provider;
+      expect(provider, isA<DefaultAssetPickerProvider>());
+      await tester.tap(find.widgetWithIcon(IconButton, Icons.close));
+      await tester.pumpAndSettle();
+      expect(() => provider.addListener(() {}), throwsA(isA<AssertionError>()));
+    });
+  });
+}