Refactor ImportTest and add YamlRecipeTest

Author: antonmedv

tests/src/Import/ImportTest.php

@@ -5,56 +5,37 @@
 namespace Deployer\Import;
 
 use Deployer\Deployer;
+use Deployer\Exception\Exception;
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\Console\Application;
+use Symfony\Component\Console\Input\Input;
+use Symfony\Component\Console\Output\Output;
 
 class ImportTest extends TestCase
 {
-    private $previousInput;
-    private $previousOutput;
+    private Deployer $deployer;
 
     public function setUp(): void
     {
-        $deployer = Deployer::get();
-        $this->previousInput = $deployer->input;
-        $this->previousOutput = $deployer->output;
+        $console = new Application();
+        $input = $this->createStub(Input::class);
+        $output = $this->createStub(Output::class);
+
+        $this->deployer = new Deployer($console);
+        $this->deployer['input'] = $input;
+        $this->deployer['output'] = $output;
     }
 
     public function tearDown(): void
     {
-        Deployer::get()->input = $this->previousInput;
-        Deployer::get()->output = $this->previousOutput;
+        unset($this->deployer);
     }
 
-    public function testImporterIgnoresYamlHiddenKeys(): void
+    public function testUnknownFileFormatThrows(): void
     {
-        $data = <<<EOL
-            .base: &base
-              remote_user: foo
-              labels:
-                stage: production
-
-            hosts:
-              acceptance:
-                <<: *base
-                labels:
-                  stage: acceptance
-
-              production:
-                <<: *base
-                remote_user: bar
-              
-              production.beta:
-                <<: *base
-            # test.yaml
-            EOL;
-
-        Import::import("data:text/yaml,$data");
-        self::assertTrue(Deployer::get()->hosts->has('production'));
-        self::assertTrue(Deployer::get()->hosts->has('acceptance'));
-        self::assertTrue(Deployer::get()->hosts->has('production.beta'));
-        self::assertEquals('acceptance', Deployer::get()->hosts->get('acceptance')->getLabels()['stage']);
-        self::assertEquals('production', Deployer::get()->hosts->get('production')->getLabels()['stage']);
-        self::assertEquals('foo', Deployer::get()->hosts->get('acceptance')->getRemoteUser());
-        self::assertEquals('bar', Deployer::get()->hosts->get('production')->getRemoteUser());
+        $this->expectException(Exception::class);
+        $this->expectExceptionMessage('Unknown file format');
+
+        Import::import('file.txt');
     }
 }

tests/src/Import/YamlRecipeTest.php

@@ -0,0 +1,65 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Deployer\Import;
+
+use Deployer\Deployer;
+use PHPUnit\Framework\TestCase;
+use Symfony\Component\Console\Application;
+use Symfony\Component\Console\Input\Input;
+use Symfony\Component\Console\Output\Output;
+
+class YamlRecipeTest extends TestCase
+{
+    private Deployer $deployer;
+
+    public function setUp(): void
+    {
+        $console = new Application();
+        $input = $this->createStub(Input::class);
+        $output = $this->createStub(Output::class);
+
+        $this->deployer = new Deployer($console);
+        $this->deployer['input'] = $input;
+        $this->deployer['output'] = $output;
+    }
+
+    public function tearDown(): void
+    {
+        unset($this->deployer);
+    }
+
+    public function testImporterIgnoresYamlHiddenKeys(): void
+    {
+        $data = <<<EOL
+            .base: &base
+              remote_user: foo
+              labels:
+                stage: production
+
+            hosts:
+              acceptance:
+                <<: *base
+                labels:
+                  stage: acceptance
+
+              production:
+                <<: *base
+                remote_user: bar
+
+              production.beta:
+                <<: *base
+            # test.yaml
+            EOL;
+
+        Import::import("data:text/yaml,$data");
+        self::assertTrue($this->deployer->hosts->has('production'));
+        self::assertTrue($this->deployer->hosts->has('acceptance'));
+        self::assertTrue($this->deployer->hosts->has('production.beta'));
+        self::assertEquals('acceptance', $this->deployer->hosts->get('acceptance')->getLabels()['stage']);
+        self::assertEquals('production', $this->deployer->hosts->get('production')->getLabels()['stage']);
+        self::assertEquals('foo', $this->deployer->hosts->get('acceptance')->getRemoteUser());
+        self::assertEquals('bar', $this->deployer->hosts->get('production')->getRemoteUser());
+    }
+}