Refactoring library (#7)

README.md

@@ -1,4 +1,4 @@
-# ElaoCommandMigration
+# CommandMigration
 
 PHP library to run commands, for example on deployment.
 
@@ -12,7 +12,7 @@ This library allow you to declare in your feature git branch what command(s) nee
 
 ## Install
 
-    $ composer require elao/elao-command-migration
+    $ composer require elao/command-migration
 
 Add a `elao_command_migration.yaml` file (in a not public directory of course!):
 
@@ -48,11 +48,11 @@ elao_command_migration:
 
 Entries in `migrations` could have whatever identifier, but we recommend to use a date + time format: YYYYMMDDHHMMSS
 
-Run `php bin/elaoCommandMigration path/to/elao_command_migration.yaml` to test it.
+Run `php bin/elao-command-migration path/to/elao_command_migration.yaml` to test it.
 
 ## Integration
 
-Add `php bin/elaoCommandMigration path/to/elao_command_migration.yaml`
+Add `php bin/elao-command-migration path/to/elao_command_migration.yaml`
 to your deployment process.
 
 ### Capifony
@@ -65,7 +65,7 @@ Set in deploy.rb:
     namespace :app_tasks do
       task :elao_command_migration do
         capifony_pretty_print "--> Run command migrations"
-        invoke_command "php bin/elaoCommandMigration path/to/elao_command_migration.yaml", :via => run_method
+        invoke_command "php bin/elao-command-migration path/to/elao_command_migration.yaml", :via => run_method
         capifony_puts_ok
       end
     end
@@ -77,19 +77,19 @@ With [Manala/ansible-role-deploy](https://github.com/manala/ansible-role-deploy)
 
 ```yaml
     manala_deploy_tasks:
-      - command: php bin/elaoCommandMigration path/to/elao_command_migration.yaml
+      - command: php bin/elao-command-migration path/to/elao_command_migration.yaml
 ```
 
 or
 
 ```yaml
     manala_deploy_post_tasks:
-      - command: php bin/elaoCommandMigration path/to/elao_command_migration.yaml
+      - command: php bin/elao-command-migration path/to/elao_command_migration.yaml
 ```
 
 ## How it works
 
-ElaoCommandMigration is very inspired by [Doctrine Migrations](https://github.com/doctrine/migrations) but
+CommandMigration is very inspired by [Doctrine Migrations](https://github.com/doctrine/migrations) but
 for running commands.
 
 The `elao:command-migration:run` command :

bin/elaoCommandMigration → bin/elao-command-migration

@@ -3,7 +3,7 @@
 
 require __DIR__ . '/../vendor/autoload.php';
 
-use Elao\ElaoCommandMigration\Command\RunCommand;
+use Elao\CommandMigration\Command\RunCommand;
 use Symfony\Component\Console\Application;
 
 $runCommand = new RunCommand();

composer.json

@@ -1,5 +1,5 @@
 {
-    "name": "elao/elao-command-migration",
+    "name": "elao/command-migration",
     "description": "PHP lib to run commands, for example on deployment.",
     "license": "MIT",
     "keywords": [],
@@ -27,12 +27,12 @@
     ],
     "autoload": {
         "psr-4": {
-            "Elao\\ElaoCommandMigration\\": "src"
+            "Elao\\CommandMigration\\": "src"
         }
     },
     "autoload-dev": {
         "psr-4": {
-            "Elao\\ElaoCommandMigration\\Tests\\": "tests"
+            "Elao\\CommandMigration\\Tests\\": "tests"
         }
     },
     "require": {
@@ -47,5 +47,5 @@
         "phpunit/phpunit": "^7.0",
         "friendsofphp/php-cs-fixer": "^2.13"
     },
-    "bin": ["bin/elaoCommandMigration"]
+    "bin": ["bin/elao-command-migration"]
 }

src/Adapter/ProcessAdapter.php

@@ -1,14 +1,6 @@
 <?php
 
-/*
- * This file is part of the ElaoCommandMigration project.
- *
- * Copyright (C) ElaoCommandMigration
- *
- * @author Elao <[email protected]>
- */
-
-namespace Elao\ElaoCommandMigration\Adapter;
+namespace Elao\CommandMigration\Adapter;
 
 use Symfony\Component\Process\Process;
 

src/Command/RunCommand.php

@@ -1,10 +1,10 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Command;
+namespace Elao\CommandMigration\Command;
 
-use Elao\ElaoCommandMigration\Process\Exception\ConnectionException;
-use Elao\ElaoCommandMigration\Process\ResultView;
-use Elao\ElaoCommandMigration\Process\Run;
+use Elao\CommandMigration\Process\Exception\ConnectionException;
+use Elao\CommandMigration\Process\ResultView;
+use Elao\CommandMigration\Process\Run;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;

src/Configuration/Factory/Doctrine/DBALConnectionFactory.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Configuration\Factory\Doctrine;
+namespace Elao\CommandMigration\Configuration\Factory\Doctrine;
 
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\ConnectionException;

src/Event/MigrationExecutedEvent.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Event;
+namespace Elao\CommandMigration\Event;
 
 use Symfony\Component\EventDispatcher\Event;
 

src/Events.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration;
+namespace Elao\CommandMigration;
 
 final class Events
 {

src/Migration/GetNotExecutedMigrations.php

@@ -1,8 +1,8 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Migration;
+namespace Elao\CommandMigration\Migration;
 
-use Elao\ElaoCommandMigration\Storage\StorageInterface;
+use Elao\CommandMigration\Storage\StorageInterface;
 
 class GetNotExecutedMigrations
 {

src/Parser/Exception/InvalidYamlSchemaException.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Parser\Exception;
+namespace Elao\CommandMigration\Parser\Exception;
 
 class InvalidYamlSchemaException extends \Exception
 {

src/Parser/ParserInterface.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Parser;
+namespace Elao\CommandMigration\Parser;
 
 interface ParserInterface
 {

src/Parser/YamlParser.php

@@ -1,8 +1,8 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Parser;
+namespace Elao\CommandMigration\Parser;
 
-use Elao\ElaoCommandMigration\Parser\Exception\InvalidYamlSchemaException;
+use Elao\CommandMigration\Parser\Exception\InvalidYamlSchemaException;
 use Symfony\Component\Yaml\Yaml;
 
 class YamlParser implements ParserInterface

src/Process/Exception/ConnectionException.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Process\Exception;
+namespace Elao\CommandMigration\Process\Exception;
 
 class ConnectionException extends \Exception
 {

src/Process/ExecuteVersion.php

@@ -1,10 +1,10 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Process;
+namespace Elao\CommandMigration\Process;
 
-use Elao\ElaoCommandMigration\Adapter\ProcessAdapter;
-use Elao\ElaoCommandMigration\Event\MigrationExecutedEvent;
-use Elao\ElaoCommandMigration\Events;
+use Elao\CommandMigration\Adapter\ProcessAdapter;
+use Elao\CommandMigration\Event\MigrationExecutedEvent;
+use Elao\CommandMigration\Events;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 
 class ExecuteVersion

src/Process/ResultView.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Process;
+namespace Elao\CommandMigration\Process;
 
 class ResultView
 {

src/Process/Run.php

@@ -1,12 +1,12 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Process;
+namespace Elao\CommandMigration\Process;
 
-use Elao\ElaoCommandMigration\Adapter\ProcessAdapter;
-use Elao\ElaoCommandMigration\Migration\GetNotExecutedMigrations;
-use Elao\ElaoCommandMigration\Parser\YamlParser;
-use Elao\ElaoCommandMigration\Storage\StorageFactory;
-use Elao\ElaoCommandMigration\Subscriber\MigrationExecutedSubscriber;
+use Elao\CommandMigration\Adapter\ProcessAdapter;
+use Elao\CommandMigration\Migration\GetNotExecutedMigrations;
+use Elao\CommandMigration\Parser\YamlParser;
+use Elao\CommandMigration\Storage\StorageFactory;
+use Elao\CommandMigration\Subscriber\MigrationExecutedSubscriber;
 use Symfony\Component\EventDispatcher\EventDispatcher;
 
 class Run

src/Storage/DoctrineStorage.php

@@ -1,13 +1,13 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Storage;
+namespace Elao\CommandMigration\Storage;
 
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Connections\MasterSlaveConnection;
 use Doctrine\DBAL\Schema\Column;
 use Doctrine\DBAL\Schema\Table;
 use Doctrine\DBAL\Types\Type;
-use Elao\ElaoCommandMigration\Configuration\Factory\Doctrine\DBALConnectionFactory;
+use Elao\CommandMigration\Configuration\Factory\Doctrine\DBALConnectionFactory;
 
 final class DoctrineStorage implements StorageInterface
 {

src/Storage/StorageFactory.php

@@ -1,8 +1,8 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Storage;
+namespace Elao\CommandMigration\Storage;
 
-use Elao\ElaoCommandMigration\Parser\Exception\InvalidYamlSchemaException;
+use Elao\CommandMigration\Parser\Exception\InvalidYamlSchemaException;
 
 final class StorageFactory
 {

src/Storage/StorageInterface.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Storage;
+namespace Elao\CommandMigration\Storage;
 
 interface StorageInterface
 {

src/Subscriber/MigrationExecutedSubscriber.php

@@ -1,10 +1,10 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Subscriber;
+namespace Elao\CommandMigration\Subscriber;
 
-use Elao\ElaoCommandMigration\Event\MigrationExecutedEvent;
-use Elao\ElaoCommandMigration\Events;
-use Elao\ElaoCommandMigration\Storage\StorageInterface;
+use Elao\CommandMigration\Event\MigrationExecutedEvent;
+use Elao\CommandMigration\Events;
+use Elao\CommandMigration\Storage\StorageInterface;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
 class MigrationExecutedSubscriber implements EventSubscriberInterface

tests/Parser/YamlParserTest.php

@@ -1,9 +1,9 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Tests\Parser;
+namespace Elao\CommandMigration\Tests\Parser;
 
-use Elao\ElaoCommandMigration\Parser\Exception\InvalidYamlSchemaException;
-use Elao\ElaoCommandMigration\Parser\YamlParser;
+use Elao\CommandMigration\Parser\Exception\InvalidYamlSchemaException;
+use Elao\CommandMigration\Parser\YamlParser;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Yaml\Exception\ParseException;
 

tests/Process/ExecuteVersionTest.php

@@ -1,12 +1,12 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Tests\Process;
+namespace Elao\CommandMigration\Tests\Process;
 
-use Elao\ElaoCommandMigration\Adapter\ProcessAdapter;
-use Elao\ElaoCommandMigration\Event\MigrationExecutedEvent;
-use Elao\ElaoCommandMigration\Events;
-use Elao\ElaoCommandMigration\Process\ExecuteVersion;
-use Elao\ElaoCommandMigration\Process\ResultView;
+use Elao\CommandMigration\Adapter\ProcessAdapter;
+use Elao\CommandMigration\Event\MigrationExecutedEvent;
+use Elao\CommandMigration\Events;
+use Elao\CommandMigration\Process\ExecuteVersion;
+use Elao\CommandMigration\Process\ResultView;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Symfony\Component\Process\Process;
@@ -17,7 +17,7 @@ public function testExecuteVersion(): void
     {
         $migrations = [
             'identifier1' => [
-                'php bin/elaoCommandMigration inception.yml',
+                'php bin/elao-command-migration inception.yml',
             ],
             '1234567890' => [
                 'php -r "echo \"toto\";"',
@@ -29,7 +29,7 @@ public function testExecuteVersion(): void
         $processAdapter = $this->prophesize(ProcessAdapter::class);
 
         $firstProcess = $this->prophesize(Process::class);
-        $processAdapter->getProcess('php bin/elaoCommandMigration inception.yml')->shouldBeCalled()->willReturn($firstProcess);
+        $processAdapter->getProcess('php bin/elao-command-migration inception.yml')->shouldBeCalled()->willReturn($firstProcess);
         $firstProcess->run()->shouldBeCalled();
         $firstProcess->isSuccessful()->shouldBeCalled()->willReturn(false);
         $firstProcess->getErrorOutput()->shouldBeCalled()->willReturn('Endless spinning');
@@ -53,7 +53,7 @@ public function testExecuteVersion(): void
         $generators = $executeVersion($migrations);
 
         $expected = [
-            0 => new ResultView(false, 'php bin/elaoCommandMigration inception.yml', 'Endless spinning'),
+            0 => new ResultView(false, 'php bin/elao-command-migration inception.yml', 'Endless spinning'),
             1 => new ResultView(true, 'php -r "echo \"toto\";"', ''),
             2 => new ResultView(true, 'php -r "echo \"test\";"', ''),
         ];

tests/Storage/StorageFactoryTest.php

@@ -1,10 +1,10 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Tests\Storage;
+namespace Elao\CommandMigration\Tests\Storage;
 
-use Elao\ElaoCommandMigration\Parser\Exception\InvalidYamlSchemaException;
-use Elao\ElaoCommandMigration\Storage\DoctrineStorage;
-use Elao\ElaoCommandMigration\Storage\StorageFactory;
+use Elao\CommandMigration\Parser\Exception\InvalidYamlSchemaException;
+use Elao\CommandMigration\Storage\DoctrineStorage;
+use Elao\CommandMigration\Storage\StorageFactory;
 use PHPUnit\Framework\TestCase;
 
 class StorageFactoryTest extends TestCase

tests/e2e/E2ETest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Elao\ElaoCommandMigration\Tests\e2e;
+namespace Elao\CommandMigration\Tests\e2e;
 
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Process\Process;
@@ -10,7 +10,7 @@ class E2ETest extends TestCase
     public function testE2E()
     {
         $pathToFile = sprintf('%s/elao_command_migration.yaml', __DIR__);
-        $process = new Process(sprintf('php bin/elaoCommandMigration %s', $pathToFile));
+        $process = new Process(sprintf('php bin/elao-command-migration %s', $pathToFile));
         $process->run();
 
         $this->assertFileExists('elao_command_migration_test.txt');

tests/e2e/elao_command_migration.yaml

@@ -4,6 +4,6 @@ elao_command_migration:
         dsn: mysql://app:password@mysql/elao_command_migration
         table_name: 'command_migrations'
     migrations:
-        20181112153014:
+        20181112153017:
             - php -r "echo \"test\";"
             - touch elao_command_migration_test.txt