Merge pull request #490 from Baseflow/implementing_test_permission_handler

Added test

permission_handler/CHANGELOG.md

+## 6.1.1
+
+* Added unit-tests to guard API against breaking changes.
+
 ## 6.1.0
 ## 6.1.0
 
 
 * Added support for bluetooth permissions;
 * Added support for bluetooth permissions;

permission_handler/README.md

-[![pub package](https://img.shields.io/pub/v/permission_handler.svg)](https://pub.dartlang.org/packages/permission_handler) [![Build status](https://github.com/Baseflow/flutter-permission-handler/actions/workflows/app_facing_package.yaml/badge.svg?branch=master)](https://github.com/Baseflow/flutter-permission-handler/actions/workflows/app_facing_package.yaml) [![style: effective dart](https://img.shields.io/badge/style-effective_dart-40c4ff.svg)](https://github.com/tenhobi/effective_dart)
+[![pub package](https://img.shields.io/pub/v/permission_handler.svg)](https://pub.dartlang.org/packages/permission_handler) [![Build status](https://github.com/Baseflow/flutter-permission-handler/actions/workflows/app_facing_package.yaml/badge.svg?branch=master)](https://github.com/Baseflow/flutter-permission-handler/actions/workflows/app_facing_package.yaml) [![style: effective dart](https://img.shields.io/badge/style-effective_dart-40c4ff.svg)](https://github.com/tenhobi/effective_dart) [![codecov](https://codecov.io/gh/Baseflow/flutter-permission-handler/branch/master/graph/badge.svg)](https://codecov.io/gh/Baseflow/flutter-permission-handler)
 
 
 On most operating systems, permissions aren't just granted to apps at install time.
 On most operating systems, permissions aren't just granted to apps at install time.
 Rather, developers have to ask the user for permissions while the app is running.
 Rather, developers have to ask the user for permissions while the app is running.

permission_handler/lib/permission_handler.dart

-import 'dart:io';
-
+import 'package:flutter/foundation.dart';
 import 'package:permission_handler_platform_interface/permission_handler_platform_interface.dart';
 import 'package:permission_handler_platform_interface/permission_handler_platform_interface.dart';
 
 
 export 'package:permission_handler_platform_interface/permission_handler_platform_interface.dart'
 export 'package:permission_handler_platform_interface/permission_handler_platform_interface.dart'
@@ -34,7 +33,7 @@ extension PermissionActions on Permission {
   /// This is only implemented on Android, calling this on iOS always returns
   /// This is only implemented on Android, calling this on iOS always returns
   /// [false].
   /// [false].
   Future<bool> get shouldShowRequestRationale async {
   Future<bool> get shouldShowRequestRationale async {
-    if (!Platform.isAndroid) {
+    if (defaultTargetPlatform != TargetPlatform.android) {
       return false;
       return false;
     }
     }
 
 

permission_handler/pubspec.yaml

 name: permission_handler
 name: permission_handler
 description: Permission plugin for Flutter. This plugin provides a cross-platform (iOS, Android) API to request and check permissions.
 description: Permission plugin for Flutter. This plugin provides a cross-platform (iOS, Android) API to request and check permissions.
-version: 6.1.0
+version: 6.1.1
 homepage: https://github.com/baseflowit/flutter-permission-handler
 homepage: https://github.com/baseflowit/flutter-permission-handler
 
 
 flutter:
 flutter:
@@ -16,9 +16,12 @@ dependencies:
   flutter:
   flutter:
     sdk: flutter
     sdk: flutter
   meta: ^1.3.0
   meta: ^1.3.0
-  permission_handler_platform_interface: ^3.1.0
+  permission_handler_platform_interface: ^3.1.1
 
 
 dev_dependencies:
 dev_dependencies:
+  flutter_test:
+    sdk: flutter
+  mockito: ^5.0.1
   effective_dart: ^1.3.0
   effective_dart: ^1.3.0
   plugin_platform_interface: ^2.0.0
   plugin_platform_interface: ^2.0.0
 
 

permission_handler/test/permission_handler_test.dart

+import 'package:flutter_test/flutter_test.dart';
+import 'package:mockito/mockito.dart';
+import 'package:permission_handler/permission_handler.dart';
+import 'package:permission_handler_platform_interface/permission_handler_platform_interface.dart';
+import 'package:plugin_platform_interface/plugin_platform_interface.dart';
+
+void main() {
+  group('PermissionHandler', () {
+    setUp(() {
+      PermissionHandlerPlatform.instance = MockPermissionHandlerPlatform();
+    });
+
+    test('openAppSettings', () async {
+      final hasOpened = await openAppSettings();
+
+      expect(hasOpened, true);
+    });
+
+    test('PermissionActions on Permission: get status', () async {
+      final permissionStatus = await Permission.calendar.status;
+
+      expect(permissionStatus, PermissionStatus.granted);
+    });
+
+    test(
+        // ignore: lines_longer_than_80_chars
+        'PermissionActions on Permission: get shouldShowRequestRationale should return true when on android',
+        () async {
+      final mockPermissionHandlerPlatform = PermissionHandlerPlatform.instance;
+
+      when(mockPermissionHandlerPlatform
+              .shouldShowRequestPermissionRationale(Permission.calendar))
+          .thenAnswer((_) => Future.value(true));
+
+      await Permission.calendar.shouldShowRequestRationale;
+
+      verify(mockPermissionHandlerPlatform
+              .shouldShowRequestPermissionRationale(Permission.calendar))
+          .called(1);
+    });
+
+    test('PermissionActions on Permission: request()', () async {
+      final permissionRequest = Permission.calendar.request();
+
+      expect(permissionRequest, isA<Future<PermissionStatus>>());
+    });
+
+    test('PermissionCheckShortcuts on Permission: get isGranted', () async {
+      final isGranted = await Permission.calendar.isGranted;
+      expect(isGranted, true);
+    });
+
+    test('PermissionCheckShortcuts on Permission: get isDenied', () async {
+      final isDenied = await Permission.calendar.isDenied;
+      expect(isDenied, false);
+    });
+
+    test('PermissionCheckShortcuts on Permission: get isRestricted', () async {
+      final isRestricted = await Permission.calendar.isRestricted;
+      expect(isRestricted, false);
+    });
+
+    test('PermissionCheckShortcuts on Permission: get isLimited', () async {
+      final isLimited = await Permission.calendar.isLimited;
+      expect(isLimited, false);
+    });
+
+    test('PermissionCheckShortcuts on Permission: get isPermanentlyDenied',
+        () async {
+      final isPermanentlyDenied = await Permission.calendar.isPermanentlyDenied;
+      expect(isPermanentlyDenied, false);
+    });
+
+    test(
+        // ignore: lines_longer_than_80_chars
+        'ServicePermissionActions on PermissionWithService: get ServiceStatus returns the right service status',
+        () async {
+      var serviceStatus = await Permission.phone.serviceStatus;
+
+      expect(serviceStatus, ServiceStatus.enabled);
+    });
+
+    test(
+        // ignore: lines_longer_than_80_chars
+        'PermissionListActions on List<Permission>: request() on  a list returns a Map<Permission, PermissionStatus>',
+        () async {
+      var permissionList = <Permission>[];
+      final permissionMap = await permissionList.request();
+
+      expect(permissionMap, isA<Map<Permission, PermissionStatus>>());
+    });
+  });
+}
+
+class MockPermissionHandlerPlatform extends Mock
+    with
+        // ignore: prefer_mixin
+        MockPlatformInterfaceMixin
+    implements
+        PermissionHandlerPlatform {
+  @override
+  Future<PermissionStatus> checkPermissionStatus(Permission permission) =>
+      Future.value(PermissionStatus.granted);
+
+  @override
+  Future<ServiceStatus> checkServiceStatus(Permission permission) =>
+      Future.value(ServiceStatus.enabled);
+
+  @override
+  Future<bool> openAppSettings() => Future.value(true);
+
+  @override
+  Future<Map<Permission, PermissionStatus>> requestPermissions(
+      List<Permission> permissions) {
+    var permissionsMap = <Permission, PermissionStatus>{};
+    return Future.value(permissionsMap);
+  }
+
+  @override
+  Future<bool> shouldShowRequestPermissionRationale(Permission? permission) {
+    return super.noSuchMethod(
+      Invocation.method(
+        #shouldShowPermissionRationale,
+        [permission],
+      ),
+      returnValue: Future.value(true),
+    );
+  }
+}