Tests prepared

src/Contracts/Translator.php

@@ -10,5 +10,5 @@ interface Translator
 {
     public function can(Locale|string $to): bool;
 
-    public function text(array|string $text, Locale|string $to, Locale|string|null $from = null): array|string;
+    public function translate(array|string $text, Locale|string $to, Locale|string|null $from = null): array|string;
 }

src/Facades/Translate.php

@@ -9,10 +9,10 @@
 use LaravelLang\Translator\Services\Translate as TranslateService;
 
 /**
- * @method static array|string text(iterable|string $text, Locale|string $to, Locale|string|null $from)
- * @method static array|string viaDeepl(iterable|string $text, Locale|string $to, Locale|string|null $from)
- * @method static array|string viaGoogleFree(iterable|string $text, Locale|string $to, Locale|string|null $from)
- * @method static array|string viaYandex(iterable|string $text, Locale|string $to, Locale|string|null $from)
+ * @method static array|string text(iterable|string $text, Locale|string $to, Locale|string|null $from = null)
+ * @method static array|string viaDeepl(iterable|string $text, Locale|string $to, Locale|string|null $from = null)
+ * @method static array|string viaGoogleFree(iterable|string $text, Locale|string $to, Locale|string|null $from = null)
+ * @method static array|string viaYandex(iterable|string $text, Locale|string $to, Locale|string|null $from = null)
  */
 class Translate extends Facade
 {

src/Integrations/Deepl.php

@@ -19,7 +19,7 @@ public function __construct(
         protected array $options
     ) {}
 
-    protected function translate(iterable|string $text, Locale|string $to, Locale|string|null $from): Collection
+    protected function request(iterable|string $text, Locale|string $to, Locale|string|null $from): Collection
     {
         return collect($this->translator->translateText($text, $this->lang($from), $this->lang($to), $this->options));
     }

src/Integrations/GoogleFree.php

@@ -16,11 +16,11 @@ class GoogleFree extends Integration
 
     public function __construct(
         protected GoogleTranslate $translator,
-        protected array $options,
-        protected bool|string $preserve
+        protected array $options = [],
+        protected bool|string $preserve = true
     ) {}
 
-    protected function translate(iterable|string $text, Locale|string $to, Locale|string|null $from): Collection
+    protected function request(iterable|string $text, Locale|string $to, Locale|string|null $from): Collection
     {
         return collect($text)->map(
             fn (string $value) => $this->translator($to, $from)->translate($value)

src/Integrations/Integration.php

@@ -12,7 +12,7 @@ abstract class Integration implements Translator
 {
     protected array $map = [];
 
-    abstract protected function translate(
+    abstract protected function request(
         iterable|string $text,
         Locale|string $to,
         Locale|string|null $from
@@ -23,14 +23,14 @@ public function can(Locale|string $to): bool
         return $this->lang($to) !== null;
     }
 
-    public function text(
+    public function translate(
         array|string $text,
         Locale|string $to,
         Locale|string|null $from = null
     ): array|string {
         return is_array($text)
-            ? $this->translate($text, $to, $from)->pluck('text')->all()
-            : $this->translate($text, $to, $from)->pluck('text')->first();
+            ? $this->request($text, $to, $from)->pluck('text')->all()
+            : $this->request($text, $to, $from)->pluck('text')->first();
     }
 
     protected function lang(Locale|string|null $lang): ?string

src/Integrations/Yandex.php

@@ -18,7 +18,7 @@ public function __construct(
         protected YandexCloud $translator,
     ) {}
 
-    protected function translate(iterable|string $text, Locale|string $to, Locale|string|null $from): Collection
+    protected function request(iterable|string $text, Locale|string $to, Locale|string|null $from): Collection
     {
         return collect($this->translator->translate($text, $this->lang($to), $this->lang($from)));
     }

src/Services/Translate.php

@@ -12,7 +12,7 @@
 
 class Translate
 {
-    public function text(iterable|string $text, Locale|string $to, Locale|string|null $from): array|string
+    public function text(iterable|string $text, Locale|string $to, Locale|string|null $from = null): array|string
     {
         foreach ($this->translators() as $class) {
             if ($translated = $this->translate($class, $text, $to, $from)) {
@@ -23,17 +23,20 @@ public function text(iterable|string $text, Locale|string $to, Locale|string|nul
         return $text;
     }
 
-    public function viaDeepl(iterable|string $text, Locale|string $to, Locale|string|null $from): array|string
+    public function viaDeepl(iterable|string $text, Locale|string $to, Locale|string|null $from = null): array|string
     {
         return $this->via(Deepl::class, $text, $to, $from);
     }
 
-    public function viaGoogleFree(iterable|string $text, Locale|string $to, Locale|string|null $from): array|string
-    {
+    public function viaGoogleFree(
+        iterable|string $text,
+        Locale|string $to,
+        Locale|string|null $from = null
+    ): array|string {
         return $this->via(GoogleFree::class, $text, $to, $from);
     }
 
-    public function viaYandex(iterable|string $text, Locale|string $to, Locale|string|null $from): array|string
+    public function viaYandex(iterable|string $text, Locale|string $to, Locale|string|null $from = null): array|string
     {
         return $this->via(Yandex::class, $text, $to, $from);
     }
@@ -56,7 +59,7 @@ protected function translate(
         $translator = $this->initialize($translator);
 
         if ($translator->can($to)) {
-            return $translator->text($text, $to, $from);
+            return $translator->translate($text, $to, $from);
         }
 
         return null;

tests/Concerns/Value.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Concerns;
+
+class Value
+{
+    public const Text1English = 'The :attribute must be accepted.';
+    public const Text1French  = 'Le champ :attribute doit être accepté.';
+    public const Text2English = 'The :attribute must be a date after :date.';
+    public const Text2French  = 'Le champ :attribute doit être une date postérieure au :date.';
+}

tests/Datasets/Translators.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+use LaravelLang\Translator\Integrations\Deepl;
+use LaravelLang\Translator\Integrations\GoogleFree;
+use LaravelLang\Translator\Integrations\Yandex;
+
+dataset('translators', fn () => [
+    Deepl::class      => [Deepl::class],
+    GoogleFree::class => [GoogleFree::class],
+    Yandex::class     => [Yandex::class],
+]);

tests/Helpers/Translators.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+use LaravelLang\Translator\Contracts\Translator;
+
+function translator(string $class): Translator
+{
+    return app($class);
+}

tests/Pest.php

@@ -1,45 +1 @@
 <?php
-
-/*
-|--------------------------------------------------------------------------
-| Test Case
-|--------------------------------------------------------------------------
-|
-| The closure you provide to your test functions is always bound to a specific PHPUnit test
-| case class. By default, that class is "PHPUnit\Framework\TestCase". Of course, you may
-| need to change it using the "uses()" function to bind a different classes or traits.
-|
-*/
-
-// uses(Tests\TestCase::class)->in('Feature');
-
-/*
-|--------------------------------------------------------------------------
-| Expectations
-|--------------------------------------------------------------------------
-|
-| When you're writing tests, you often need to check that values meet certain conditions. The
-| "expect()" function gives you access to a set of "expectations" methods that you can use
-| to assert different things. Of course, you may extend the Expectation API at any time.
-|
-*/
-
-expect()->extend('toBeOne', function () {
-    return $this->toBe(1);
-});
-
-/*
-|--------------------------------------------------------------------------
-| Functions
-|--------------------------------------------------------------------------
-|
-| While Pest is very powerful out-of-the-box, you may have some testing code specific to your
-| project that you don't want to repeat in every file. Here you can also expose helpers as
-| global functions to help you to reduce the number of lines of code in your test files.
-|
-*/
-
-function something()
-{
-    // ..
-}

tests/Unit/Can/IntegrationTest.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+use LaravelLang\LocaleList\Locale;
+
+test('can be translatable', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->can('fr'))->toBeTrue();
+    expect($translator->can(Locale::French))->toBeTrue();
+})->with('translators');
+
+test('cannot be translatable', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->can('qwerty'))->toBeTrue();
+})->with('translators');

tests/Unit/Translate/FacadeTest.php

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+use LaravelLang\LocaleList\Locale;
+use LaravelLang\Translator\Facades\Translate;
+use Tests\Concerns\Value;
+
+test('translate', function () {
+    expect(Translate::text(Value::Text1English, Locale::French))->toBe(
+        Value::Text1French
+    );
+});
+
+test('cannot be translated', function () {
+    expect(Translate::text(Value::Text1English, 'qwerty'))->toBe(
+        Value::Text1English
+    );
+});
+
+test('via deepl', function () {
+    expect(Translate::viaDeepl(Value::Text1English, Locale::French))->toBe(
+        Value::Text1French
+    );
+});
+
+test('via google free', function () {
+    expect(Translate::viaGoogleFree(Value::Text1English, Locale::French))->toBe(
+        Value::Text1French
+    );
+});
+
+test('via yandex', function () {
+    expect(Translate::viaYandex(Value::Text1English, Locale::French))->toBe(
+        Value::Text1French
+    );
+});

tests/Unit/Translate/IntegrationCanTest.php

@@ -0,0 +1,47 @@
+<?php
+
+declare(strict_types=1);
+
+use LaravelLang\LocaleList\Locale;
+use Tests\Concerns\Value;
+
+test('as string', function (string $translator) {
+    $translator = translator($translator);
+
+    expect(
+        $translator->translate(Value::Text1English, Locale::French)
+    )->toBe(Value::Text1French);
+})->with('translators');
+
+test('as array without keys', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->translate([Value::Text1English, Value::Text2English], Locale::French))->toBe([
+        Value::Text1French,
+        Value::Text2French,
+    ]);
+})->with('translators');
+
+test('as array with keys', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->translate([
+        'foo' => Value::Text1English,
+        'bar' => Value::Text2English,
+    ], Locale::French))->toBe([
+        'foo' => Value::Text1French,
+        'bar' => Value::Text2French,
+    ]);
+})->with('translators');
+
+test('as collection', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->translate(collect([
+        'foo' => Value::Text1English,
+        'bar' => Value::Text2English,
+    ]), Locale::French))->toBe([
+        'foo' => Value::Text1French,
+        'bar' => Value::Text2French,
+    ]);
+})->with('translators');

tests/Unit/Translate/IntegrationCannotTest.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+
+use Tests\Concerns\Value;
+
+test('as string', function (string $translator) {
+    $translator = translator($translator);
+
+    expect(
+        $translator->translate(Value::Text1English, 'qwerty')
+    )->toBe(Value::Text1English);
+})->with('translators');
+
+test('as array without keys', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->translate([Value::Text1English, Value::Text2English], 'qwerty'))->toBe([
+        Value::Text1English,
+        Value::Text2English,
+    ]);
+})->with('translators');
+
+test('as array with keys', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->translate([
+        'foo' => Value::Text1English,
+        'bar' => Value::Text2English,
+    ], 'qwerty'))->toBe([
+        'foo' => Value::Text1English,
+        'bar' => Value::Text2English,
+    ]);
+})->with('translators');
+
+test('as collection', function (string $translator) {
+    $translator = translator($translator);
+
+    expect($translator->translate(collect([
+        'foo' => Value::Text1English,
+        'bar' => Value::Text2English,
+    ]), 'qwerty'))->toBe([
+        'foo' => Value::Text1English,
+        'bar' => Value::Text2English,
+    ]);
+})->with('translators');