fix: add tests and ensure that the sub processes can be mocked out.

Author: Nexushunter

.github/workflows/code_quality.yml

@@ -36,6 +36,9 @@ jobs:
         run: dart pub get
       - name: Validate formatting
         run: dart format ./ --set-exit-if-changed
+      - name: Generate Mocks
+        if: ${{ matrix.work_dir == 'openapi-generator' }}
+        run: dart run build_runner build --delete-conflicting-outputs
       - name: Run analyzer
         run: dart analyze --fatal-warnings
       - name: Run tests

openapi-generator/lib/src/models/command.dart

@@ -1,3 +1,7 @@
+import 'dart:io';
+
+import 'package:openapi_generator/src/determine_flutter_project_status.dart';
+import 'package:openapi_generator/src/gen_on_spec_changes.dart';
 import 'package:openapi_generator_annotations/openapi_generator_annotations.dart';
 
 /// Creates a representation of a cli request for Flutter or Dart.
@@ -29,3 +33,52 @@ class Command {
           arguments,
         );
 }
+
+/// CommandRunner provides an abstraction layer to external functions / processes.
+class CommandRunner {
+  const CommandRunner();
+
+  Future<ProcessResult> runCommand({
+    required Command command,
+    required String workingDirectory,
+  }) async =>
+      Process.run(
+        command.executable,
+        command.arguments,
+        workingDirectory: workingDirectory,
+        runInShell: Platform.isWindows,
+      );
+
+  Future<List<String>> loadAnnotatedFile({required String path}) async {
+    final f = File(path);
+    return f.readAsLines();
+  }
+
+  Future<void> writeAnnotatedFile(
+      {required String path, required List<String> content}) async {
+    final f = File(path);
+    return f.writeAsStringSync(content.join('\n'), flush: true);
+  }
+
+  Future<void> cacheSpecFile({
+    required Map<String, dynamic> updatedSpec,
+    required String cachedPath,
+  }) async =>
+      cacheSpec(outputLocation: cachedPath, spec: updatedSpec);
+
+  Future<Map<String, dynamic>> loadSpecFile(
+          {required InputSpec specConfig, bool isCached = false}) async =>
+      loadSpec(specConfig: specConfig);
+
+  Future<bool> isSpecFileDirty({
+    required Map<String, dynamic> cachedSpec,
+    required Map<String, dynamic> loadedSpec,
+  }) async =>
+      isSpecDirty(cachedSpec: cachedSpec, loadedSpec: loadedSpec);
+
+  Future<bool> checkForFlutterEnvironemt(
+          {Wrapper? wrapper = Wrapper.none,
+          String? providedPubspecPath}) async =>
+      checkPubspecAndWrapperForFlutterSupport(
+          wrapper: wrapper, providedPubspecPath: providedPubspecPath);
+}