Merge pull request #251 from osociety/codecov-fix

Fix codecov issue

Author: git-elliot

.github/workflows/dart.yml

@@ -45,11 +45,43 @@ jobs:
       - name: Analyze project source
         run: dart analyze --fatal-infos 
 
-      # Your project will need to have tests in test/ and a dependency on
-      # package:test for this step to succeed. Note that Flutter projects will
-      # want to change this to 'flutter test'.
       - name: Run tests
-        run: dart test
+        run: dart test --coverage="./coverage"
+
+      - name: Install coverage tools
+        if: runner.os == 'Linux'
+        run: dart pub global activate coverage
+
+      - name: Install lcov
+        if: runner.os == 'Linux'
+        run: sudo apt-get update && sudo apt-get install -y lcov
+
+      - name: Format Coverage
+        if: runner.os == 'Linux'
+        run: dart pub global run coverage:format_coverage --packages=.dart_tool/package_config.json --report-on=lib --lcov -o ./coverage/coverage.lcov -i ./coverage
+
+      - name: Excluded generated files
+        if: runner.os == 'Linux'
+        run: lcov --remove ./coverage/coverage.lcov '*/lib/src/*.g.dart' '*/lib/src/models/drift/*' -o ./coverage/coverage.lcov
+
+      - name: Generate HTML coverage report
+        if: runner.os == 'Linux'
+        run: genhtml -o ./coverage/report ./coverage/coverage.lcov
+
+      - name: Upload artifact
+        if: runner.os == 'Linux'
+        uses: actions/upload-pages-artifact@v3
+        with:
+          # Upload entire repository
+          path: './coverage/report'
 
+      - name: Upload coverage to Codecov
+        if: runner.os == 'Linux'
+        uses: codecov/codecov-action@v3
+        with:
+          files: ./coverage/coverage.lcov
+          flags: unittests
+          name: codecov-umbrella
+
       - name: Check Publish Warnings
         run: dart pub publish --dry-run

test/network_tools_test.dart

@@ -310,18 +310,6 @@ Future<void> main() async {
     });
   });
 
-  group("Testing mdns scanner group", () {
-    test('Running searchMdnsDevices tests', () async {
-      final mdnsDevices = await MdnsScannerService.instance.searchMdnsDevices();
-      expectLater(mdnsDevices, isA<List<ActiveHost>>());
-      //todo: mdnsDevices are empty in GHA, open one to be discoverable
-      // expectLater(
-      //   mdnsDevices,
-      //   contains(predicate<ActiveHost>((host) => host.address == myOwnHost)),
-      // );
-    });
-  });
-
   tearDownAll(() {
     server.close();
   });