Merge pull request #8 from dsavina/feature/unique-index

Implement `unique` based on unique index instead of column option

Author: moufmouf

src/FluidColumnOptions.php

@@ -58,9 +58,9 @@ public function null(): FluidColumnOptions
      *
      * @return FluidColumnOptions
      */
-    public function unique(): FluidColumnOptions
+    public function unique(?string $indexName = null): FluidColumnOptions
     {
-        $this->column->setCustomSchemaOption('unique', true);
+        $this->fluidTable->unique([$this->namingStrategy->quoteIdentifier($this->column->getName())], $indexName);
         return $this;
     }
 
@@ -69,9 +69,9 @@ public function unique(): FluidColumnOptions
      *
      * @return FluidColumnOptions
      */
-    public function index(): FluidColumnOptions
+    public function index(?string $indexName = null): FluidColumnOptions
     {
-        $this->fluidTable->index([$this->namingStrategy->quoteIdentifier($this->column->getName())]);
+        $this->fluidTable->index([$this->namingStrategy->quoteIdentifier($this->column->getName())], $indexName);
         return $this;
     }
     public function comment(string $comment): FluidColumnOptions

src/FluidTable.php

@@ -53,15 +53,15 @@ public function column(string $name): FluidColumn
         return $this->fluidColumns[$name];
     }
 
-    public function index(array $columnNames): FluidTable
+    public function index(array $columnNames, ?string $indexName = null): FluidTable
     {
-        $this->table->addIndex($this->quoteArray($columnNames));
+        $this->table->addIndex($this->quoteArray($columnNames), $indexName);
         return $this;
     }
 
-    public function unique(array $columnNames): FluidTable
+    public function unique(array $columnNames, ?string $indexName = null): FluidTable
     {
-        $this->table->addUniqueIndex($this->quoteArray($columnNames));
+        $this->table->addUniqueIndex($this->quoteArray($columnNames), $indexName);
         return $this;
     }
 

tests/FluidColumnOptionsTest.php

@@ -26,8 +26,8 @@ public function testOptions()
         $columnOptions->notNull();
         $this->assertSame(true, $dbalColumn->getNotnull());
 
-        $columnOptions->unique();
-        $this->assertSame(true, $dbalColumn->getCustomSchemaOption('unique'));
+        $columnOptions->unique('unique_foo');
+        $this->assertSame(true, $posts->getDbalTable()->getIndex('unique_foo')->isUnique());
 
         $columnOptions->comment('foo');
         $this->assertSame('foo', $dbalColumn->getComment());