466 testing gui

takeoff/takeoff_gui/generate_coverage.ps1

@@ -0,0 +1,8 @@
+flutter clean
+flutter pub get
+flutter pub run build_runner build --delete-conflicting-outputs
+flutter test --coverage
+remove_from_coverage -f coverage/lcov.info -r '.g.dart$'
+remove_from_coverage -f coverage/lcov.info -r '_libw.dart$'
+remove_from_coverage -f coverage/lcov.info -r 'options.dart$'
+remove_from_coverage -f coverage/lcov.info -r 'mock_projects.dart$'

takeoff/takeoff_gui/lib/common/custom_button.dart

@@ -15,21 +15,15 @@ class CustomButton extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    return ElevatedButton(
+    return ElevatedButton.icon(
       onPressed: onPressed == null ? null : () => onPressed!(),
       style: ElevatedButton.styleFrom(
         backgroundColor: color,
         minimumSize: const Size(150, 50),
         maximumSize: const Size(170, 50),
       ),
-      child: Row(
-        mainAxisAlignment: MainAxisAlignment.center,
-        children: [
-          Icon(icon),
-          const SizedBox(width: 10),
-          Text(text),
-        ],
-      ),
+      icon: Icon(icon),
+      label: Text(text),
     );
   }
 }

takeoff/takeoff_gui/lib/common/monitor/pages/monitor_dialog.dart

@@ -30,20 +30,22 @@ class MonitorDialog extends StatelessWidget {
                 child: Column(
                   children: [
                     Observer(
-                      builder: (_) => Row(
-                        mainAxisAlignment: MainAxisAlignment.spaceEvenly,
-                        children: [
-                          Text(
-                              controller.hasFinished
-                                  ? AppLocalizations.of(context)!
-                                      .projectCreationFinishedMessage
-                                  : AppLocalizations.of(context)!
-                                      .creatingProjectMessage,
-                              style: const TextStyle(fontSize: 30)),
-                          if (!controller.hasFinished)
-                            const CircularProgressIndicator(),
-                        ],
-                      ),
+                      builder: (_) => SizedBox(
+                          //mainAxisAlignment: MainAxisAlignment.spaceEvenly,
+                          child: controller.hasFinished
+                              ? Text(
+                                  AppLocalizations.of(context)!
+                                      .projectCreationFinishedMessage,
+                                  style: const TextStyle(fontSize: 30))
+                              : Row(
+                                  children: [
+                                    Text(
+                                        AppLocalizations.of(context)!
+                                            .creatingProjectMessage,
+                                        style: const TextStyle(fontSize: 30)),
+                                    const CircularProgressIndicator(),
+                                  ],
+                                )),
                     ),
                     const SizedBox(height: 50),
                     SizedBox(
@@ -98,6 +100,7 @@ class MonitorDialog extends StatelessWidget {
         });
   }
 
+  @visibleForTesting
   _generateDialog(GuiMessage message, BuildContext context) {
     switch (message.type) {
       case MessageType.info:

takeoff/takeoff_gui/lib/common/monitor/pages/user_interaction_dialog.dart

@@ -35,15 +35,11 @@ class _UserInteractionDialogState extends State<UserInteractionDialog> {
         padding: const EdgeInsets.symmetric(horizontal: 30.0),
         child: SingleChildScrollView(
           child: Column(
+            mainAxisAlignment: MainAxisAlignment.spaceEvenly,
             children: [
-              Row(
-                mainAxisAlignment: MainAxisAlignment.spaceEvenly,
-                children: [
-                  Text(
-                    AppLocalizations.of(context)!.followStepsMessage,
-                    style: const TextStyle(fontSize: 30),
-                  ),
-                ],
+              Text(
+                AppLocalizations.of(context)!.followStepsMessage,
+                style: const TextStyle(fontSize: 30),
               ),
               const SizedBox(height: 50),
               SelectableText(

takeoff/takeoff_gui/lib/features/create/controllers/create_controller.dart

@@ -64,7 +64,7 @@ abstract class _CreateController with Store {
     repoProvider = CloudProvidersComb.cicd[cloudProvider]![0];
   }
 
-  void createProject() async {
+  void createProject() {
     monitorController.monitorProcess(() async => formController.create(
         backendLanguage: backendLanguage,
         backendVersion: backendVersion,

takeoff/takeoff_gui/lib/features/create/widgets/project_forms/google_form.dart

@@ -50,33 +50,25 @@ class GoogleForm extends StatelessWidget {
           ],
         ),
         const SizedBox(height: 15),
-        Row(
-          children: [
-            Expanded(
-              child: Observer(builder: (_) {
-                if (controller.region.isEmpty) {
-                  controller.region = googleCloudRegions.first;
-                }
+        Observer(builder: (_) {
+          if (controller.region.isEmpty) {
+            controller.region = googleCloudRegions.first;
+          }
 
-                return DropdownButtonFormField(
-                    decoration: InputDecoration(
-                      label: Text(AppLocalizations.of(context)!.region),
-                      border: const OutlineInputBorder(),
-                    ),
-                    items: googleCloudRegions
-                        .map((e) => DropdownMenuItem(
-                              value: e,
-                              child: Text(e),
-                            ))
-                        .toList(),
-                    value: controller.region,
-                    onChanged: (value) => controller.region = value!);
-              }),
-            ),
-            const SizedBox(width: 20),
-            Expanded(child: Container()),
-          ],
-        ),
+          return DropdownButtonFormField(
+              decoration: InputDecoration(
+                label: Text(AppLocalizations.of(context)!.region),
+                border: const OutlineInputBorder(),
+              ),
+              items: googleCloudRegions
+                  .map((e) => DropdownMenuItem(
+                        value: e,
+                        child: Text(e),
+                      ))
+                  .toList(),
+              value: controller.region,
+              onChanged: (value) => controller.region = value!);
+        }),
       ],
     );
   }

takeoff/takeoff_gui/lib/features/details/pages/project_details.dart

@@ -16,20 +16,22 @@ class ProjectDetails extends StatelessWidget {
       body: Row(
         children: [
           SideBar(),
-          Padding(
-            padding: const EdgeInsets.all(20.0),
-            child: Column(
-              crossAxisAlignment: CrossAxisAlignment.start,
-              children: [
-                // TODO add dropdown to select project here
-                const SizedBox(height: 40),
-                Text(
-                  "${project.name} ${AppLocalizations.of(context)!.projectResources}",
-                  style: const TextStyle(fontSize: 30),
-                ),
-                const SizedBox(height: 40),
-                ResourceDetails(),
-              ],
+          Expanded(
+            child: Padding(
+              padding: const EdgeInsets.all(20.0),
+              child: Column(
+                crossAxisAlignment: CrossAxisAlignment.start,
+                children: [
+                  // TODO add dropdown to select project here
+                  const SizedBox(height: 40),
+                  Text(
+                    "${project.name} ${AppLocalizations.of(context)!.projectResources}",
+                    style: const TextStyle(fontSize: 30),
+                  ),
+                  const SizedBox(height: 40),
+                  ResourceDetails(),
+                ],
+              ),
             ),
           )
         ],

takeoff/takeoff_gui/lib/features/details/widgets/clean_dialog.dart

@@ -15,7 +15,7 @@ class CleanDialog extends StatelessWidget {
       backgroundColor: Colors.red.shade200,
       title: Text(
         AppLocalizations.of(context)!.removeProject,
-        style: TextStyle(fontSize: 30),
+        style: const TextStyle(fontSize: 30),
       ),
       content: SingleChildScrollView(
           child: Padding(

takeoff/takeoff_gui/lib/features/quickstart/controllers/quickstart_controller.dart

@@ -26,12 +26,13 @@ abstract class _QuickstartController with Store {
   @computed
   bool get isValidForm => billingAccount.isNotEmpty && region.isNotEmpty;
 
-  void createWayat() {
-    monitorController.monitorProcess(() async => await facade.quickstartWayat(
-        billingAccount: billingAccount,
-        googleCloudRegion: region,
-        outputStream: monitorController.outputChannel,
-        inputStream: monitorController.inputChannel));
+  Future<void> createWayat() async {
+    await monitorController.monitorProcess(() async =>
+        await facade.quickstartWayat(
+            billingAccount: billingAccount,
+            googleCloudRegion: region,
+            outputStream: monitorController.outputChannel,
+            inputStream: monitorController.inputChannel));
   }
 
   @action

takeoff/takeoff_gui/test/common/error_loading_test.dart

@@ -0,0 +1,39 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:flutter_gen/gen_l10n/app_localizations.dart';
+import 'package:takeoff_gui/common/error_loading_page.dart';
+
+// @GenerateMocks([ClassToMock])
+void main() async {
+  setUpAll(() async {});
+
+  Widget createApp(Widget body) {
+    return MaterialApp(
+      onGenerateTitle: (context) => AppLocalizations.of(context)!.appTitle,
+      localizationsDelegates: AppLocalizations.localizationsDelegates,
+      supportedLocales: AppLocalizations.supportedLocales,
+      localeResolutionCallback:
+          (Locale? locale, Iterable<Locale> supportedLocales) {
+        for (Locale supportedLocale in supportedLocales) {
+          if (supportedLocale.languageCode == locale?.languageCode) {
+            return supportedLocale;
+          }
+        }
+        return const Locale("en", "US");
+      },
+      home: Scaffold(
+        body: body,
+      ),
+    );
+  }
+
+  testWidgets('Widget error widget', (tester) async {
+    // Avoid overflow due to test conditions
+    FlutterError.onError = null;
+    String messageText = "TestText";
+    IconData icon = Icons.warning_amber_outlined;
+    await tester.pumpWidget(createApp(ErrorLoadingPage(message: messageText)));
+    expect(find.text(messageText), findsOneWidget);
+    expect(find.byIcon(icon), findsOneWidget);
+  });
+}

takeoff/takeoff_gui/test/common/icon_text_button_test.dart

@@ -0,0 +1,52 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:takeoff_gui/common/icon_text_button.dart';
+import 'package:flutter_gen/gen_l10n/app_localizations.dart';
+
+// @GenerateMocks([ClassToMock])
+void main() async {
+  setUpAll(() async {});
+
+  Widget createApp(Widget body) {
+    return MaterialApp(
+      onGenerateTitle: (context) => AppLocalizations.of(context)!.appTitle,
+      localizationsDelegates: AppLocalizations.localizationsDelegates,
+      supportedLocales: AppLocalizations.supportedLocales,
+      localeResolutionCallback:
+          (Locale? locale, Iterable<Locale> supportedLocales) {
+        for (Locale supportedLocale in supportedLocales) {
+          if (supportedLocale.languageCode == locale?.languageCode) {
+            return supportedLocale;
+          }
+        }
+        return const Locale("en", "US");
+      },
+      home: Scaffold(
+        body: body,
+      ),
+    );
+  }
+
+  testWidgets('Widget test', (tester) async {
+    // Avoid overflow due to test conditions
+    FlutterError.onError = null;
+    String buttonText = "TestText";
+    IconData icon = Icons.account_circle;
+    await tester.pumpWidget(createApp(
+      IconTextButton(
+        text: buttonText,
+        icon: icon,
+        onPressed: () => true,
+      ),
+    ));
+    await tester.pumpWidget(createApp(IconTextButton(
+      icon: icon,
+      text: buttonText,
+    )));
+
+    await tester.pumpAndSettle();
+
+    expect(find.text(buttonText), findsOneWidget);
+    expect(find.byIcon(icon), findsOneWidget);
+  });
+}

takeoff/takeoff_gui/test/common/loading_page_test.dart

@@ -0,0 +1,42 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:flutter_gen/gen_l10n/app_localizations.dart';
+import 'package:takeoff_gui/common/loading_page.dart';
+
+// @GenerateMocks([ClassToMock])
+void main() async {
+  setUpAll(() async {});
+
+  Widget createApp(Widget body) {
+    return MaterialApp(
+      onGenerateTitle: (context) => AppLocalizations.of(context)!.appTitle,
+      localizationsDelegates: AppLocalizations.localizationsDelegates,
+      supportedLocales: AppLocalizations.supportedLocales,
+      localeResolutionCallback:
+          (Locale? locale, Iterable<Locale> supportedLocales) {
+        for (Locale supportedLocale in supportedLocales) {
+          if (supportedLocale.languageCode == locale?.languageCode) {
+            return supportedLocale;
+          }
+        }
+        return const Locale("en", "US");
+      },
+      home: Scaffold(
+        body: body,
+      ),
+    );
+  }
+
+  testWidgets('Widget loading page', (tester) async {
+    // Avoid overflow due to test conditions
+    String text =
+        "Launching the app, please while checking the requirements...";
+    ImageProvider image = const AssetImage("assets/gifs/rocket.gif");
+    await tester.pumpWidget(createApp(LoadingPage(
+      message: text,
+    )));
+    expect(find.text(text), findsOneWidget);
+    expect(find.byType(Image), findsOneWidget);
+    expect(find.widgetWithImage(Image, image), findsNothing);
+  });
+}