Restructure tests to improve readability

Author: curry684

tests/Unit/DataTableTest.php

@@ -9,13 +9,6 @@
  */
 
 declare(strict_types=1);
-/*
- * Symfony DataTables Bundle
- * (c) Omines Internetbureau B.V. - https://omines.nl/
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
 
 namespace Tests\Unit;
 
@@ -78,7 +71,7 @@ public function testFactoryRemembersInstances()
 
     public function testDataTableState()
     {
-        $datatable = new DataTable($this->createMock(EventDispatcher::class));
+        $datatable = $this->createMockDataTable();
         $datatable->add('foo', TextColumn::class)->setMethod(Request::METHOD_GET);
         $datatable->handleRequest(Request::create('/?_dt=' . $datatable->getName()));
         $state = $datatable->getState();
@@ -104,7 +97,7 @@ public function testDataTableState()
 
     public function testPostMethod()
     {
-        $datatable = new DataTable($this->createMock(EventDispatcher::class));
+        $datatable = $this->createMockDataTable();
         $datatable->handleRequest(Request::create('/foo', Request::METHOD_POST, ['_dt' => $datatable->getName(), 'draw' => 684]));
 
         $this->assertSame(684, $datatable->getState()->getDraw());
@@ -129,23 +122,23 @@ public function testFactoryFailsOnInvalidType()
      */
     public function testInvalidOption()
     {
-        new DataTable($this->createMock(EventDispatcher::class), ['option' => 'bar']);
+        $this->createMockDataTable(['option' => 'bar']);
     }
 
     /**
      * @expectedException \InvalidArgumentException
      */
     public function testDataTableInvalidColumn()
     {
-        (new DataTable($this->createMock(EventDispatcher::class)))->getColumn(5);
+        $this->createMockDataTable()->getColumn(5);
     }
 
     /**
      * @expectedException \InvalidArgumentException
      */
     public function testDataTableInvalidColumnByName()
     {
-        (new DataTable($this->createMock(EventDispatcher::class)))->getColumnByName('foo');
+        $this->createMockDataTable()->getColumnByName('foo');
     }
 
     /**
@@ -154,7 +147,7 @@ public function testDataTableInvalidColumnByName()
      */
     public function testDuplicateColumnNameThrows()
     {
-        (new DataTable($this->createMock(EventDispatcher::class)))
+        $this->createMockDataTable()
             ->add('foo', TextColumn::class)
             ->add('foo', TextColumn::class)
         ;
@@ -166,9 +159,7 @@ public function testDuplicateColumnNameThrows()
      */
     public function testInvalidAdapterThrows()
     {
-        (new DataTable($this->createMock(EventDispatcher::class)))
-            ->createAdapter('foo\bar')
-        ;
+        $this->createMockDataTable()->createAdapter('foo\bar');
     }
 
     /**
@@ -177,8 +168,7 @@ public function testInvalidAdapterThrows()
      */
     public function testInvalidColumnThrows()
     {
-        (new DataTable($this->createMock(EventDispatcher::class)))
-            ->add('foo', 'bar');
+        $this->createMockDataTable()->add('foo', 'bar');
     }
 
     /**
@@ -187,7 +177,7 @@ public function testInvalidColumnThrows()
      */
     public function testMissingAdapterThrows()
     {
-        $datatable = new DataTable($this->createMock(EventDispatcher::class));
+        $datatable = $this->createMockDataTable();
         $datatable
             ->setMethod(Request::METHOD_GET)
             ->handleRequest(Request::create('/?_dt=' . $datatable->getName()))
@@ -201,7 +191,7 @@ public function testMissingAdapterThrows()
      */
     public function testEmptyNameThrows()
     {
-        (new DataTable($this->createMock(EventDispatcher::class)))->setName('');
+        $this->createMockDataTable()->setName('');
     }
 
     /**
@@ -210,7 +200,7 @@ public function testEmptyNameThrows()
      */
     public function testStateWillNotProcessInvalidMethod()
     {
-        $datatable = new DataTable($this->createMock(EventDispatcher::class));
+        $datatable = $this->createMockDataTable();
         $datatable->setMethod(Request::METHOD_OPTIONS);
         $datatable->handleRequest(Request::create('/foo'));
     }
@@ -221,8 +211,7 @@ public function testStateWillNotProcessInvalidMethod()
      */
     public function testMissingStateThrows()
     {
-        (new DataTable($this->createMock(EventDispatcher::class)))
-            ->getResponse();
+        $this->createMockDataTable()->getResponse();
     }
 
     /**
@@ -234,4 +223,9 @@ public function testInvalidDataTableTypeThrows()
         (new DataTableFactory([], $this->createMock(DataTableRendererInterface::class), new Instantiator(), $this->createMock(EventDispatcher::class)))
             ->createFromType('foo');
     }
+
+    private function createMockDataTable(array $options = [])
+    {
+        return new DataTable($this->createMock(EventDispatcher::class), $options);
+    }
 }