Merge dev into main

.github/workflows/flutter_release.yml

@@ -40,9 +40,6 @@ jobs:
       - name: Download pub dependencies
         run: flutter pub get
 
-      - name: Upgrade pub dependencies
-        run: flutter pub upgrade
-
       - name: Run build_runner
         run: dart run build_runner build --delete-conflicting-outputs
 
@@ -150,9 +147,6 @@ jobs:
       - name: Download pub dependencies
         run: flutter pub get
 
-      - name: Upgrade pub dependencies
-        run: flutter pub upgrade
-
       - name: Run build_runner
         run: dart run build_runner build --delete-conflicting-outputs
 
@@ -204,9 +198,6 @@ jobs:
       - name: Download pub dependencies
         run: flutter pub get
 
-      - name: Upgrade pub dependencies
-        run: flutter pub upgrade
-
       - name: Run build_runner
         run: dart run build_runner build --delete-conflicting-outputs
 

.github/workflows/flutter_test.yml

@@ -51,10 +51,14 @@ jobs:
     name: 'Flutter Analyze + Test'
     needs: changes
     if: ${{ needs.changes.outputs.lib == 'true' || needs.changes.outputs.test == 'true' || needs.changes.outputs.yaml == 'true' || needs.changes.outputs.android == 'true' || needs.changes.outputs.linux == 'true' || needs.changes.outputs.macos == 'true' || needs.changes.outputs.windows == 'true' }}
-    runs-on: ubuntu-latest
+    runs-on: macos-latest
     steps:
       - name: Checkout
         uses: actions/[email protected]
+      - name: Extract branch name
+        shell: bash
+        run: echo "branch=${GITHUB_HEAD_REF:-${GITHUB_REF#refs/heads/}}" >> $GITHUB_OUTPUT
+        id: extract_branch
       - name: Setup Java JDK
         uses: actions/[email protected]
         with:
@@ -85,14 +89,15 @@ jobs:
             build-runner-
       - name: Download pub dependencies
         run: flutter pub get
-      - name: Upgrade pub dependencies
-        run: flutter pub upgrade
       - name: Run build_runner
         run: flutter pub run build_runner build
       - name: Run analyzer
         run: flutter analyze
       - name: Run tests
         run: flutter test
+      - name: Run integration tests
+        if: ${{ steps.extract_branch.outputs.branch == 'main' }}
+        run: flutter test integration_test -d macos
 
   android-linux-build:
     name: 'Build Android + Linux'
@@ -130,8 +135,6 @@ jobs:
             build-runner-
       - name: Download pub dependencies
         run: flutter pub get
-      - name: Upgrade pub dependencies
-        run: flutter pub upgrade
       - name: Run build_runner
         run: flutter pub run build_runner build
       - name: Build Android
@@ -186,8 +189,6 @@ jobs:
             ${{ runner.os }}-pods-
       - name: Download pub dependencies
         run: flutter pub get
-      - name: Upgrade pub dependencies
-        run: flutter pub upgrade
       - name: Run build_runner
         run: flutter pub run build_runner build
       - name: Build macos
@@ -224,8 +225,6 @@ jobs:
             build-runner-
       - name: Download pub dependencies
         run: flutter pub get
-      - name: Upgrade pub dependencies
-        run: flutter pub upgrade
       - name: Run build_runner
         run: flutter pub run build_runner build
       - name: Build windows

generate_coverage.sh

@@ -0,0 +1,2 @@
+genhtml coverage/lcov.info -o coverage/html
+open coverage/html/index.html

integration_test/app_test.dart

@@ -0,0 +1,112 @@
+// This is a basic Flutter widget test.
+//
+// To perform an interaction with a widget in your test, use the WidgetTester
+// utility that Flutter provides. For example, you can send tap and scroll
+// gestures. You can also use WidgetTester to find child widgets in the widget
+// tree, read text, and verify that the values of widget properties are correct.
+
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:integration_test/integration_test.dart';
+import 'package:network_tools_flutter/network_tools_flutter.dart';
+import 'package:path_provider/path_provider.dart';
+import 'package:vernet/injection.dart';
+import 'package:vernet/main.dart';
+import 'package:vernet/ui/adaptive/adaptive_list.dart';
+import 'package:vernet/values/keys.dart';
+
+void main() {
+  IntegrationTestWidgetsFlutterBinding.ensureInitialized();
+  setUpAll(() async {
+    configureDependencies(Env.test);
+    final appDocDirectory = await getApplicationDocumentsDirectory();
+    await configureNetworkToolsFlutter(appDocDirectory.path);
+  });
+
+  group('host scanner end-to-end test', () {
+    testWidgets('just test if app is able to launch and display homepage',
+        (tester) async {
+      // Load app widget.
+      await tester.pumpWidget(const MyApp(true));
+      await tester.pumpAndSettle();
+
+      // Verify that there are 4 widgets at homepage
+      expect(find.bySubtype<AdaptiveListTile>(), findsAtLeastNWidgets(4));
+    });
+
+    testWidgets('tap on the scan for devices button, verify device found',
+        (tester) async {
+      // Load app widget.
+      await tester.pumpWidget(const MyApp(true));
+      await tester.pumpAndSettle();
+
+      // Verify that there are 4 widgets at homepage
+      expect(find.bySubtype<AdaptiveListTile>(), findsAtLeastNWidgets(4));
+
+      // Finds the scan for devices button to tap on.
+      final devicesButton = find.byKey(Keys.scanForDevicesButton);
+
+      // Emulate a tap on the button.
+      await tester.tap(devicesButton);
+      await tester.pump();
+      expect(find.byType(AdaptiveListTile), findsAny);
+      await tester.pumpAndSettle(const Duration(seconds: 20));
+      await tester.pump();
+      expect(find.byType(AdaptiveListTile), findsAtLeast(2));
+
+      final routerIconButton = find.byKey(Keys.routerOrGatewayTileIconButton);
+
+      await tester.tap(routerIconButton);
+      await tester.pumpAndSettle();
+      expect(find.byType(AppBar), findsOne);
+
+      final radioButton = find.byKey(Keys.rangePortScanRadioButton);
+      await tester.tap(radioButton);
+      await tester.pumpAndSettle();
+
+      final portChip = find.byKey(Keys.knownPortChip);
+
+      await tester.tap(portChip);
+      await tester.pumpAndSettle();
+
+      final portScanButton = find.byKey(Keys.portScanButton);
+      await tester.tap(portScanButton);
+      await tester.pumpAndSettle();
+    });
+
+    testWidgets('port scan returns open port for google.com', (tester) async {
+      // Load app widget.
+      await tester.pumpWidget(const MyApp(true));
+      await tester.pumpAndSettle();
+
+      // Verify that there are 4 widgets at homepage
+      expect(find.bySubtype<AdaptiveListTile>(), findsAtLeastNWidgets(4));
+
+      // Finds the open ports button to tap on.
+      final scanForOpenPortsButton = find.byKey(Keys.scanForOpenPortsButton);
+
+      await tester.tap(scanForOpenPortsButton);
+      await tester.pumpAndSettle();
+      expect(find.byType(AppBar), findsOne);
+
+      final googleChip = find.byKey(Keys.googleChip);
+      await tester.tap(googleChip);
+      await tester.pumpAndSettle();
+
+      final radioButton = find.byKey(Keys.rangePortScanRadioButton);
+      await tester.tap(radioButton);
+      await tester.pumpAndSettle();
+
+      final portChip = find.byKey(Keys.knownPortChip);
+      await tester.tap(portChip);
+      await tester.pumpAndSettle();
+
+      final portScanButton = find.byKey(Keys.portScanButton);
+      await tester.tap(portScanButton);
+      await tester.pumpAndSettle(const Duration(seconds: 20));
+      await tester.pump();
+      //TODO: not passing in github actions
+      expect(find.byType(AdaptiveListTile), findsAny);
+    });
+  });
+}

lib/helper/app_settings.dart

@@ -113,4 +113,8 @@ class AppSettings {
             _customSubnet;
     debugPrint("Custom Subnet : $_customSubnet");
   }
+
+  Future<bool> clearAll() async {
+    return (await SharedPreferences.getInstance()).clear();
+  }
 }

lib/pages/home_page.dart

@@ -19,6 +19,7 @@ import 'package:vernet/repository/notification_service.dart';
 import 'package:vernet/services/impls/device_scanner_service.dart';
 import 'package:vernet/ui/adaptive/adaptive_list.dart';
 import 'package:vernet/ui/custom_tile.dart';
+import 'package:vernet/values/keys.dart';
 import 'package:vernet/values/strings.dart';
 
 class HomePage extends StatefulWidget {
@@ -168,6 +169,7 @@ class _WifiDetailState extends State<HomePage> {
                               width: 4,
                             ),
                             ElevatedButton(
+                              key: Keys.scanForDevicesButton,
                               onPressed: () {
                                 Navigator.push(
                                   context,
@@ -192,7 +194,7 @@ class _WifiDetailState extends State<HomePage> {
                 } else if (snapshot.hasError) {
                   return const Text("Unable to fetch WiFi details");
                 } else {
-                  return const CircularProgressIndicator.adaptive();
+                  return const Text('Loading...');
                 }
               },
             ),
@@ -221,6 +223,7 @@ class _WifiDetailState extends State<HomePage> {
                       ),
                       const SizedBox(width: 10),
                       ElevatedButton.icon(
+                        key: Keys.scanForOpenPortsButton,
                         onPressed: () {
                           Navigator.push(
                             context,

lib/pages/host_scan_page/widgets/host_scan_widget.dart

@@ -6,6 +6,9 @@ import 'package:vernet/models/isar/device.dart';
 import 'package:vernet/pages/host_scan_page/host_scan_bloc/host_scan_bloc.dart';
 import 'package:vernet/pages/network_troubleshoot/port_scan_page.dart';
 import 'package:vernet/ui/adaptive/adaptive_list.dart';
+import 'package:vernet/values/keys.dart';
+import 'package:vernet/values/strings.dart';
+import 'package:vernet/values/tooltip_messages.dart';
 
 //TODO: Device doesn't refresh when active scan going on
 class HostScanWidget extends StatelessWidget {
@@ -29,7 +32,7 @@ class HostScanWidget extends StatelessWidget {
                     Text(
                       appSettings.gatewayIP.isNotEmpty
                           ? 'Searching for devices in ${appSettings.gatewayIP} network'
-                          : 'Searching for devices in your local network',
+                          : StringValue.loadingDevicesMessage,
                       textAlign: TextAlign.center,
                     ),
                   ],
@@ -68,15 +71,9 @@ class HostScanWidget extends StatelessWidget {
             textAlign: TextAlign.center,
           ),
           trailing: loading
-              ? const Padding(
-                  padding: EdgeInsets.all(8.0),
-                  child: SizedBox(
-                    height: 25.0,
-                    width: 25.0,
-                    child: Center(child: CircularProgressIndicator.adaptive()),
-                  ),
-                )
+              ? const SizedBox()
               : IconButton(
+                  key: Keys.rescanIconButton,
                   onPressed: () {
                     context
                         .read<HostScanBloc>()
@@ -97,7 +94,10 @@ class HostScanWidget extends StatelessWidget {
                   '${host.internetAddress}, ${host.macAddress ?? ''}',
                 ),
                 trailing: IconButton(
-                  tooltip: 'Scan open ports for this target',
+                  key: host.deviceMake == 'Router/Gateway'
+                      ? Keys.routerOrGatewayTileIconButton
+                      : null,
+                  tooltip: TooltipMessages.currentDevicePortScan,
                   icon: const Icon(Icons.radar),
                   onPressed: () {
                     Navigator.push(