fix(ui): set the default variant of LoadingButton to outlined (#8443) (…

…#8545)

Co-authored-by: andrastimar <andrastimardev@gmail.com>

packages/flutterfire_ui/lib/src/auth/widgets/internal/loading_button.dart

@@ -44,7 +44,7 @@ class LoadingButton extends StatelessWidget {
     this.isLoading = false,
     this.icon,
     this.color,
-    this.variant = ButtonVariant.text,
+    this.variant = ButtonVariant.outlined,
   }) : super(key: key);
 
   @override

packages/flutterfire_ui/pubspec.yaml

@@ -32,6 +32,7 @@ dependencies:
   twitter_login: ^4.2.2
 
 dev_dependencies:
+  firebase_auth_mocks: ^0.8.4
   flutter_test:
     sdk: flutter
 

packages/flutterfire_ui/test/auth/widgets/email_form_test.dart

@@ -0,0 +1,53 @@
+import 'package:firebase_auth_mocks/firebase_auth_mocks.dart';
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:flutterfire_ui/auth.dart';
+import 'package:flutterfire_ui/src/auth/widgets/internal/loading_button.dart';
+import 'package:flutterfire_ui/src/auth/widgets/internal/universal_button.dart';
+
+import '../../test_utils.dart';
+
+void main() {
+  group('EmailForm', () {
+    late Widget widget;
+
+    setUp(() {
+      widget = TestMaterialApp(
+        child: EmailForm(
+          auth: MockFirebaseAuth(),
+          action: AuthAction.signIn,
+        ),
+      );
+    });
+
+    testWidgets('Has a Sign in button of outlined variant', (tester) async {
+      await tester.pumpWidget(widget);
+      expect(
+        find.descendant(
+          of: find.byWidgetPredicate(
+            (widget) =>
+                widget is LoadingButton &&
+                widget.variant == ButtonVariant.outlined,
+          ),
+          matching: find.text('Sign in'),
+        ),
+        findsOneWidget,
+      );
+    });
+
+    testWidgets('Has a Forgot password button of text variant', (tester) async {
+      await tester.pumpWidget(widget);
+      expect(
+        find.descendant(
+          of: find.byWidgetPredicate(
+            (widget) =>
+                widget is UniversalButton &&
+                widget.variant == ButtonVariant.text,
+          ),
+          matching: find.text('Forgot password?'),
+        ),
+        findsOneWidget,
+      );
+    });
+  });
+}

packages/flutterfire_ui/test/test_utils.dart

@@ -0,0 +1,16 @@
+import 'package:flutter/material.dart';
+
+class TestMaterialApp extends StatelessWidget {
+  final Widget child;
+
+  const TestMaterialApp({Key? key, required this.child}) : super(key: key);
+
+  @override
+  Widget build(BuildContext context) {
+    return MaterialApp(
+      home: Scaffold(
+        body: child,
+      ),
+    );
+  }
+}