chore: Failure implement Exception

chore: upgraded mek_lints package

.gitignore

@@ -1,3 +1,5 @@
+**/.DS_Store
+
 # IntelliJ related
 *.iml
 *.ipr

kuama_core/lib/src/failures/failure.dart

@@ -9,7 +9,7 @@ class ExceptionCaught {
 }
 
 /// Maintainers: Keep same structure as [Error]/[Exception]
-abstract class Failure {
+abstract class Failure implements Exception {
   final ExceptionCaught? exceptionCaught;
 
   Failure({required this.exceptionCaught});

kuama_core/pubspec.yaml

@@ -1,7 +1,7 @@
 name: kuama_core
 description: A starting point for Dart libraries or applications.
 publish_to: 'none'
-version: 1.3.0
+version: 1.3.1
 
 environment:
   sdk: '>=3.0.0 <4.0.0'
@@ -15,7 +15,7 @@ dependencies:
   rxdart: '>=0.27.0 <0.29.0'
 
 dev_dependencies:
-  mek_lints: ^1.0.0
+  mek_lints: ^4.0.0
 
   test: ^1.21.1
 

kuama_permissions/build.yaml

@@ -2,6 +2,9 @@ targets:
   $default:
     builders:
       source_gen:combining_builder:
+        options:
+          ignore_for_file:
+            - unnecessary_parenthesis
         generate_for:
           exclude:
             - test

kuama_permissions/lib/src/blocs/_permissions_event.dart

@@ -70,6 +70,7 @@ class _AskPermissionsEvent extends _PermissionsBlocEvent with _$_AskPermissionsE
 class _ConfirmAskPermissionsEvent extends _PermissionsBlocEvent with _$_ConfirmAskPermissionsEvent {
   final bool? canRequest;
 
+  // ignore: avoid_positional_boolean_parameters
   _ConfirmAskPermissionsEvent(this.canRequest);
 
   @override

kuama_permissions/lib/src/blocs/_permissions_state.dart

@@ -49,7 +49,7 @@ abstract class PermissionsBlocState with _$PermissionsBlocState {
     final isAllEnabled = permissions
         .map((e) => e.toService())
         .whereNotNull()
-        .every((element) => servicesStatus[element] == true);
+        .every((element) => servicesStatus[element] ?? false);
     if (!isAllEnabled) return true;
     return permissions.any((permission) {
       return permissionsStatus[permission] != PermissionStatus.granted;
@@ -235,15 +235,11 @@ class CheckingPermissionState extends PermissionsBlocState with _$CheckingPermis
   final Set<Permission> payload;
 
   const CheckingPermissionState({
-    required bool isRefreshing,
-    required Map<Permission, PermissionStatus> permissionsStatus,
-    required Map<Service, bool> servicesStatus,
+    required super.isRefreshing,
+    required super.permissionsStatus,
+    required super.servicesStatus,
     required this.payload,
-  }) : super(
-          isRefreshing: isRefreshing,
-          permissionsStatus: permissionsStatus,
-          servicesStatus: servicesStatus,
-        );
+  });
 
   @override
   bool get isLoading => true;
@@ -267,15 +263,11 @@ class CheckedPermissionState extends PermissionsBlocState with _$CheckedPermissi
   final Set<Permission> payload;
 
   const CheckedPermissionState({
-    required bool isRefreshing,
-    required Map<Permission, PermissionStatus> permissionsStatus,
-    required Map<Service, bool> servicesStatus,
+    required super.isRefreshing,
+    required super.permissionsStatus,
+    required super.servicesStatus,
     required this.payload,
-  }) : super(
-          isRefreshing: isRefreshing,
-          permissionsStatus: permissionsStatus,
-          servicesStatus: servicesStatus,
-        );
+  });
 
   @override
   bool get isLoading => false;
@@ -299,15 +291,11 @@ class RequestingPermissionsState extends PermissionsBlocState with _$RequestingP
   final Set<Permission> payload;
 
   const RequestingPermissionsState({
-    required bool isRefreshing,
-    required Map<Permission, PermissionStatus> permissionsStatus,
-    required Map<Service, bool> servicesStatus,
+    required super.isRefreshing,
+    required super.permissionsStatus,
+    required super.servicesStatus,
     required this.payload,
-  }) : super(
-          isRefreshing: isRefreshing,
-          permissionsStatus: permissionsStatus,
-          servicesStatus: servicesStatus,
-        );
+  });
 
   @override
   bool get isLoading => true;
@@ -334,16 +322,12 @@ class RequestedPermissionsState extends PermissionsBlocState with _$RequestedPer
   final bool isRequested;
 
   const RequestedPermissionsState({
-    required bool isRefreshing,
-    required Map<Permission, PermissionStatus> permissionsStatus,
-    required Map<Service, bool> servicesStatus,
+    required super.isRefreshing,
+    required super.permissionsStatus,
+    required super.servicesStatus,
     required this.payload,
     required this.isRequested,
-  }) : super(
-          isRefreshing: isRefreshing,
-          permissionsStatus: permissionsStatus,
-          servicesStatus: servicesStatus,
-        );
+  });
 
   @override
   bool get isLoading => false;
@@ -367,15 +351,11 @@ class AskingPermissionsState extends PermissionsBlocState with _$AskingPermissio
   final Set<Permission> payload;
 
   const AskingPermissionsState({
-    required bool isRefreshing,
-    required Map<Permission, PermissionStatus> permissionsStatus,
-    required Map<Service, bool> servicesStatus,
+    required super.isRefreshing,
+    required super.permissionsStatus,
+    required super.servicesStatus,
     required this.payload,
-  }) : super(
-          isRefreshing: isRefreshing,
-          permissionsStatus: permissionsStatus,
-          servicesStatus: servicesStatus,
-        );
+  });
 
   @override
   bool get isLoading => true;
@@ -403,16 +383,12 @@ class ConfirmableAskPermissionsState extends PermissionsBlocState
   final bool isRestored;
 
   const ConfirmableAskPermissionsState({
-    required bool isRefreshing,
-    required Map<Permission, PermissionStatus> permissionsStatus,
-    required Map<Service, bool> servicesStatus,
+    required super.isRefreshing,
+    required super.permissionsStatus,
+    required super.servicesStatus,
     required this.payload,
     required this.isRestored,
-  }) : super(
-          isRefreshing: isRefreshing,
-          permissionsStatus: permissionsStatus,
-          servicesStatus: servicesStatus,
-        );
+  });
 
   @override
   bool get isLoading => false;
@@ -441,17 +417,13 @@ class AskedPermissionsState extends PermissionsBlocState with _$AskedPermissions
   final bool isRequested;
 
   const AskedPermissionsState({
-    required bool isRefreshing,
-    required Map<Permission, PermissionStatus> permissionsStatus,
-    required Map<Service, bool> servicesStatus,
+    required super.isRefreshing,
+    required super.permissionsStatus,
+    required super.servicesStatus,
     required this.payload,
     required this.isCancelled,
     required this.isRequested,
-  }) : super(
-          isRefreshing: isRefreshing,
-          permissionsStatus: permissionsStatus,
-          servicesStatus: servicesStatus,
-        );
+  });
 
   @override
   bool get isLoading => false;

kuama_permissions/lib/src/blocs/permissions_bloc.b.dart

@@ -88,6 +88,7 @@ class PermissionsBloc extends Bloc<_PermissionsBlocEvent, PermissionsBlocState>
   /// If `null`, the request is discarded.
   ///
   /// Note: The confirmation may be completed after a refresh. See [PermissionsBlocState.isRefreshing].
+  // ignore: avoid_positional_boolean_parameters
   void confirmAsk(bool? canRequest) => add(_ConfirmAskPermissionsEvent(canRequest));
 
   Future<void> _onEvent(_PermissionsBlocEvent event, Emitter<PermissionsBlocState> emit) async {

kuama_permissions/lib/src/repositories/app_lifecycle_state_repository.dart

@@ -9,13 +9,14 @@ class AppLifecycleStateRepository {
   WidgetsBindingObserver? _observer;
 
   static AppLifecycleStateRepository? _instance;
-  AppLifecycleStateRepository._();
 
   /// It is a singleton because only one observer should exist
   factory AppLifecycleStateRepository() {
     return _instance ??= AppLifecycleStateRepository._();
   }
 
+  AppLifecycleStateRepository._();
+
   Stream<AppLifecycleState> get onChanges {
     _subject ??= StreamController.broadcast(
       sync: true,
@@ -29,6 +30,8 @@ class AppLifecycleStateRepository {
 
     return _subject!.stream;
   }
+
+  void dispose() => unawaited(_subject?.close());
 }
 
 class _Observer extends WidgetsBindingObserver {

kuama_permissions/lib/src/repositories/permissions_manger_repository.dart

@@ -5,7 +5,7 @@ import 'package:permission_handler_platform_interface/permission_handler_platfor
 class PermissionsManagerRepository {
   PermissionHandlerPlatform get _handler => GetIt.I();
 
-  /// Returns [true] if the app settings page could be opened, otherwise  [false].
+  /// Returns `true` if the app settings page could be opened, otherwise `false`.
   Future<bool> openAppSettings() async {
     return await _handler.openAppSettings();
   }

kuama_permissions/lib/src/services/permissions_service.dart

@@ -83,7 +83,7 @@ class PermissionsService {
   }
 
   Stream<void> get onRequiredPermissionsRefresh async* {
-    AppLifecycleState prevState = AppLifecycleState.paused;
+    var prevState = AppLifecycleState.paused;
     await for (final state in _appLifecycleState.onChanges) {
       if (prevState != AppLifecycleState.resumed &&
           prevState != AppLifecycleState.inactive &&

kuama_permissions/pubspec.yaml

@@ -1,7 +1,7 @@
 name: kuama_permissions
 description: A starting point for Dart libraries or applications.
 publish_to: 'none'
-version: 1.3.0
+version: 1.3.1
 
 environment:
   sdk: '>=3.0.0 <4.0.0'
@@ -20,7 +20,7 @@ dependencies:
     git:
       url: https://github.com/Kuama-IT/kuama_dart.git
       path: kuama_core
-      ref: 1.3.0
+      ref: 1.3.1
 
   permission_handler_platform_interface: ^4.2.1
   permission_handler: ^11.3.1
@@ -36,7 +36,7 @@ dependencies:
   mek_data_class: ^1.4.0
 
 dev_dependencies:
-  mek_lints: ^1.0.0
+  mek_lints: ^4.0.0
 
   flutter_test: { sdk: flutter }
 

kuama_permissions/test/blocs/permissions_bloc_test.dart

@@ -38,9 +38,9 @@ void main() {
     bloc = PermissionsBloc();
   });
 
-  tearDown(GetIt.instance.reset);
+  tearDown(() async => Future.wait([GetIt.instance.reset(), onRequiredPermissionsRefresh.close()]));
 
-  group("PermissionsBloc", () {
+  group('PermissionsBloc', () {
     const tPermission1 = Permission.storage;
     const tPermission2 = Permission.camera;
     const tPermissionService = Permission.location;
@@ -49,7 +49,7 @@ void main() {
     group('PermissionsBloc.check', () {
       test('cant check because cant operate', () {
         bloc.emit(mockState);
-        var state = bloc.state;
+        final state = bloc.state;
 
         when(() => mockState.checkCanCheck(any())).thenReturn(false);
 
@@ -123,7 +123,7 @@ void main() {
     group('PermissionsBloc.request', () {
       test('cant request because cant operate', () {
         bloc.emit(mockState);
-        var state = bloc.state;
+        final state = bloc.state;
 
         when(() => mockState.checkCanRequest(any())).thenReturn(false);
 
@@ -207,7 +207,7 @@ void main() {
     group('PermissionsBloc.confirmAsk', () {
       test('cant confirm ask because cant operate', () {
         bloc.emit(mockState);
-        var state = bloc.state;
+        final state = bloc.state;
 
         when(() => mockState.checkCanConfirmAsk(any())).thenReturn(false);
 
@@ -305,7 +305,7 @@ void main() {
     group('PermissionsBloc.ask', () {
       test('cant ask because cant operate', () {
         bloc.emit(mockState);
-        var state = bloc.state;
+        final state = bloc.state;
 
         when(() => mockState.checkCanAsk(any())).thenReturn(false);
 

kuama_permissions/test/services/permissions_service_test.dart

@@ -1,3 +1,5 @@
+// ignore_for_file: avoid_implementing_value_types
+
 import 'package:flutter_test/flutter_test.dart';
 import 'package:get_it/get_it.dart';
 import 'package:kuama_permissions/src/entities/permissions_status_entity.b.dart';
@@ -35,7 +37,7 @@ void main() {
 
   tearDown(GetIt.instance.reset);
 
-  group("PermissionsService", () {
+  group('PermissionsService', () {
     const tPermission1 = Permission.storage;
     const tService = Permission.locationWhenInUse;
 

kuama_permissions/test/t_utils.dart

@@ -1,6 +1,7 @@
 import 'package:flutter_test/flutter_test.dart';
 import 'package:meta/meta.dart';
 
+// ignore: avoid_void_async
 void after(void Function() fn) async {
   await Future.delayed(const Duration(milliseconds: 1));
   fn();

kuama_position/lib/src/blocs/_position_event.dart

@@ -21,7 +21,7 @@ class _UpdateStatusEvent extends _PositionBlocEvent {
   final bool hasPermissionGranted;
   final bool isServiceEnabled;
 
-  const _UpdateStatusEvent(this.hasPermissionGranted, this.isServiceEnabled);
+  const _UpdateStatusEvent({required this.hasPermissionGranted, required this.isServiceEnabled});
 
   @override
   R map<R>({