CrossKnowledge / DataTableBundle

Tests/UsesContainerTrait.php

Unused Use Statements -2 removed lines

@@ -5,9 +5,7 @@
 use CrossKnowledge\DataTableBundle\DataTable\Table\AbstractTable;
 use CrossKnowledge\DataTableBundle\DependencyInjection\Compiler\DatatablePass;
 use CrossKnowledge\DataTableBundle\DependencyInjection\CrossKnowledgeDataTableExtension;
-use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Definition;
 
 trait UsesContainerTrait
 {

Tests/DataTable/Column/ColumnTest.php

Indentation +2 added lines, -2 removed lines

@@ -28,10 +28,10 @@
     public function testBasicGetterSetter()
     {
         $column = (new Column("test", ['auto_escape' => false]))
-                   ->setFormatValueCallback(function() {
+                    ->setFormatValueCallback(function() {
                         return 'test ok';
                     })
-                   ->setIdentifier('test');
+                    ->setIdentifier('test');
 
         $this->assertEquals("test", $column->getOptions()['title']);
         $this->assertEquals("test ok", call_user_func($column->getFormatValueCallback()));

Tests/DataTable/Renderer/TwigRendererTest.php

Indentation +3 added lines, -3 removed lines

@@ -20,11 +20,11 @@
                     ->getMock();
 
         $templateMock = $this->getMockBuilder(\Twig_TemplateInterface::class)
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $twig->expects($this->once())->method('loadTemplate')
-             ->will($this->returnValue($templateMock));
+                ->will($this->returnValue($templateMock));
 
         $renderer = new TwigRenderer($twig);
         $renderer->render($mock);

Tests/DataTable/RegistryTest.php

Indentation +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
     public function testRetrieveTableById()
     {
         $table = $this->getMockBuilder(AbstractTable::class)
-                      ->disableOriginalConstructor()
-                      ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $registry = new DataTableRegistry(['test' => $table]);
         $this->assertEquals($table, $registry->retrieveByTableId('test'));

Tests/DataTable/Table/AbstractTableTest.php

Indentation +27 added lines, -27 removed lines

@@ -20,12 +20,12 @@ 
     protected function getTableMock($cols, $rows)
     {
         $formFactory = $this->getMockBuilder(FormFactory::class)
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $form = $this->getMockBuilder('Symfony\Component\Form\Form')
-                     ->disableOriginalConstructor()
-                     ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $authChecker = $this->getMockBuilder(AuthorizationChecker::class)
                             ->disableOriginalConstructor()
@@ -39,20 +39,20 @@ 
         $layout    = new Bootstrap();
 
         $mock = $this->getMockBuilder(AbstractTable::class)
-                     ->setConstructorArgs([
-                         $formFactory, $authChecker,
-                         $router, $formatter, $layout
-                     ])
-                     ->setMethods(['getColumns', 'getFilterForm'])
-                     ->getMockForAbstractClass();
+                        ->setConstructorArgs([
+                            $formFactory, $authChecker,
+                            $router, $formatter, $layout
+                        ])
+                        ->setMethods(['getColumns', 'getFilterForm'])
+                        ->getMockForAbstractClass();
 
         $mock->expects($this->any())
             ->method('getDataIterator')
             ->will($this->returnValue($rows));
 
         $mock->expects($this->any())
-             ->method('getColumns')
-             ->will($this->returnValue($cols));
+                ->method('getColumns')
+                ->will($this->returnValue($cols));
 
         $mock->expects($this->any())
             ->method('getFilterForm')
@@ -72,7 +72,7 @@ 
 
         $filterForm = $table->getFilterForm();
         $filterForm->expects($this->never())
-                   ->method('createView');
+                    ->method('createView');
 
         $viewData = $table->buildView();
         $this->assertEquals($viewData['datatable'], $table);
@@ -85,7 +85,7 @@ 
 
         $filterForm = $table->getFilterForm();
         $filterForm->expects($this->once())
-                   ->method('createView');
+                    ->method('createView');
         $table->buildView();
     }
     /**
@@ -97,7 +97,7 @@ 
         $table->setOptions(['has_filter_form' => false]);
         $filterForm = $table->getFilterForm();
         $filterForm->expects($this->never())
-                   ->method('createView');
+                    ->method('createView');
 
         $table->buildView();
 
@@ -106,7 +106,7 @@ 
 
         $filterForm = $table->getFilterForm();
         $filterForm->expects($this->once())
-                   ->method('createView');
+                    ->method('createView');
         $table->buildView();
     }
     /**
@@ -131,17 +131,17 @@ 
     public function buildViewProvider()
     {
         return [
-          [
-              [
-                  'firstCol' => new Column('title 1st col'),
-                  'secondCol' => new Column('title 2nd col')
-              ],
-              [
-                  ['firstCol' => 'row1column1value', 'secondCol' => 'row2column2value'],
-                  ['firstCol' => 'row2column1value', 'secondCol' => 'row2column2value']
-              ],
-              2
-          ]
+            [
+                [
+                    'firstCol' => new Column('title 1st col'),
+                    'secondCol' => new Column('title 2nd col')
+                ],
+                [
+                    ['firstCol' => 'row1column1value', 'secondCol' => 'row2column2value'],
+                    ['firstCol' => 'row2column1value', 'secondCol' => 'row2column2value']
+                ],
+                2
+            ]
         ];
     }
 }
\ No newline at end of file

Tests/DataTable/Request/PaginateRequestTest.php

Indentation +3 added lines, -3 removed lines

@@ -13,9 +13,9 @@
     {
         $tableMock = $this->getTableMock();
         $request = new Request([], [
-           'order' => [['column' => '1', 'dir' => 'desc']],
-           'start'  => 242,
-           'length' => 342
+            'order' => [['column' => '1', 'dir' => 'desc']],
+            'start'  => 242,
+            'length' => 342
         ]);
 
         $paginateRequest = PaginateRequest::fromHttpRequest($request, $tableMock);

Tests/DataTable/Formatter/DefaultFormatterTest.php

Indentation +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
         $formatter = new DefaultFormatter();
 
         $row = [
-          'escapedcolumn' => '<testval',
-          'unescapedcolumn' => '<testval',
+            'escapedcolumn' => '<testval',
+            'unescapedcolumn' => '<testval',
         ];
 
         $expected = [

Tests/Controller/DataTableControllerTest.php

Indentation +7 added lines, -7 removed lines

@@ -19,19 +19,19 @@
     {
         $tableMock = $this->getDataTableMock();
         $registryMock = $this->getMockBuilder('CrossKnowledge\DataTableBundle\DataTable\DataTableRegistry')
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $registryMock->expects($this->once())
-                     ->method('retrieveByTableId')
-                     ->will($this->returnValue($tableMock));
+                        ->method('retrieveByTableId')
+                        ->will($this->returnValue($tableMock));
 
         $rendererMock = $this->getMockBuilder('CrossKnowledge\DataTableBundle\DataTable\Renderer\JsonRenderer')
-                             ->getMock();
+                                ->getMock();
 
         $rendererMock->expects($this->once())
-                         ->method('renderJsonResponse')
-                         ->with($tableMock);
+                            ->method('renderJsonResponse')
+                            ->with($tableMock);
 
         $request = new Request([], [], ['tableid' => 'testtable_id']);