make all file paths absolute (#4)

* make all file paths absolute

* restore dockerfile

Author: Harry Bragg

src/Command/ChopCommand.php

@@ -76,7 +76,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $config = (new Config())->parse($input->getOption('config'));
         $group = $input->getOption('group') ?: $config->get(Config::CONFIG_DEFAULT_GROUP);
 
-        $tablePopulator = new TablePopulator(new Local('.'));
+        $tablePopulator = new TablePopulator(new Local('/'));
         $schemaParser = new SchemaParser($tablePopulator, $config, $group);
         $parsedSchemas = $schemaParser->extractSchemas($schemas);
 
@@ -94,10 +94,11 @@ function (ParsedSchema $schema) {
 
         foreach ($parsedSchemas as $schema) {
             $output->writeln(sprintf(
-                'Chopping down <info>%d</info> tables in <info>%s</info> schema in group <info>%s</info>',
+                'Chopping down <info>%d</info> tables in <info>%s</info> schema in group <info>%s</info> from <info>%s</info>',
                 count($schema->getTables()),
                 $schema->getSchemaName(),
-                $group
+                $group,
+                $schema->getPath()
             ));
 
             if ($useGlobal) {

src/Command/DumpCommand.php

@@ -73,7 +73,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $config = (new Config())->parse($input->getOption('config'));
         $group = $input->getOption('group') ?: $config->get(Config::CONFIG_DEFAULT_GROUP);
 
-        $fileSystem = new Local('.');
+        $fileSystem = new Local('/');
         $tablePopulator = new TablePopulator($fileSystem);
         $schemaParser = new SchemaParser($tablePopulator, $config, $group);
         $parsedSchemas = $schemaParser->extractSchemas($schemas);
@@ -92,10 +92,11 @@ function (ParsedSchema $schema) {
 
         foreach ($parsedSchemas as $schema) {
             $output->writeln(sprintf(
-                'Dumping <info>%d</info> tables in <info>%s</info> schema in group <info>%s</info>',
+                'Dumping <info>%d</info> tables in <info>%s</info> schema in group <info>%s</info> to <info>%s</info>',
                 count($schema->getTables()),
                 $schema->getSchemaName(),
-                $group
+                $group,
+                $schema->getPath()
             ));
 
             if ($useGlobal) {

src/Command/SeedCommand.php

@@ -99,7 +99,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
             }
         }
 
-        $tablePopulator = new TablePopulator(new Local('.'));
+        $tablePopulator = new TablePopulator(new Local('/'));
         $schemaParser = new SchemaParser($tablePopulator, $config, $group);
         $parsedSchemas = $schemaParser->extractSchemas($schemas);
 
@@ -117,10 +117,11 @@ function (ParsedSchema $schema) {
 
         foreach ($parsedSchemas as $schema) {
             $output->writeln(sprintf(
-                'Seeding <info>%d</info> tables in <info>%s</info> schema in group <info>%s</info>',
+                'Seeding <info>%d</info> tables in <info>%s</info> schema in group <info>%s</info> from <info>%s</info>',
                 count($schema->getTables()),
                 $schema->getSchemaName(),
-                $group
+                $group,
+                $schema->getPath()
             ));
 
             if ($useGlobal) {

src/Config/Config.php

@@ -155,7 +155,7 @@ public function getSchemaConfiguration(string $schema): SchemaConfigInterface
     public function getSchemaPath(SchemaConfigInterface $schema, string $group = null): string
     {
         $group = $group ?: $this->get("defaults.group");
-        return sprintf('%s/%s/', $this->getGroupPath($group), $schema->getDirName());
+        return realpath(sprintf('%s/%s/', $this->getGroupPath($group), $schema->getDirName()));
     }
 
     /**

src/Parser/TablePopulator.php

@@ -38,7 +38,7 @@ public function __construct(AdapterInterface $filesystem)
     public function populateTables(ParsedSchema $parsedSchema)
     {
         if (count($parsedSchema->getTables()) === 0) {
-            if ($this->filesystem->has($parsedSchema->getPath()) === false) {
+            if ($parsedSchema->getPath() === '' || $this->filesystem->has($parsedSchema->getPath()) === false) {
                 return null;
             }
 

tests/unit/Parser/TablePopulatorTest.php

@@ -4,7 +4,6 @@
 
 use Graze\Sprout\Config;
 use Graze\Sprout\Parser\ParsedSchema;
-use Graze\Sprout\Parser\SchemaParser;
 use Graze\Sprout\Parser\TablePopulator;
 use Graze\Sprout\Test\TestCase;
 use League\Flysystem\AdapterInterface;
@@ -89,4 +88,14 @@ public function testPopulateTablesWithNoFilesReturnsNull()
 
         $this->assertNull($output);
     }
+
+    public function testEmptyPathReturnsNull()
+    {
+        $config = Mockery::mock(Config\SchemaConfigInterface::class);
+        $parsedSchema = new ParsedSchema($config, '', []);
+
+        $output = $this->schemaParser->populateTables($parsedSchema);
+
+        $this->assertNull($output);
+    }
 }