Inspection of "Improve search date handling" - kitodo/kitodo-publication - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#64)

unknown

created 2018-01-12 12:44 UTC

Status

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

@@ -29,14 +29,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/_mapping',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/_mapping',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->setIndex('testIndex')
@@ -48,14 +48,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_mapping/testType',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_mapping/testType',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->setType('testType')
@@ -67,14 +67,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_mapping',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_mapping',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->performRequest();
@@ -85,14 +85,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/_mapping/testType',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/_mapping/testType',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->setIndex('testIndex')->setType('testType')

Please login to merge, or discard this patch.

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -33,7 +33,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/_mapping',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/_mapping/testType',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/_mapping',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/_mapping/testType',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +5 added lines, -10 removed lines patch added patch discarded remove patch

@@ -19,14 +19,12 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class GetTest extends \PHPUnit_Framework_TestCase
-{
+class GetTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
 
-    public function testValidGetWithIndex()
-    {
+    public function testValidGetWithIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -44,8 +42,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidGetWithType()
-    {
+    public function testValidGetWithType() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -63,8 +60,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidGetWithNoIndexOrType()
-    {
+    public function testValidGetWithNoIndexOrType() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -81,8 +77,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidGetWithIndexAndType()
-    {
+    public function testValidGetWithIndexAndType() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()

Please login to merge, or discard this patch.

Upper-Lower-Casing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -34,7 +34,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/_mapping',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/_mapping/testType',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/_mapping',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/_mapping/testType',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/Mapping/PutTest.php 3 patches

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -59,14 +59,14 @@
 block discarded – undo
         $body = array('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'PUT',
-                                 '/testIndex/testType/_mapping',
-                                 array(),
-                                 $body
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'PUT',
+                                    '/testIndex/testType/_mapping',
+                                    array(),
+                                    $body
+                                )
+                            ->getMock();
 
         $action = new Put($mockTransport);
         $action->setIndex('testIndex')->setType('testType')->setBody($body)

Please login to merge, or discard this patch.

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -35,7 +35,7 @@  discard block
 block discarded – undo
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
         $index = new Put($mockTransport);
-        $index->setIndex('testIndex')->setBody(array())->performRequest();
+        $index->setIndex('testIndex')->setBody(array ())->performRequest();
 
     }
 
@@ -56,14 +56,14 @@  discard block
 block discarded – undo
     public function testValidPutWithIndexTypeAndBody()
     {
 
-        $body = array('field' => 'value');
+        $body = array ('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
                          ->with(
                                  'PUT',
                                  '/testIndex/testType/_mapping',
-                                 array(),
+                                 array (),
                                  $body
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +4 added lines, -8 removed lines patch added patch discarded remove patch

@@ -20,8 +20,7 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class PutTest extends \PHPUnit_Framework_TestCase
-{
+class PutTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
@@ -29,8 +28,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testNoType()
-    {
+    public function testNoType() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -42,8 +40,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testNoBody()
-    {
+    public function testNoBody() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -53,8 +50,7 @@  discard block
 block discarded – undo
     }
 
 
-    public function testValidPutWithIndexTypeAndBody()
-    {
+    public function testValidPutWithIndexTypeAndBody() {
 
         $body = array('field' => 'value');
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/FlushTest.php 4 patches

Indentation +16 added lines, -16 removed lines patch added patch discarded remove patch

@@ -26,14 +26,14 @@  discard block
 block discarded – undo
     public function testValidFlushWithNoIndex()
     {
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_flush',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_flush',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Flush($mockTransport);
         $action->performRequest();
@@ -44,14 +44,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/_flush',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/_flush',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Flush($mockTransport);
         $action->setIndex('testIndex')

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -30,7 +30,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/_flush',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/_flush',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -17,14 +17,12 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class FlushTest extends \PHPUnit_Framework_TestCase
-{
+class FlushTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
 
-    public function testValidFlushWithNoIndex()
-    {
+    public function testValidFlushWithNoIndex() {
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
                          ->with(
@@ -40,8 +38,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidFlushWithIndex()
-    {
+    public function testValidFlushWithIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()

Please login to merge, or discard this patch.

Upper-Lower-Casing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -31,7 +31,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/_flush',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/_flush',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/Warmer/DeleteTest.php 4 patches

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -68,14 +68,14 @@
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'DELETE',
-                                 '/testIndex/_warmer/testName',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'DELETE',
+                                    '/testIndex/_warmer/testName',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Delete($mockTransport);
         $action->setIndex('testIndex')->setName('testName')->performRequest();

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -72,7 +72,7 @@
 block discarded – undo
                          ->with(
                                  'DELETE',
                                  '/testIndex/_warmer/testName',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +5 added lines, -10 removed lines patch added patch discarded remove patch

@@ -18,8 +18,7 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class DeleteTest extends \PHPUnit_Framework_TestCase
-{
+class DeleteTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
@@ -28,8 +27,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testDeleteNoIndex()
-    {
+    public function testDeleteNoIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -42,8 +40,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testDeleteWithOnlyIndex()
-    {
+    public function testDeleteWithOnlyIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -55,8 +52,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testDeleteWithOnlyName()
-    {
+    public function testDeleteWithOnlyName() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -64,8 +60,7 @@  discard block
 block discarded – undo
         $action->setName('testName')->performRequest();
 
     }
-    public function testDeleteWithIndexAndName()
-    {
+    public function testDeleteWithIndexAndName() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()

Please login to merge, or discard this patch.

Upper-Lower-Casing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -73,7 +73,7 @@
 block discarded – undo
                                  'DELETE',
                                  '/testIndex/_warmer/testName',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/Warmer/GetTest.php 4 patches

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

@@ -29,14 +29,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_warmer',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_warmer',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->performRequest();
@@ -47,14 +47,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/_warmer',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/_warmer',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->setIndex('testIndex')->performRequest();
@@ -79,14 +79,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/_warmer/testName',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/_warmer/testName',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->setIndex('testIndex')->setName('testName')->performRequest();
@@ -97,14 +97,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/testType/_warmer/testName',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/testType/_warmer/testName',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Get($mockTransport);
         $action->setIndex('testIndex')->setType('testType')->setName('testName')->performRequest();

Please login to merge, or discard this patch.

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -33,7 +33,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/_warmer',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/_warmer',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/_warmer/testName',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/testType/_warmer/testName',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

@@ -18,15 +18,13 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class GetTest extends \PHPUnit_Framework_TestCase
-{
+class GetTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
 
 
-    public function testGetNoIndex()
-    {
+    public function testGetNoIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -43,8 +41,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testGetWithIndex()
-    {
+    public function testGetWithIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -65,8 +62,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testGetWithIndexAndType()
-    {
+    public function testGetWithIndexAndType() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -75,8 +71,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testGetWithIndexAndName()
-    {
+    public function testGetWithIndexAndName() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -93,8 +88,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testGetWithIndexTypeAndName()
-    {
+    public function testGetWithIndexTypeAndName() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()

Please login to merge, or discard this patch.

Upper-Lower-Casing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -34,7 +34,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/_warmer',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/_warmer',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/_warmer/testName',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/testType/_warmer/testName',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/Warmer/PutTest.php 4 patches

Indentation +24 added lines, -24 removed lines patch added patch discarded remove patch

@@ -30,14 +30,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'PUT',
-                                 '/_warmer/testName',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'PUT',
+                                    '/_warmer/testName',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Put($mockTransport);
         $action->setName('testName')->setBody(array())->performRequest();
@@ -77,14 +77,14 @@  discard block
 block discarded – undo
         $body = array('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'PUT',
-                                 '/testIndex/_warmer/testName',
-                                 array(),
-                                 $body
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'PUT',
+                                    '/testIndex/_warmer/testName',
+                                    array(),
+                                    $body
+                                )
+                            ->getMock();
 
         $action = new Put($mockTransport);
         $action->setIndex('testIndex')->setName('testName')->setBody($body)->performRequest();
@@ -96,14 +96,14 @@  discard block
 block discarded – undo
         $body = array('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'PUT',
-                                 '/testIndex/testType/_warmer/testName',
-                                 array(),
-                                 $body
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'PUT',
+                                    '/testIndex/testType/_warmer/testName',
+                                    array(),
+                                    $body
+                                )
+                            ->getMock();
 
         $action = new Put($mockTransport);
         $action->setIndex('testIndex')->setName('testName')->setBody($body)->setType('testType')->performRequest();

Please login to merge, or discard this patch.

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch

@@ -34,13 +34,13 @@  discard block
 block discarded – undo
                          ->with(
                                  'PUT',
                                  '/_warmer/testName',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
 
         $action = new Put($mockTransport);
-        $action->setName('testName')->setBody(array())->performRequest();
+        $action->setName('testName')->setBody(array ())->performRequest();
 
     }
 
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
         $action = new Put($mockTransport);
-        $action->setIndex('testIndex')->setBody(array())->performRequest();
+        $action->setIndex('testIndex')->setBody(array ())->performRequest();
 
     }
 
@@ -74,14 +74,14 @@  discard block
 block discarded – undo
 
     public function testPutWithIndexNameAndBody()
     {
-        $body = array('field' => 'value');
+        $body = array ('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
                          ->with(
                                  'PUT',
                                  '/testIndex/_warmer/testName',
-                                 array(),
+                                 array (),
                                  $body
                              )
                          ->getMock();
@@ -93,14 +93,14 @@  discard block
 block discarded – undo
 
     public function testPutWithIndexTypeNameAndBody()
     {
-        $body = array('field' => 'value');
+        $body = array ('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
                          ->with(
                                  'PUT',
                                  '/testIndex/testType/_warmer/testName',
-                                 array(),
+                                 array (),
                                  $body
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

@@ -19,15 +19,13 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class PutTest extends \PHPUnit_Framework_TestCase
-{
+class PutTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
 
 
-    public function testPutNoIndex()
-    {
+    public function testPutNoIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -47,8 +45,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testPutNoBody()
-    {
+    public function testPutNoBody() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -60,8 +57,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testPutNoName()
-    {
+    public function testPutNoName() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -72,8 +68,7 @@  discard block
 block discarded – undo
 
 
 
-    public function testPutWithIndexNameAndBody()
-    {
+    public function testPutWithIndexNameAndBody() {
         $body = array('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
@@ -91,8 +86,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testPutWithIndexTypeNameAndBody()
-    {
+    public function testPutWithIndexTypeNameAndBody() {
         $body = array('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')

Please login to merge, or discard this patch.

Upper-Lower-Casing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -35,7 +35,7 @@
 block discarded – undo
                                  'PUT',
                                  '/_warmer/testName',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/SegmentsTest.php 4 patches

Indentation +16 added lines, -16 removed lines patch added patch discarded remove patch

@@ -26,14 +26,14 @@  discard block
 block discarded – undo
     public function testValidSegmentsWithNoIndex()
     {
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_segments',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_segments',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Segments($mockTransport);
         $action->performRequest();
@@ -44,14 +44,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/_segments',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/_segments',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Segments($mockTransport);
         $action->setIndex('testIndex')

Please login to merge, or discard this patch.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -30,7 +30,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/_segments',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/_segments',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -17,14 +17,12 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class SegmentsTest extends \PHPUnit_Framework_TestCase
-{
+class SegmentsTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
 
-    public function testValidSegmentsWithNoIndex()
-    {
+    public function testValidSegmentsWithNoIndex() {
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
                          ->with(
@@ -40,8 +38,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidSegmentsWithIndex()
-    {
+    public function testValidSegmentsWithIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()

Please login to merge, or discard this patch.

Upper-Lower-Casing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -31,7 +31,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/_segments',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/_segments',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/Validate/QueryTest.php 4 patches

Indentation +40 added lines, -40 removed lines patch added patch discarded remove patch

@@ -27,14 +27,14 @@  discard block
 block discarded – undo
     public function testValidQueryWithNoIndexNoType()
     {
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_all/_validate/query',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_all/_validate/query',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Query($mockTransport);
         $action->performRequest();
@@ -45,14 +45,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/_validate/query',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/_validate/query',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Query($mockTransport);
         $action->setIndex('testIndex')
@@ -64,14 +64,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_all/testType/_validate/query',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_all/testType/_validate/query',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Query($mockTransport);
         $action->setType('testType')
@@ -83,14 +83,14 @@  discard block
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/testIndex/testType/_validate/query',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/testIndex/testType/_validate/query',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Query($mockTransport);
         $action->setIndex('testIndex')->setType('testType')
@@ -104,14 +104,14 @@  discard block
 block discarded – undo
         $body = array('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'GET',
-                                 '/_all/_validate/query',
-                                 array(),
-                                 $body
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'GET',
+                                    '/_all/_validate/query',
+                                    array(),
+                                    $body
+                                )
+                            ->getMock();
 
         $action = new Query($mockTransport);
         $action->setBody($body)

Please login to merge, or discard this patch.

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -31,7 +31,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/_all/_validate/query',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/_validate/query',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/_all/testType/_validate/query',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
                          ->with(
                                  'GET',
                                  '/testIndex/testType/_validate/query',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();
@@ -101,14 +101,14 @@  discard block
 block discarded – undo
     public function testValidQueryWithBody()
     {
 
-        $body = array('field' => 'value');
+        $body = array ('field' => 'value');
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
                          ->with(
                                  'GET',
                                  '/_all/_validate/query',
-                                 array(),
+                                 array (),
                                  $body
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

@@ -18,14 +18,12 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class QueryTest extends \PHPUnit_Framework_TestCase
-{
+class QueryTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
 
-    public function testValidQueryWithNoIndexNoType()
-    {
+    public function testValidQueryWithNoIndexNoType() {
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
                          ->with(
@@ -41,8 +39,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidQueryWithIndex()
-    {
+    public function testValidQueryWithIndex() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -60,8 +57,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidQueryWithType()
-    {
+    public function testValidQueryWithType() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -79,8 +75,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidQueryWithIndexAndType()
-    {
+    public function testValidQueryWithIndexAndType() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()
@@ -98,8 +93,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidQueryWithBody()
-    {
+    public function testValidQueryWithBody() {
 
         $body = array('field' => 'value');
 

Please login to merge, or discard this patch.

Upper-Lower-Casing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -32,7 +32,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/_all/_validate/query',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/_validate/query',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/_all/testType/_validate/query',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
                                  'GET',
                                  '/testIndex/testType/_validate/query',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

tests/Elasticsearch/Tests/Endpoints/Indices/Template/DeleteTest.php 4 patches

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -41,14 +41,14 @@
 block discarded – undo
     {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
-                         ->shouldReceive('performRequest')->once()
-                         ->with(
-                                 'DELETE',
-                                 '/_template/testName',
-                                 array(),
-                                 null
-                             )
-                         ->getMock();
+                            ->shouldReceive('performRequest')->once()
+                            ->with(
+                                    'DELETE',
+                                    '/_template/testName',
+                                    array(),
+                                    null
+                                )
+                            ->getMock();
 
         $action = new Delete($mockTransport);
         $action->setIndex('testIndex')->setName('testName')

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -45,7 +45,7 @@
 block discarded – undo
                          ->with(
                                  'DELETE',
                                  '/_template/testName',
-                                 array(),
+                                 array (),
                                  null
                              )
                          ->getMock();

Please login to merge, or discard this patch.

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -18,8 +18,7 @@  discard block
 block discarded – undo
  * @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link    http://elasticsearch.org
  */
-class DeleteTest extends \PHPUnit_Framework_TestCase
-{
+class DeleteTest extends \PHPUnit_Framework_TestCase {
     public function tearDown() {
         m::close();
     }
@@ -27,8 +26,7 @@  discard block
 block discarded – undo
     /**
      * @expectedException RuntimeException
      */
-    public function testNoName()
-    {
+    public function testNoName() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport');
 
@@ -37,8 +35,7 @@  discard block
 block discarded – undo
 
     }
 
-    public function testValidDelete()
-    {
+    public function testValidDelete() {
 
         $mockTransport = m::mock('\Elasticsearch\Transport')
                          ->shouldReceive('performRequest')->once()

Please login to merge, or discard this patch.

Upper-Lower-Casing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -46,7 +46,7 @@
 block discarded – undo
                                  'DELETE',
                                  '/_template/testName',
                                  array(),
-                                 null
+                                 NULL
                              )
                          ->getMock();
 

Please login to merge, or discard this patch.

		@@ -29,14 +29,14 @@ discard block
		block discarded – undo
29	29	{
30	30
31	31	$mockTransport = m::mock('\Elasticsearch\Transport')
32		- ->shouldReceive('performRequest')->once()
33		- ->with(
34		- 'GET',
35		- '/testIndex/_mapping',
36		- array(),
37		- null
38		- )
39		- ->getMock();
	32	+ ->shouldReceive('performRequest')->once()
	33	+ ->with(
	34	+ 'GET',
	35	+ '/testIndex/_mapping',
	36	+ array(),
	37	+ null
	38	+ )
	39	+ ->getMock();
40	40
41	41	$action = new Get($mockTransport);
42	42	$action->setIndex('testIndex')
		@@ -48,14 +48,14 @@ discard block
		block discarded – undo
48	48	{
49	49
50	50	$mockTransport = m::mock('\Elasticsearch\Transport')
51		- ->shouldReceive('performRequest')->once()
52		- ->with(
53		- 'GET',
54		- '/_mapping/testType',
55		- array(),
56		- null
57		- )
58		- ->getMock();
	51	+ ->shouldReceive('performRequest')->once()
	52	+ ->with(
	53	+ 'GET',
	54	+ '/_mapping/testType',
	55	+ array(),
	56	+ null
	57	+ )
	58	+ ->getMock();
59	59
60	60	$action = new Get($mockTransport);
61	61	$action->setType('testType')
		@@ -67,14 +67,14 @@ discard block
		block discarded – undo
67	67	{
68	68
69	69	$mockTransport = m::mock('\Elasticsearch\Transport')
70		- ->shouldReceive('performRequest')->once()
71		- ->with(
72		- 'GET',
73		- '/_mapping',
74		- array(),
75		- null
76		- )
77		- ->getMock();
	70	+ ->shouldReceive('performRequest')->once()
	71	+ ->with(
	72	+ 'GET',
	73	+ '/_mapping',
	74	+ array(),
	75	+ null
	76	+ )
	77	+ ->getMock();
78	78
79	79	$action = new Get($mockTransport);
80	80	$action->performRequest();
		@@ -85,14 +85,14 @@ discard block
		block discarded – undo
85	85	{
86	86
87	87	$mockTransport = m::mock('\Elasticsearch\Transport')
88		- ->shouldReceive('performRequest')->once()
89		- ->with(
90		- 'GET',
91		- '/testIndex/_mapping/testType',
92		- array(),
93		- null
94		- )
95		- ->getMock();
	88	+ ->shouldReceive('performRequest')->once()
	89	+ ->with(
	90	+ 'GET',
	91	+ '/testIndex/_mapping/testType',
	92	+ array(),
	93	+ null
	94	+ )
	95	+ ->getMock();
96	96
97	97	$action = new Get($mockTransport);
98	98	$action->setIndex('testIndex')->setType('testType')

		@@ -33,7 +33,7 @@ discard block
		block discarded – undo
33	33	->with(
34	34	'GET',
35	35	'/testIndex/_mapping',
36		- array(),
	36	+ array (),
37	37	null
38	38	)
39	39	->getMock();
		@@ -52,7 +52,7 @@ discard block
		block discarded – undo
52	52	->with(
53	53	'GET',
54	54	'/_mapping/testType',
55		- array(),
	55	+ array (),
56	56	null
57	57	)
58	58	->getMock();
		@@ -71,7 +71,7 @@ discard block
		block discarded – undo
71	71	->with(
72	72	'GET',
73	73	'/_mapping',
74		- array(),
	74	+ array (),
75	75	null
76	76	)
77	77	->getMock();
		@@ -89,7 +89,7 @@ discard block
		block discarded – undo
89	89	->with(
90	90	'GET',
91	91	'/testIndex/_mapping/testType',
92		- array(),
	92	+ array (),
93	93	null
94	94	)
95	95	->getMock();

		@@ -19,14 +19,12 @@ discard block
		block discarded – undo
19	19	* @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
20	20	* @link http://elasticsearch.org
21	21	*/
22		-class GetTest extends \PHPUnit_Framework_TestCase
23		-{
	22	+class GetTest extends \PHPUnit_Framework_TestCase {
24	23	public function tearDown() {
25	24	m::close();
26	25	}
27	26
28		- public function testValidGetWithIndex()
29		- {
	27	+ public function testValidGetWithIndex() {
30	28
31	29	$mockTransport = m::mock('\Elasticsearch\Transport')
32	30	->shouldReceive('performRequest')->once()
		@@ -44,8 +42,7 @@ discard block
		block discarded – undo
44	42
45	43	}
46	44
47		- public function testValidGetWithType()
48		- {
	45	+ public function testValidGetWithType() {
49	46
50	47	$mockTransport = m::mock('\Elasticsearch\Transport')
51	48	->shouldReceive('performRequest')->once()
		@@ -63,8 +60,7 @@ discard block
		block discarded – undo
63	60
64	61	}
65	62
66		- public function testValidGetWithNoIndexOrType()
67		- {
	63	+ public function testValidGetWithNoIndexOrType() {
68	64
69	65	$mockTransport = m::mock('\Elasticsearch\Transport')
70	66	->shouldReceive('performRequest')->once()
		@@ -81,8 +77,7 @@ discard block
		block discarded – undo
81	77
82	78	}
83	79
84		- public function testValidGetWithIndexAndType()
85		- {
	80	+ public function testValidGetWithIndexAndType() {
86	81
87	82	$mockTransport = m::mock('\Elasticsearch\Transport')
88	83	->shouldReceive('performRequest')->once()

		@@ -34,7 +34,7 @@ discard block
		block discarded – undo
34	34	'GET',
35	35	'/testIndex/_mapping',
36	36	array(),
37		- null
	37	+ NULL
38	38	)
39	39	->getMock();
40	40
		@@ -53,7 +53,7 @@ discard block
		block discarded – undo
53	53	'GET',
54	54	'/_mapping/testType',
55	55	array(),
56		- null
	56	+ NULL
57	57	)
58	58	->getMock();
59	59
		@@ -72,7 +72,7 @@ discard block
		block discarded – undo
72	72	'GET',
73	73	'/_mapping',
74	74	array(),
75		- null
	75	+ NULL
76	76	)
77	77	->getMock();
78	78
		@@ -90,7 +90,7 @@ discard block
		block discarded – undo
90	90	'GET',
91	91	'/testIndex/_mapping/testType',
92	92	array(),
93		- null
	93	+ NULL
94	94	)
95	95	->getMock();
96	96

		@@ -59,14 +59,14 @@
		block discarded – undo
59	59	$body = array('field' => 'value');
60	60
61	61	$mockTransport = m::mock('\Elasticsearch\Transport')
62		- ->shouldReceive('performRequest')->once()
63		- ->with(
64		- 'PUT',
65		- '/testIndex/testType/_mapping',
66		- array(),
67		- $body
68		- )
69		- ->getMock();
	62	+ ->shouldReceive('performRequest')->once()
	63	+ ->with(
	64	+ 'PUT',
	65	+ '/testIndex/testType/_mapping',
	66	+ array(),
	67	+ $body
	68	+ )
	69	+ ->getMock();
70	70
71	71	$action = new Put($mockTransport);
72	72	$action->setIndex('testIndex')->setType('testType')->setBody($body)

		@@ -35,7 +35,7 @@ discard block
		block discarded – undo
35	35	$mockTransport = m::mock('\Elasticsearch\Transport');
36	36
37	37	$index = new Put($mockTransport);
38		- $index->setIndex('testIndex')->setBody(array())->performRequest();
	38	+ $index->setIndex('testIndex')->setBody(array ())->performRequest();
39	39
40	40	}
41	41
		@@ -56,14 +56,14 @@ discard block
		block discarded – undo
56	56	public function testValidPutWithIndexTypeAndBody()
57	57	{
58	58
59		- $body = array('field' => 'value');
	59	+ $body = array ('field' => 'value');
60	60
61	61	$mockTransport = m::mock('\Elasticsearch\Transport')
62	62	->shouldReceive('performRequest')->once()
63	63	->with(
64	64	'PUT',
65	65	'/testIndex/testType/_mapping',
66		- array(),
	66	+ array (),
67	67	$body
68	68	)
69	69	->getMock();

		@@ -20,8 +20,7 @@ discard block
		block discarded – undo
20	20	* @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
21	21	* @link http://elasticsearch.org
22	22	*/
23		-class PutTest extends \PHPUnit_Framework_TestCase
24		-{
	23	+class PutTest extends \PHPUnit_Framework_TestCase {
25	24	public function tearDown() {
26	25	m::close();
27	26	}
		@@ -29,8 +28,7 @@ discard block
		block discarded – undo
29	28	/**
30	29	* @expectedException RuntimeException
31	30	*/
32		- public function testNoType()
33		- {
	31	+ public function testNoType() {
34	32
35	33	$mockTransport = m::mock('\Elasticsearch\Transport');
36	34
		@@ -42,8 +40,7 @@ discard block
		block discarded – undo
42	40	/**
43	41	* @expectedException RuntimeException
44	42	*/
45		- public function testNoBody()
46		- {
	43	+ public function testNoBody() {
47	44
48	45	$mockTransport = m::mock('\Elasticsearch\Transport');
49	46
		@@ -53,8 +50,7 @@ discard block
		block discarded – undo
53	50	}
54	51
55	52
56		- public function testValidPutWithIndexTypeAndBody()
57		- {
	53	+ public function testValidPutWithIndexTypeAndBody() {
58	54
59	55	$body = array('field' => 'value');
60	56

		@@ -26,14 +26,14 @@ discard block
		block discarded – undo
26	26	public function testValidFlushWithNoIndex()
27	27	{
28	28	$mockTransport = m::mock('\Elasticsearch\Transport')
29		- ->shouldReceive('performRequest')->once()
30		- ->with(
31		- 'GET',
32		- '/_flush',
33		- array(),
34		- null
35		- )
36		- ->getMock();
	29	+ ->shouldReceive('performRequest')->once()
	30	+ ->with(
	31	+ 'GET',
	32	+ '/_flush',
	33	+ array(),
	34	+ null
	35	+ )
	36	+ ->getMock();
37	37
38	38	$action = new Flush($mockTransport);
39	39	$action->performRequest();
		@@ -44,14 +44,14 @@ discard block
		block discarded – undo
44	44	{
45	45
46	46	$mockTransport = m::mock('\Elasticsearch\Transport')
47		- ->shouldReceive('performRequest')->once()
48		- ->with(
49		- 'GET',
50		- '/testIndex/_flush',
51		- array(),
52		- null
53		- )
54		- ->getMock();
	47	+ ->shouldReceive('performRequest')->once()
	48	+ ->with(
	49	+ 'GET',
	50	+ '/testIndex/_flush',
	51	+ array(),
	52	+ null
	53	+ )
	54	+ ->getMock();
55	55
56	56	$action = new Flush($mockTransport);
57	57	$action->setIndex('testIndex')

		@@ -30,7 +30,7 @@ discard block
		block discarded – undo
30	30	->with(
31	31	'GET',
32	32	'/_flush',
33		- array(),
	33	+ array (),
34	34	null
35	35	)
36	36	->getMock();
		@@ -48,7 +48,7 @@ discard block
		block discarded – undo
48	48	->with(
49	49	'GET',
50	50	'/testIndex/_flush',
51		- array(),
	51	+ array (),
52	52	null
53	53	)
54	54	->getMock();

		@@ -17,14 +17,12 @@ discard block
		block discarded – undo
17	17	* @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
18	18	* @link http://elasticsearch.org
19	19	*/
20		-class FlushTest extends \PHPUnit_Framework_TestCase
21		-{
	20	+class FlushTest extends \PHPUnit_Framework_TestCase {
22	21	public function tearDown() {
23	22	m::close();
24	23	}
25	24
26		- public function testValidFlushWithNoIndex()
27		- {
	25	+ public function testValidFlushWithNoIndex() {
28	26	$mockTransport = m::mock('\Elasticsearch\Transport')
29	27	->shouldReceive('performRequest')->once()
30	28	->with(
		@@ -40,8 +38,7 @@ discard block
		block discarded – undo
40	38
41	39	}
42	40
43		- public function testValidFlushWithIndex()
44		- {
	41	+ public function testValidFlushWithIndex() {
45	42
46	43	$mockTransport = m::mock('\Elasticsearch\Transport')
47	44	->shouldReceive('performRequest')->once()

		@@ -31,7 +31,7 @@ discard block
		block discarded – undo
31	31	'GET',
32	32	'/_flush',
33	33	array(),
34		- null
	34	+ NULL
35	35	)
36	36	->getMock();
37	37
		@@ -49,7 +49,7 @@ discard block
		block discarded – undo
49	49	'GET',
50	50	'/testIndex/_flush',
51	51	array(),
52		- null
	52	+ NULL
53	53	)
54	54	->getMock();
55	55

		@@ -68,14 +68,14 @@
		block discarded – undo
68	68	{
69	69
70	70	$mockTransport = m::mock('\Elasticsearch\Transport')
71		- ->shouldReceive('performRequest')->once()
72		- ->with(
73		- 'DELETE',
74		- '/testIndex/_warmer/testName',
75		- array(),
76		- null
77		- )
78		- ->getMock();
	71	+ ->shouldReceive('performRequest')->once()
	72	+ ->with(
	73	+ 'DELETE',
	74	+ '/testIndex/_warmer/testName',
	75	+ array(),
	76	+ null
	77	+ )
	78	+ ->getMock();
79	79
80	80	$action = new Delete($mockTransport);
81	81	$action->setIndex('testIndex')->setName('testName')->performRequest();

		@@ -72,7 +72,7 @@
		block discarded – undo
72	72	->with(
73	73	'DELETE',
74	74	'/testIndex/_warmer/testName',
75		- array(),
	75	+ array (),
76	76	null
77	77	)
78	78	->getMock();

		@@ -18,8 +18,7 @@ discard block
		block discarded – undo
18	18	* @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
19	19	* @link http://elasticsearch.org
20	20	*/
21		-class DeleteTest extends \PHPUnit_Framework_TestCase
22		-{
	21	+class DeleteTest extends \PHPUnit_Framework_TestCase {
23	22	public function tearDown() {
24	23	m::close();
25	24	}
		@@ -28,8 +27,7 @@ discard block
		block discarded – undo
28	27	/**
29	28	* @expectedException RuntimeException
30	29	*/
31		- public function testDeleteNoIndex()
32		- {
	30	+ public function testDeleteNoIndex() {
33	31
34	32	$mockTransport = m::mock('\Elasticsearch\Transport');
35	33
		@@ -42,8 +40,7 @@ discard block
		block discarded – undo
42	40	/**
43	41	* @expectedException RuntimeException
44	42	*/
45		- public function testDeleteWithOnlyIndex()
46		- {
	43	+ public function testDeleteWithOnlyIndex() {
47	44
48	45	$mockTransport = m::mock('\Elasticsearch\Transport');
49	46
		@@ -55,8 +52,7 @@ discard block
		block discarded – undo
55	52	/**
56	53	* @expectedException RuntimeException
57	54	*/
58		- public function testDeleteWithOnlyName()
59		- {
	55	+ public function testDeleteWithOnlyName() {
60	56
61	57	$mockTransport = m::mock('\Elasticsearch\Transport');
62	58
		@@ -64,8 +60,7 @@ discard block
		block discarded – undo
64	60	$action->setName('testName')->performRequest();
65	61
66	62	}
67		- public function testDeleteWithIndexAndName()
68		- {
	63	+ public function testDeleteWithIndexAndName() {
69	64
70	65	$mockTransport = m::mock('\Elasticsearch\Transport')
71	66	->shouldReceive('performRequest')->once()

		@@ -73,7 +73,7 @@
		block discarded – undo
73	73	'DELETE',
74	74	'/testIndex/_warmer/testName',
75	75	array(),
76		- null
	76	+ NULL
77	77	)
78	78	->getMock();
79	79

		@@ -18,15 +18,13 @@ discard block
		block discarded – undo
18	18	* @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
19	19	* @link http://elasticsearch.org
20	20	*/
21		-class GetTest extends \PHPUnit_Framework_TestCase
22		-{
	21	+class GetTest extends \PHPUnit_Framework_TestCase {
23	22	public function tearDown() {
24	23	m::close();
25	24	}
26	25
27	26
28		- public function testGetNoIndex()
29		- {
	27	+ public function testGetNoIndex() {
30	28
31	29	$mockTransport = m::mock('\Elasticsearch\Transport')
32	30	->shouldReceive('performRequest')->once()
		@@ -43,8 +41,7 @@ discard block
		block discarded – undo
43	41
44	42	}
45	43
46		- public function testGetWithIndex()
47		- {
	44	+ public function testGetWithIndex() {
48	45
49	46	$mockTransport = m::mock('\Elasticsearch\Transport')
50	47	->shouldReceive('performRequest')->once()
		@@ -65,8 +62,7 @@ discard block
		block discarded – undo
65	62	/**
66	63	* @expectedException RuntimeException
67	64	*/
68		- public function testGetWithIndexAndType()
69		- {
	65	+ public function testGetWithIndexAndType() {
70	66
71	67	$mockTransport = m::mock('\Elasticsearch\Transport');
72	68
		@@ -75,8 +71,7 @@ discard block
		block discarded – undo
75	71
76	72	}
77	73
78		- public function testGetWithIndexAndName()
79		- {
	74	+ public function testGetWithIndexAndName() {
80	75
81	76	$mockTransport = m::mock('\Elasticsearch\Transport')
82	77	->shouldReceive('performRequest')->once()
		@@ -93,8 +88,7 @@ discard block
		block discarded – undo
93	88
94	89	}
95	90
96		- public function testGetWithIndexTypeAndName()
97		- {
	91	+ public function testGetWithIndexTypeAndName() {
98	92
99	93	$mockTransport = m::mock('\Elasticsearch\Transport')
100	94	->shouldReceive('performRequest')->once()

		@@ -30,14 +30,14 @@ discard block
		block discarded – undo
30	30	{
31	31
32	32	$mockTransport = m::mock('\Elasticsearch\Transport')
33		- ->shouldReceive('performRequest')->once()
34		- ->with(
35		- 'PUT',
36		- '/_warmer/testName',
37		- array(),
38		- null
39		- )
40		- ->getMock();
	33	+ ->shouldReceive('performRequest')->once()
	34	+ ->with(
	35	+ 'PUT',
	36	+ '/_warmer/testName',
	37	+ array(),
	38	+ null
	39	+ )
	40	+ ->getMock();
41	41
42	42	$action = new Put($mockTransport);
43	43	$action->setName('testName')->setBody(array())->performRequest();
		@@ -77,14 +77,14 @@ discard block
		block discarded – undo
77	77	$body = array('field' => 'value');
78	78
79	79	$mockTransport = m::mock('\Elasticsearch\Transport')
80		- ->shouldReceive('performRequest')->once()
81		- ->with(
82		- 'PUT',
83		- '/testIndex/_warmer/testName',
84		- array(),
85		- $body
86		- )
87		- ->getMock();
	80	+ ->shouldReceive('performRequest')->once()
	81	+ ->with(
	82	+ 'PUT',
	83	+ '/testIndex/_warmer/testName',
	84	+ array(),
	85	+ $body
	86	+ )
	87	+ ->getMock();
88	88
89	89	$action = new Put($mockTransport);
90	90	$action->setIndex('testIndex')->setName('testName')->setBody($body)->performRequest();
		@@ -96,14 +96,14 @@ discard block
		block discarded – undo
96	96	$body = array('field' => 'value');
97	97
98	98	$mockTransport = m::mock('\Elasticsearch\Transport')
99		- ->shouldReceive('performRequest')->once()
100		- ->with(
101		- 'PUT',
102		- '/testIndex/testType/_warmer/testName',
103		- array(),
104		- $body
105		- )
106		- ->getMock();
	99	+ ->shouldReceive('performRequest')->once()
	100	+ ->with(
	101	+ 'PUT',
	102	+ '/testIndex/testType/_warmer/testName',
	103	+ array(),
	104	+ $body
	105	+ )
	106	+ ->getMock();
107	107
108	108	$action = new Put($mockTransport);
109	109	$action->setIndex('testIndex')->setName('testName')->setBody($body)->setType('testType')->performRequest();

		@@ -34,13 +34,13 @@ discard block
		block discarded – undo
34	34	->with(
35	35	'PUT',
36	36	'/_warmer/testName',
37		- array(),
	37	+ array (),
38	38	null
39	39	)
40	40	->getMock();
41	41
42	42	$action = new Put($mockTransport);
43		- $action->setName('testName')->setBody(array())->performRequest();
	43	+ $action->setName('testName')->setBody(array ())->performRequest();
44	44
45	45	}
46	46
		@@ -66,7 +66,7 @@ discard block
		block discarded – undo
66	66	$mockTransport = m::mock('\Elasticsearch\Transport');
67	67
68	68	$action = new Put($mockTransport);
69		- $action->setIndex('testIndex')->setBody(array())->performRequest();
	69	+ $action->setIndex('testIndex')->setBody(array ())->performRequest();
70	70
71	71	}
72	72
		@@ -74,14 +74,14 @@ discard block
		block discarded – undo
74	74
75	75	public function testPutWithIndexNameAndBody()
76	76	{
77		- $body = array('field' => 'value');
	77	+ $body = array ('field' => 'value');
78	78
79	79	$mockTransport = m::mock('\Elasticsearch\Transport')
80	80	->shouldReceive('performRequest')->once()
81	81	->with(
82	82	'PUT',
83	83	'/testIndex/_warmer/testName',
84		- array(),
	84	+ array (),
85	85	$body
86	86	)
87	87	->getMock();
		@@ -93,14 +93,14 @@ discard block
		block discarded – undo
93	93
94	94	public function testPutWithIndexTypeNameAndBody()
95	95	{
96		- $body = array('field' => 'value');
	96	+ $body = array ('field' => 'value');
97	97
98	98	$mockTransport = m::mock('\Elasticsearch\Transport')
99	99	->shouldReceive('performRequest')->once()
100	100	->with(
101	101	'PUT',
102	102	'/testIndex/testType/_warmer/testName',
103		- array(),
	103	+ array (),
104	104	$body
105	105	)
106	106	->getMock();

		@@ -19,15 +19,13 @@ discard block
		block discarded – undo
19	19	* @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
20	20	* @link http://elasticsearch.org
21	21	*/
22		-class PutTest extends \PHPUnit_Framework_TestCase
23		-{
	22	+class PutTest extends \PHPUnit_Framework_TestCase {
24	23	public function tearDown() {
25	24	m::close();
26	25	}
27	26
28	27
29		- public function testPutNoIndex()
30		- {
	28	+ public function testPutNoIndex() {
31	29
32	30	$mockTransport = m::mock('\Elasticsearch\Transport')
33	31	->shouldReceive('performRequest')->once()
		@@ -47,8 +45,7 @@ discard block
		block discarded – undo
47	45	/**
48	46	* @expectedException RuntimeException
49	47	*/
50		- public function testPutNoBody()
51		- {
	48	+ public function testPutNoBody() {
52	49
53	50	$mockTransport = m::mock('\Elasticsearch\Transport');
54	51
		@@ -60,8 +57,7 @@ discard block
		block discarded – undo
60	57	/**
61	58	* @expectedException RuntimeException
62	59	*/
63		- public function testPutNoName()
64		- {
	60	+ public function testPutNoName() {
65	61
66	62	$mockTransport = m::mock('\Elasticsearch\Transport');
67	63
		@@ -72,8 +68,7 @@ discard block
		block discarded – undo
72	68
73	69
74	70
75		- public function testPutWithIndexNameAndBody()
76		- {
	71	+ public function testPutWithIndexNameAndBody() {
77	72	$body = array('field' => 'value');
78	73
79	74	$mockTransport = m::mock('\Elasticsearch\Transport')
		@@ -91,8 +86,7 @@ discard block
		block discarded – undo
91	86
92	87	}
93	88
94		- public function testPutWithIndexTypeNameAndBody()
95		- {
	89	+ public function testPutWithIndexTypeNameAndBody() {
96	90	$body = array('field' => 'value');
97	91
98	92	$mockTransport = m::mock('\Elasticsearch\Transport')

		@@ -35,7 +35,7 @@
		block discarded – undo
35	35	'PUT',
36	36	'/_warmer/testName',
37	37	array(),
38		- null
	38	+ NULL
39	39	)
40	40	->getMock();
41	41

kitodo / kitodo-publication

Pull Request — master (#64)

Status

Category

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

Braces +5 added lines, -10 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +4 added lines, -4 removed lines patch added patch discarded remove patch

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Braces +4 added lines, -8 removed lines patch added patch discarded remove patch

Indentation +16 added lines, -16 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +2 added lines, -2 removed lines patch added patch discarded remove patch

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Braces +5 added lines, -10 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +4 added lines, -4 removed lines patch added patch discarded remove patch

Indentation +24 added lines, -24 removed lines patch added patch discarded remove patch

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +16 added lines, -16 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +2 added lines, -2 removed lines patch added patch discarded remove patch

Indentation +40 added lines, -40 removed lines patch added patch discarded remove patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +4 added lines, -4 removed lines patch added patch discarded remove patch

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

Upper-Lower-Casing +1 added lines, -1 removed lines patch added patch discarded remove patch

		@@ -26,14 +26,14 @@ discard block
		block discarded – undo
26	26	public function testValidSegmentsWithNoIndex()
27	27	{
28	28	$mockTransport = m::mock('\Elasticsearch\Transport')
29		- ->shouldReceive('performRequest')->once()
30		- ->with(
31		- 'GET',
32		- '/_segments',
33		- array(),
34		- null
35		- )
36		- ->getMock();
	29	+ ->shouldReceive('performRequest')->once()
	30	+ ->with(
	31	+ 'GET',
	32	+ '/_segments',
	33	+ array(),
	34	+ null
	35	+ )
	36	+ ->getMock();
37	37
38	38	$action = new Segments($mockTransport);
39	39	$action->performRequest();
		@@ -44,14 +44,14 @@ discard block
		block discarded – undo
44	44	{
45	45
46	46	$mockTransport = m::mock('\Elasticsearch\Transport')
47		- ->shouldReceive('performRequest')->once()
48		- ->with(
49		- 'GET',
50		- '/testIndex/_segments',
51		- array(),
52		- null
53		- )
54		- ->getMock();
	47	+ ->shouldReceive('performRequest')->once()
	48	+ ->with(
	49	+ 'GET',
	50	+ '/testIndex/_segments',
	51	+ array(),
	52	+ null
	53	+ )
	54	+ ->getMock();
55	55
56	56	$action = new Segments($mockTransport);
57	57	$action->setIndex('testIndex')

		@@ -27,14 +27,14 @@ discard block
		block discarded – undo
27	27	public function testValidQueryWithNoIndexNoType()
28	28	{
29	29	$mockTransport = m::mock('\Elasticsearch\Transport')
30		- ->shouldReceive('performRequest')->once()
31		- ->with(
32		- 'GET',
33		- '/_all/_validate/query',
34		- array(),
35		- null
36		- )
37		- ->getMock();
	30	+ ->shouldReceive('performRequest')->once()
	31	+ ->with(
	32	+ 'GET',
	33	+ '/_all/_validate/query',
	34	+ array(),
	35	+ null
	36	+ )
	37	+ ->getMock();
38	38
39	39	$action = new Query($mockTransport);
40	40	$action->performRequest();
		@@ -45,14 +45,14 @@ discard block
		block discarded – undo
45	45	{
46	46
47	47	$mockTransport = m::mock('\Elasticsearch\Transport')
48		- ->shouldReceive('performRequest')->once()
49		- ->with(
50		- 'GET',
51		- '/testIndex/_validate/query',
52		- array(),
53		- null
54		- )
55		- ->getMock();
	48	+ ->shouldReceive('performRequest')->once()
	49	+ ->with(
	50	+ 'GET',
	51	+ '/testIndex/_validate/query',
	52	+ array(),
	53	+ null
	54	+ )
	55	+ ->getMock();
56	56
57	57	$action = new Query($mockTransport);
58	58	$action->setIndex('testIndex')
		@@ -64,14 +64,14 @@ discard block
		block discarded – undo
64	64	{
65	65
66	66	$mockTransport = m::mock('\Elasticsearch\Transport')
67		- ->shouldReceive('performRequest')->once()
68		- ->with(
69		- 'GET',
70		- '/_all/testType/_validate/query',
71		- array(),
72		- null
73		- )
74		- ->getMock();
	67	+ ->shouldReceive('performRequest')->once()
	68	+ ->with(
	69	+ 'GET',
	70	+ '/_all/testType/_validate/query',
	71	+ array(),
	72	+ null
	73	+ )
	74	+ ->getMock();
75	75
76	76	$action = new Query($mockTransport);
77	77	$action->setType('testType')
		@@ -83,14 +83,14 @@ discard block
		block discarded – undo
83	83	{
84	84
85	85	$mockTransport = m::mock('\Elasticsearch\Transport')
86		- ->shouldReceive('performRequest')->once()
87		- ->with(
88		- 'GET',
89		- '/testIndex/testType/_validate/query',
90		- array(),
91		- null
92		- )
93		- ->getMock();
	86	+ ->shouldReceive('performRequest')->once()
	87	+ ->with(
	88	+ 'GET',
	89	+ '/testIndex/testType/_validate/query',
	90	+ array(),
	91	+ null
	92	+ )
	93	+ ->getMock();
94	94
95	95	$action = new Query($mockTransport);
96	96	$action->setIndex('testIndex')->setType('testType')
		@@ -104,14 +104,14 @@ discard block
		block discarded – undo
104	104	$body = array('field' => 'value');
105	105
106	106	$mockTransport = m::mock('\Elasticsearch\Transport')
107		- ->shouldReceive('performRequest')->once()
108		- ->with(
109		- 'GET',
110		- '/_all/_validate/query',
111		- array(),
112		- $body
113		- )
114		- ->getMock();
	107	+ ->shouldReceive('performRequest')->once()
	108	+ ->with(
	109	+ 'GET',
	110	+ '/_all/_validate/query',
	111	+ array(),
	112	+ $body
	113	+ )
	114	+ ->getMock();
115	115
116	116	$action = new Query($mockTransport);
117	117	$action->setBody($body)

		@@ -18,14 +18,12 @@ discard block
		block discarded – undo
18	18	* @license http://www.apache.org/licenses/LICENSE-2.0 Apache2
19	19	* @link http://elasticsearch.org
20	20	*/
21		-class QueryTest extends \PHPUnit_Framework_TestCase
22		-{
	21	+class QueryTest extends \PHPUnit_Framework_TestCase {
23	22	public function tearDown() {
24	23	m::close();
25	24	}
26	25
27		- public function testValidQueryWithNoIndexNoType()
28		- {
	26	+ public function testValidQueryWithNoIndexNoType() {
29	27	$mockTransport = m::mock('\Elasticsearch\Transport')
30	28	->shouldReceive('performRequest')->once()
31	29	->with(
		@@ -41,8 +39,7 @@ discard block
		block discarded – undo
41	39
42	40	}
43	41
44		- public function testValidQueryWithIndex()
45		- {
	42	+ public function testValidQueryWithIndex() {
46	43
47	44	$mockTransport = m::mock('\Elasticsearch\Transport')
48	45	->shouldReceive('performRequest')->once()
		@@ -60,8 +57,7 @@ discard block
		block discarded – undo
60	57
61	58	}
62	59
63		- public function testValidQueryWithType()
64		- {
	60	+ public function testValidQueryWithType() {
65	61
66	62	$mockTransport = m::mock('\Elasticsearch\Transport')
67	63	->shouldReceive('performRequest')->once()
		@@ -79,8 +75,7 @@ discard block
		block discarded – undo
79	75
80	76	}
81	77
82		- public function testValidQueryWithIndexAndType()
83		- {
	78	+ public function testValidQueryWithIndexAndType() {
84	79
85	80	$mockTransport = m::mock('\Elasticsearch\Transport')
86	81	->shouldReceive('performRequest')->once()
		@@ -98,8 +93,7 @@ discard block
		block discarded – undo
98	93
99	94	}
100	95
101		- public function testValidQueryWithBody()
102		- {
	96	+ public function testValidQueryWithBody() {
103	97
104	98	$body = array('field' => 'value');
105	99

		@@ -32,7 +32,7 @@ discard block
		block discarded – undo
32	32	'GET',
33	33	'/_all/_validate/query',
34	34	array(),
35		- null
	35	+ NULL
36	36	)
37	37	->getMock();
38	38
		@@ -50,7 +50,7 @@ discard block
		block discarded – undo
50	50	'GET',
51	51	'/testIndex/_validate/query',
52	52	array(),
53		- null
	53	+ NULL
54	54	)
55	55	->getMock();
56	56
		@@ -69,7 +69,7 @@ discard block
		block discarded – undo
69	69	'GET',
70	70	'/_all/testType/_validate/query',
71	71	array(),
72		- null
	72	+ NULL
73	73	)
74	74	->getMock();
75	75
		@@ -88,7 +88,7 @@ discard block
		block discarded – undo
88	88	'GET',
89	89	'/testIndex/testType/_validate/query',
90	90	array(),
91		- null
	91	+ NULL
92	92	)
93	93	->getMock();
94	94

		@@ -41,14 +41,14 @@
		block discarded – undo
41	41	{
42	42
43	43	$mockTransport = m::mock('\Elasticsearch\Transport')
44		- ->shouldReceive('performRequest')->once()
45		- ->with(
46		- 'DELETE',
47		- '/_template/testName',
48		- array(),
49		- null
50		- )
51		- ->getMock();
	44	+ ->shouldReceive('performRequest')->once()
	45	+ ->with(
	46	+ 'DELETE',
	47	+ '/_template/testName',
	48	+ array(),
	49	+ null
	50	+ )
	51	+ ->getMock();
52	52
53	53	$action = new Delete($mockTransport);
54	54	$action->setIndex('testIndex')->setName('testName')

		@@ -45,7 +45,7 @@
		block discarded – undo
45	45	->with(
46	46	'DELETE',
47	47	'/_template/testName',
48		- array(),
	48	+ array (),
49	49	null
50	50	)
51	51	->getMock();

		@@ -46,7 +46,7 @@
		block discarded – undo
46	46	'DELETE',
47	47	'/_template/testName',
48	48	array(),
49		- null
	49	+ NULL
50	50	)
51	51	->getMock();
52	52