fix(firebase_ui_auth): sendPasswordResetEmail should not be executed when isLoading=true (#315)

Co-authored-by: russellwheatley <[email protected]>

Author: kzrnm

Diff for: packages/firebase_ui_auth/lib/src/views/forgot_password_view.dart

@@ -54,6 +54,10 @@ class _ForgotPasswordViewState extends State<ForgotPasswordView> {
   fba.FirebaseAuthException? exception;
 
   Future<void> _submit(String email) async {
+    if (isLoading) {
+      return;
+    }
+
     setState(() {
       exception = null;
       isLoading = true;

Diff for: packages/firebase_ui_auth/test/views/forgot_password_view_test.dart

@@ -4,6 +4,7 @@
 
 import 'package:firebase_auth/firebase_auth.dart' as fba;
 import 'package:firebase_ui_auth/firebase_ui_auth.dart';
+import 'package:firebase_ui_shared/firebase_ui_shared.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:mockito/mockito.dart';
@@ -53,6 +54,10 @@ void main() {
       when(auth.sendPasswordResetEmail(email: '[email protected]')).thenAnswer(
         (_) => Future.value(),
       );
+
+      when(auth.sendPasswordResetEmail(email: '[email protected]')).thenAnswer(
+        (_) => Future.delayed(const Duration(milliseconds: 200)),
+      );
     });
 
     testWidgets('shows error if sendPasswordResetEmail failed', (tester) async {
@@ -81,5 +86,20 @@ void main() {
         expect(find.byType(ErrorText), findsNothing);
       },
     );
+
+    testWidgets("doesn't sendPasswordResetEmail on loading", (tester) async {
+      await tester.pumpWidget(widget);
+
+      final input = find.byType(TextField);
+      await tester.enterText(input, '[email protected]');
+
+      final button = find.byType(LoadingButton);
+      await tester.tap(button);
+      await tester.tap(button);
+      await tester.pumpAndSettle();
+
+      expect(find.byType(ErrorText), findsNothing);
+      verify(auth.sendPasswordResetEmail(email: '[email protected]')).called(1);
+    });
   });
 }