refactor: improve project (#25)

* chore: update gradle

* chore: set android:exported on `AndroidManifest`

* chore: update deps

* style: fix formatting

* refactor: replace `WillPopScope` with `PopScope`

* refactor: rename `requiresReauthentication` to `needsReauthentication`

* refactor: move `needsReauthentication` on base `Failure`

* refactor: rename `UserFailure.none` to `UserFailure.empty`

* refactor: rename `TopicsFailure.none` to `TopicsFailure.empty`

* refactor: rename `QuizzesFailure.none` to `QuizzesFailure.empty`

* refactor: rename `User.none` to `User.empty`

* refactor: rename `Option.none` to `Option.empty`

* refactor: rename `Question.none` to `Question.empty`

* refactor: rename `Quiz.none` to `Quiz.empty`

* refactor: rename `Topic.none` to `Topic.empty`

* refactor: rename `Quiz.none` to `Quiz.empty`

* docs(user_repository): improve comments

* chore: apply flutter grade plugins declaratively

android/app/build.gradle

@@ -1,3 +1,11 @@
+plugins {
+    id 'com.android.application'
+    id 'kotlin-android'
+    id "org.jetbrains.kotlin.android"
+    id 'dev.flutter.flutter-gradle-plugin'
+    id 'com.google.gms.google-services'
+}
+
 def localProperties = new Properties()
 def localPropertiesFile = rootProject.file('local.properties')
 if (localPropertiesFile.exists()) {
@@ -6,11 +14,6 @@ if (localPropertiesFile.exists()) {
     }
 }
 
-def flutterRoot = localProperties.getProperty('flutter.sdk')
-if (flutterRoot == null) {
-    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
-}
-
 def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
 if (flutterVersionCode == null) {
     flutterVersionCode = '1'
@@ -27,11 +30,6 @@ if (keystorePropertiesFile.exists()) {
     keystoreProperties.load(new FileInputStream(keystorePropertiesFile))
 }
 
-apply plugin: 'com.android.application'
-apply plugin: 'com.google.gms.google-services'
-apply plugin: 'kotlin-android'
-apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
-
 android {
     compileSdkVersion flutter.compileSdkVersion
     ndkVersion flutter.ndkVersion
@@ -109,6 +107,5 @@ flutter {
 }
 
 dependencies {
-    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
     implementation 'com.android.support:multidex:1.0.3'
 }

android/build.gradle

@@ -1,17 +1,3 @@
-buildscript {
-    ext.kotlin_version = '1.7.10'
-    repositories {
-        google()
-        mavenCentral()
-    }
-
-    dependencies {
-        classpath 'com.android.tools.build:gradle:7.4.2'
-        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
-        classpath 'com.google.gms:google-services:4.4.1'
-    }
-}
-
 allprojects {
     repositories {
         google()

android/settings.gradle

@@ -1,11 +1,26 @@
-include ':app'
+pluginManagement {
+    def flutterSdkPath = {
+        def properties = new Properties()
+        file("local.properties").withInputStream { properties.load(it) }
+        def flutterSdkPath = properties.getProperty("flutter.sdk")
+        assert flutterSdkPath != null, "flutter.sdk not set in local.properties"
+        return flutterSdkPath
+    }()
 
-def localPropertiesFile = new File(rootProject.projectDir, "local.properties")
-def properties = new Properties()
+    includeBuild("$flutterSdkPath/packages/flutter_tools/gradle")
 
-assert localPropertiesFile.exists()
-localPropertiesFile.withReader("UTF-8") { reader -> properties.load(reader) }
+    repositories {
+        google()
+        mavenCentral()
+        gradlePluginPortal()
+    }
+}
 
-def flutterSdkPath = properties.getProperty("flutter.sdk")
-assert flutterSdkPath != null, "flutter.sdk not set in local.properties"
-apply from: "$flutterSdkPath/packages/flutter_tools/gradle/app_plugin_loader.gradle"
+plugins {
+    id "dev.flutter.flutter-plugin-loader" version "1.0.0"
+    id "com.android.application" version "7.4.2" apply false
+    id "org.jetbrains.kotlin.android" version "1.7.10" apply false
+    id "com.google.gms.google-services" version "4.4.0" apply false
+}
+
+include ":app"

lib/app/cubit/app_cubit.dart

@@ -8,7 +8,7 @@ class AppCubit extends Cubit<AppState> {
     required UserRepository userRepository,
   })  : _userRepository = userRepository,
         super(
-          userRepository.user.isNone
+          userRepository.user.isEmpty
               ? const AppState.unauthenticated()
               : AppState.newlyAuthenticated(userRepository.user),
         ) {
@@ -32,7 +32,7 @@ class AppCubit extends Cubit<AppState> {
   }
 
   void _onUserChanged(User user) {
-    if (user.isNone) {
+    if (user.isEmpty) {
       emit(const AppState.unauthenticated());
     } else if (state.isUnauthenticated) {
       emit(AppState.newlyAuthenticated(user));
@@ -44,7 +44,7 @@ class AppCubit extends Cubit<AppState> {
   void _onUserFailed(UserFailure failure) {
     final currentState = state;
     emit(AppState.failure(failure: failure, user: currentState.user));
-    if (failure.requiresReauthentication) {
+    if (failure.needsReauthentication) {
       emit(const AppState.unauthenticated());
     } else {
       emit(currentState);

lib/app/cubit/app_state.dart

@@ -17,8 +17,8 @@ extension AppStatusExtensions on AppStatus {
 final class AppState extends Equatable {
   const AppState._({
     required this.status,
-    this.user = User.none,
-    this.failure = UserFailure.none,
+    this.user = User.empty,
+    this.failure = UserFailure.empty,
   });
 
   const AppState.unauthenticated() : this._(status: AppStatus.unauthenticated);

lib/login/cubit/login_state.dart

@@ -18,7 +18,7 @@ final class LoginState extends Equatable {
   const LoginState._({
     this.status = LoginStatus.initial,
     this.signInMethod = SignInMethod.none,
-    this.failure = UserFailure.none,
+    this.failure = UserFailure.empty,
   });
 
   const LoginState.initial() : this._();

lib/profile/cubit/profile_cubit.dart

@@ -4,7 +4,7 @@ import 'package:user_repository/user_repository.dart';
 class ProfileCubit extends Cubit<User> {
   ProfileCubit({required UserRepository userRepository})
       : _userRepository = userRepository,
-        super(User.none) {
+        super(User.empty) {
     _watchUser();
   }
 

lib/quiz/cubit/quiz_cubit.dart

@@ -19,7 +19,7 @@ class QuizCubit extends Cubit<QuizState> {
     try {
       emit(state.fromQuizLoading());
       final quiz = await _quizzesRepository.getQuiz(quizId);
-      if (quiz.isNone) {
+      if (quiz.isEmpty) {
         emit(state.fromQuizEmpty());
         return;
       }
@@ -38,7 +38,7 @@ class QuizCubit extends Cubit<QuizState> {
   }
 
   void unselectOption() {
-    emit(state.copyWith(selectedOption: Option.none));
+    emit(state.copyWith(selectedOption: Option.empty));
   }
 
   void validateAnswer() {

lib/quiz/cubit/quiz_state.dart

@@ -5,10 +5,10 @@ enum QuizStatus { initial, loading, empty, loaded, failure }
 final class QuizState extends Equatable {
   const QuizState._({
     this.status = QuizStatus.initial,
-    this.quiz = Quiz.none,
-    this.selectedOption = Option.none,
+    this.quiz = Quiz.empty,
+    this.selectedOption = Option.empty,
     this.step = 0,
-    this.failure = QuizzesFailure.none,
+    this.failure = QuizzesFailure.empty,
   });
 
   const QuizState.initial() : this._();
@@ -57,5 +57,5 @@ extension QuizStateExtensions on QuizState {
   double get progress => step / steps;
 
   Question operator [](int step) =>
-      quiz.questions.isEmpty ? Question.none : quiz.questions[step - 1];
+      quiz.questions.isEmpty ? Question.empty : quiz.questions[step - 1];
 }

lib/quiz/view/quiz_page.dart

@@ -91,7 +91,7 @@ class _QuizBodyState extends State<QuizBody> {
           listenWhen: (previous, current) =>
               previous.quiz.id != current.quiz.id,
           listener: (_, state) {
-            if (state.quiz.isNone) {
+            if (state.quiz.isEmpty) {
               context.read<VoidCallback>()();
             }
           },
@@ -110,7 +110,7 @@ class _QuizBodyState extends State<QuizBody> {
           listenWhen: (previous, current) =>
               previous.selectedOption != current.selectedOption,
           listener: (_, state) {
-            if (state.selectedOption.isNotNone) {
+            if (state.selectedOption.isNotEmpty) {
               context.showScrollControlledBottomSheet<void>(
                 builder: (_) {
                   return BlocProvider.value(

lib/topics/cubit/topics_state.dart

@@ -4,10 +4,10 @@ enum TopicsStatus { initial, loading, empty, loaded, failure }
 
 final class TopicsState extends Equatable {
   const TopicsState._({
-    this.user = User.none,
+    this.user = User.empty,
     this.status = TopicsStatus.initial,
     this.topics = const [],
-    this.failure = TopicsFailure.none,
+    this.failure = TopicsFailure.empty,
   });
 
   const TopicsState.initial() : this._();

lib/topics/view/topics_flow.dart

@@ -27,7 +27,7 @@ List<Page<dynamic>> onGenerateTopicsPages(
 
 class TopicsFlowState extends Equatable {
   const TopicsFlowState._({
-    this.selectedTopic = Topic.none,
+    this.selectedTopic = Topic.empty,
     this.selectedQuizId = '',
   });
 
@@ -36,7 +36,7 @@ class TopicsFlowState extends Equatable {
   final Topic selectedTopic;
   final String selectedQuizId;
 
-  bool get hasTopicSelected => selectedTopic.isNotNone;
+  bool get hasTopicSelected => selectedTopic.isNotEmpty;
   bool get hasQuizSelected => selectedQuizId.isNotEmpty;
 
   @override

packages/app_core/lib/src/failure.dart

@@ -1,3 +1,5 @@
 abstract class Failure implements Exception {
   const Failure();
+
+  bool get needsReauthentication => false;
 }

packages/quizzes_repository/lib/src/failures.dart

@@ -5,11 +5,11 @@ class QuizzesFailure extends Failure {
 
   factory QuizzesFailure.fromGetQuiz() => const GetQuizFailure();
 
-  static const none = QuizzesNoFailure();
+  static const empty = EmptyQuizzesFailure();
 }
 
-class QuizzesNoFailure extends QuizzesFailure {
-  const QuizzesNoFailure() : super._();
+class EmptyQuizzesFailure extends QuizzesFailure {
+  const EmptyQuizzesFailure() : super._();
 }
 
 class GetQuizFailure extends QuizzesFailure {

packages/quizzes_repository/lib/src/models/models.dart

@@ -17,7 +17,7 @@ class Option extends Equatable {
   final String detail;
   final bool? correct;
 
-  static const none = Option(value: '', detail: '', correct: null);
+  static const empty = Option(value: '', detail: '', correct: null);
 
   @override
   List<Object?> get props => [value, detail, correct];
@@ -26,7 +26,7 @@ class Option extends Equatable {
 }
 
 extension OptionExtensions on Option {
-  bool get isNotNone => this != Option.none;
+  bool get isNotEmpty => this != Option.empty;
 }
 
 @JsonSerializable(createToJson: false)
@@ -40,7 +40,7 @@ class Question extends Equatable {
   @JsonKey(defaultValue: <Option>[])
   final List<Option> options;
 
-  static const none = Question(text: '', options: []);
+  static const empty = Question(text: '', options: []);
 
   @override
   List<Object?> get props => [text, options];
@@ -69,7 +69,7 @@ class Quiz extends Equatable {
   @JsonKey(defaultValue: <Question>[])
   final List<Question> questions;
 
-  static const none = Quiz(
+  static const empty = Quiz(
     id: '',
     title: '',
     description: '',
@@ -92,5 +92,5 @@ class Quiz extends Equatable {
 }
 
 extension QuizExtensions on Quiz {
-  bool get isNone => this == Quiz.none;
+  bool get isEmpty => this == Quiz.empty;
 }

packages/quizzes_repository/lib/src/quizzes_repository.dart

@@ -15,7 +15,7 @@ class QuizzesRepository {
       if (doc.exists) {
         return Quiz.fromJson(doc.data()!);
       }
-      return Quiz.none;
+      return Quiz.empty;
     } on FirebaseException {
       throw QuizzesFailure.fromGetQuiz();
     }

packages/topics_repository/lib/src/failures.dart

@@ -5,11 +5,11 @@ class TopicsFailure extends Failure {
 
   factory TopicsFailure.fromGetTopics() => const GetTopicsFailure();
 
-  static const none = TopicsNoFailure();
+  static const empty = EmptyTopicsFailure();
 }
 
-class TopicsNoFailure extends TopicsFailure {
-  const TopicsNoFailure() : super._();
+class EmptyTopicsFailure extends TopicsFailure {
+  const EmptyTopicsFailure() : super._();
 }
 
 class GetTopicsFailure extends TopicsFailure {

packages/topics_repository/lib/src/models/models.dart

@@ -16,7 +16,7 @@ class Quiz extends Equatable {
   final String title;
   final String description;
 
-  static const none = Quiz(
+  static const empty = Quiz(
     id: '',
     title: '',
     description: '',
@@ -46,7 +46,7 @@ class Topic extends Equatable {
   final String imageName;
   final List<Quiz> quizzes;
 
-  static const none = Topic(
+  static const empty = Topic(
     id: '',
     title: '',
     description: '',
@@ -61,8 +61,8 @@ class Topic extends Equatable {
 }
 
 extension TopicExtensions on Topic {
-  bool get isNone => this == Topic.none;
-  bool get isNotNone => !isNone;
+  bool get isEmpty => this == Topic.empty;
+  bool get isNotEmpty => !isEmpty;
 
   int get totalQuizzes => quizzes.length;
 

packages/user_repository/lib/src/failures.dart

@@ -13,19 +13,18 @@ class UserFailure extends Failure {
   factory UserFailure.fromSignInWithAppleNotSupported() =>
       const AppleSignInNotSupportedFailure();
 
-  static const none = UserNoFailure();
-
-  bool get requiresReauthentication {
-    return this is AuthUserChangesFailure;
-  }
+  static const empty = EmptyUserFailure();
 }
 
-class UserNoFailure extends UserFailure {
-  const UserNoFailure() : super._();
+class EmptyUserFailure extends UserFailure {
+  const EmptyUserFailure() : super._();
 }
 
 class AuthUserChangesFailure extends UserFailure {
   const AuthUserChangesFailure() : super._();
+
+  @override
+  bool get needsReauthentication => true;
 }
 
 class SignOutFailure extends UserFailure {