Merge pull request #32 from leancodepl/feat/ci-setup-and-model-provider-tests

Add model provider tests and ci for tests

Author: Polyphemus980

.github/workflows/publish.yaml

@@ -0,0 +1,28 @@
+name: Publish to pub.dev
+
+on:
+  push:
+    tags:
+      - 'v[0-9]+.[0-9]+.[0-9]+'
+
+jobs:
+  publish:
+    name: Publish to pub.dev
+
+    runs-on: ubuntu-latest
+
+    permissions:
+      id-token: write
+      contents: write
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Set up Dart
+        uses: dart-lang/setup-dart@v1
+        with:
+          sdk: stable
+
+      - name: Publish and release
+        uses: leancodepl/mobile-tools/.github/actions/pub-release@pub-release-v1

.github/workflows/run-tests.yaml

@@ -0,0 +1,34 @@
+name: test
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  run-tests:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Dart
+        uses: dart-lang/setup-dart@v1
+        with:
+          sdk: stable
+
+      - name: Get dependencies
+        run: dart pub get
+
+      - name: Analyze code
+        run: dart analyze
+
+      - name: Run tests (for now only translation options)
+        run: dart test test/translation_options_test.dart
+
+      - name: Dry run pub publish
+        run: dart pub publish --dry-run || true

test/translation_options_test.dart

@@ -3,26 +3,26 @@ import 'package:file/memory.dart';
 import 'package:test/test.dart';
 
 void main() {
+  const argResultsApiKey = 'argResultsApiKey';
+  const argResultsDisableSafety = true;
+  const argResultsContext = 'argResultsContext';
+  const argResultsExcludeLocales = ['pl'];
+  const argResultsBatchSize = 4096;
+  const argResultsArbDir = 'argResultsArbDir';
+  const argResultsTemplateArbFile = 'argResultsTemplateArbFile';
+  const argResultsUseEscaping = true;
+  const argResultsRelaxSyntax = true;
+
   group(
     'TranslateOptions',
     () {
       test(
         'resolve returns options with values from argResults over yamlResults',
         () {
-          const argResultsModelProvider = ModelProvider.gemini;
+          const argResultsModel = Model.gemini25Flash;
           const argResultsCustomModelProviderUrl =
               'http://argResultsCustomModelProviderBaseUrl';
-          const argResultsModel = Model.gemini25Flash;
           const argResultsCustomModel = 'argResultsCustomModel';
-          const argResultsApiKey = 'argResultsApiKey';
-          const argResultsDisableSafety = true;
-          const argResultsContext = 'argResultsContext';
-          const argResultsExcludeLocales = ['pl'];
-          const argResultsBatchSize = 4096;
-          const argResultsArbDir = 'argResultsArbDir';
-          const argResultsTemplateArbFile = 'argResultsTemplateArbFile';
-          const argResultsUseEscaping = true;
-          const argResultsRelaxSyntax = true;
 
           final argResults = TranslateArgResults(
             help: false,
@@ -64,11 +64,6 @@ void main() {
           expect(
             translateOptions,
             isA<TranslateOptions>()
-                .having(
-                  (options) => options.modelProvider,
-                  'modelProvider',
-                  argResultsModelProvider,
-                )
                 .having(
                   (options) => options.customModelProviderBaseUrl,
                   'customModelProviderBaseUrl',
@@ -134,4 +129,114 @@ void main() {
       );
     },
   );
+
+  group(
+    'modelProvider resolving',
+    () {
+      test('resolves gemini provider', () {
+        const argResultsModel = Model.gemini25Flash;
+
+        final argResults = TranslateArgResults(
+          help: false,
+          customModelProviderBaseUrl: null,
+          model: argResultsModel,
+          customModel: null,
+          apiKey: argResultsApiKey,
+          disableSafety: argResultsDisableSafety,
+          context: argResultsContext,
+          excludeLocales: argResultsExcludeLocales,
+          batchSize: argResultsBatchSize,
+          arbDir: argResultsArbDir,
+          templateArbFile: argResultsTemplateArbFile,
+          useEscaping: argResultsUseEscaping,
+          relaxSyntax: argResultsRelaxSyntax,
+        );
+        final yamlResults = TranslateYamlResults.empty();
+
+        final translateOptions = TranslateOptions.resolve(
+          MemoryFileSystem(),
+          argResults,
+          yamlResults,
+        );
+
+        expect(
+            translateOptions,
+            isA<TranslateOptions>().having((options) => options.modelProvider,
+                'modelProvider', ModelProvider.gemini));
+      });
+
+      test('resolves openAI provider', () {
+        const argResultsModel = Model.gpt5Mini;
+
+        final argResults = TranslateArgResults(
+          help: false,
+          customModelProviderBaseUrl: null,
+          model: argResultsModel,
+          customModel: null,
+          apiKey: argResultsApiKey,
+          disableSafety: argResultsDisableSafety,
+          context: argResultsContext,
+          excludeLocales: argResultsExcludeLocales,
+          batchSize: argResultsBatchSize,
+          arbDir: argResultsArbDir,
+          templateArbFile: argResultsTemplateArbFile,
+          useEscaping: argResultsUseEscaping,
+          relaxSyntax: argResultsRelaxSyntax,
+        );
+        final yamlResults = TranslateYamlResults.empty();
+
+        final translateOptions = TranslateOptions.resolve(
+          MemoryFileSystem(),
+          argResults,
+          yamlResults,
+        );
+
+        expect(
+            translateOptions,
+            isA<TranslateOptions>().having((options) => options.modelProvider,
+                'modelProvider', ModelProvider.openAi));
+      });
+
+      test(
+          'resolves customOpenAiCompatible provider with custom model taking precedence',
+          () {
+        const argResultsModel = Model.gemini25Flash;
+        const argResultsCustomModelProviderUrl =
+            'http://argResultsCustomModelProviderBaseUrl';
+        const argResultsCustomModel = 'argResultsCustomModel';
+
+        final argResults = TranslateArgResults(
+          help: false,
+          customModelProviderBaseUrl: argResultsCustomModelProviderUrl,
+          model: argResultsModel,
+          customModel: argResultsCustomModel,
+          apiKey: argResultsApiKey,
+          disableSafety: argResultsDisableSafety,
+          context: argResultsContext,
+          excludeLocales: argResultsExcludeLocales,
+          batchSize: argResultsBatchSize,
+          arbDir: argResultsArbDir,
+          templateArbFile: argResultsTemplateArbFile,
+          useEscaping: argResultsUseEscaping,
+          relaxSyntax: argResultsRelaxSyntax,
+        );
+        final yamlResults = TranslateYamlResults.empty();
+
+        final translateOptions = TranslateOptions.resolve(
+          MemoryFileSystem(),
+          argResults,
+          yamlResults,
+        );
+
+        expect(
+          translateOptions,
+          isA<TranslateOptions>()
+              .having((o) => o.modelProvider, 'modelProvider',
+                  ModelProvider.customOpenAiCompatible)
+              .having(
+                  (o) => o.customModel, 'customModel', argResultsCustomModel),
+        );
+      });
+    },
+  );
 }