Apply style fixes

tests/Database/ConnectionTest.php

@@ -28,25 +28,25 @@ protected function setUp()
     }
 
     /** @test */
-    function get_a_new_query_instance()
+    public function get_a_new_query_instance()
     {
         $this->assertInstanceOf(Query::class, $this->connection->query());
     }
 
     /** @test */
-    function get_a_new_statement_instance()
+    public function get_a_new_statement_instance()
     {
         $this->assertInstanceOf(Statement::class, $this->connection->statement());
     }
 
     /** @test */
-    function get_a_new_transaction_instance()
+    public function get_a_new_transaction_instance()
     {
         $this->assertInstanceOf(Transaction::class, $this->connection->transaction(true));
     }
 
     /** @test */
-    function dynamically_pass_method_calls_to_the_pdo_instance()
+    public function dynamically_pass_method_calls_to_the_pdo_instance()
     {
         $this->pdo->shouldReceive('method')->once()->with('param');
 

tests/Database/ManagerTest.php

@@ -14,7 +14,7 @@ class ManagerTest extends TestCase
     use MockeryPHPUnitIntegration;
 
     /** @test */
-    function create_a_new_connection()
+    public function create_a_new_connection()
     {
         Etl::addConnection([
             'driver' => 'sqlite',

tests/Database/QueryTest.php

@@ -26,7 +26,7 @@ protected function setUp()
     }
 
     /** @test */
-    function select()
+    public function select()
     {
         $query = new Query($this->connection);
         $query->select('users');
@@ -40,7 +40,7 @@ function select()
     }
 
     /** @test */
-    function insert()
+    public function insert()
     {
         $query = new Query($this->connection);
         $query->insert('users', ['name' => 'Jane Doe', 'email' => 'janedoe@example.com']);
@@ -50,7 +50,7 @@ function insert()
     }
 
     /** @test */
-    function update()
+    public function update()
     {
         $query = new Query($this->connection);
         $query->update('users', ['name' => 'Jane Doe', 'email' => 'janedoe@example.com']);
@@ -60,7 +60,7 @@ function update()
     }
 
     /** @test */
-    function delete()
+    public function delete()
     {
         $query = new Query($this->connection);
         $query->delete('users');
@@ -70,7 +70,7 @@ function delete()
     }
 
     /** @test */
-    function where()
+    public function where()
     {
         $query = new Query($this->connection);
         $query->where(['name' => 'Jane Doe', 'email' => 'janedoe@example.com']);
@@ -80,7 +80,7 @@ function where()
     }
 
     /** @test */
-    function where_in()
+    public function where_in()
     {
         $query = new Query($this->connection);
         $query->whereIn('id', ['1', '2']);
@@ -90,7 +90,7 @@ function where_in()
     }
 
     /** @test */
-    function where_not_in()
+    public function where_not_in()
     {
         $query = new Query($this->connection);
         $query->whereNotIn('id', ['1', '2']);
@@ -100,7 +100,7 @@ function where_not_in()
     }
 
     /** @test */
-    function composite_where_in()
+    public function composite_where_in()
     {
         $query = new Query($this->connection);
         $query->whereIn(['id', 'company'], [['id' => '1', 'company' => '1'], ['id' => '2', 'company' => '1']]);
@@ -110,7 +110,7 @@ function composite_where_in()
     }
 
     /** @test */
-    function composite_where_not_in()
+    public function composite_where_not_in()
     {
         $query = new Query($this->connection);
         $query->whereNotIn(['id', 'company'], [['id' => '1', 'company' => '1'], ['id' => '2', 'company' => '1']]);
@@ -120,7 +120,7 @@ function composite_where_not_in()
     }
 
     /** @test */
-    function execute()
+    public function execute()
     {
         $this->connection->shouldReceive('prepare')->once()->with('')->andReturn($this->statement);
         $this->statement->shouldReceive('execute')->once()->with([]);

tests/Database/StatementTest.php

@@ -26,7 +26,7 @@ protected function setUp()
     }
 
     /** @test */
-    function select()
+    public function select()
     {
         $statement = new Statement($this->connection);
 
@@ -42,7 +42,7 @@ function select()
     }
 
     /** @test */
-    function insert()
+    public function insert()
     {
         $statement = new Statement($this->connection);
 
@@ -52,7 +52,7 @@ function insert()
     }
 
     /** @test */
-    function update()
+    public function update()
     {
         $statement = new Statement($this->connection);
 
@@ -62,7 +62,7 @@ function update()
     }
 
     /** @test */
-    function delete()
+    public function delete()
     {
         $statement = new Statement($this->connection);
 
@@ -72,7 +72,7 @@ function delete()
     }
 
     /** @test */
-    function where()
+    public function where()
     {
         $statement = new Statement($this->connection);
 
@@ -82,7 +82,7 @@ function where()
     }
 
     /** @test */
-    function prepare()
+    public function prepare()
     {
         $this->connection->shouldReceive('prepare')->once()->with('')->andReturn($this->statement);
 

tests/Database/TransactionTest.php

@@ -27,11 +27,13 @@ protected function setUp()
     }
 
     /** @test */
-    function no_transaction()
+    public function no_transaction()
     {
         $data = range(1, 20);
         $mode = 'none';
-        $callback = function ($row) { return $row; };
+        $callback = function ($row) {
+            return $row;
+        };
 
         $this->connection->shouldNotReceive('beginTransaction');
         $this->connection->shouldNotReceive('commit');
@@ -41,11 +43,13 @@ function no_transaction()
     }
 
     /** @test */
-    function single_transaction()
+    public function single_transaction()
     {
         $data = range(1, 20);
         $mode = 'single';
-        $callback = function ($row) { return $row; };
+        $callback = function ($row) {
+            return $row;
+        };
 
         $this->connection->shouldReceive('beginTransaction')->once();
         $this->connection->shouldReceive('commit')->once();
@@ -54,11 +58,13 @@ function single_transaction()
     }
 
     /** @test */
-    function transaction_size_is_multiple_of_data_size()
+    public function transaction_size_is_multiple_of_data_size()
     {
         $data = range(1, 20);
         $mode = 10;
-        $callback = function ($row) { return $row; };
+        $callback = function ($row) {
+            return $row;
+        };
 
         $this->connection->shouldReceive('beginTransaction')->times(2);
         $this->connection->shouldReceive('commit')->times(2);
@@ -67,11 +73,13 @@ function transaction_size_is_multiple_of_data_size()
     }
 
     /** @test */
-    function transaction_size_is_not_multiple_of_data_size()
+    public function transaction_size_is_not_multiple_of_data_size()
     {
         $data = range(1, 17);
         $mode = 10;
-        $callback = function ($row) { return $row; };
+        $callback = function ($row) {
+            return $row;
+        };
 
         $this->connection->shouldReceive('beginTransaction')->times(2);
         $this->connection->shouldReceive('commit')->times(2);
@@ -80,11 +88,13 @@ function transaction_size_is_not_multiple_of_data_size()
     }
 
     /** @test */
-    function transaction_rollback_on_error()
+    public function transaction_rollback_on_error()
     {
         $data = range(1, 20);
         $mode = 10;
-        $callback = function () { throw new Exception; };
+        $callback = function () {
+            throw new Exception;
+        };
 
         $this->connection->shouldReceive('beginTransaction')->times(1);
         $this->connection->shouldReceive('rollBack')->times(1);
@@ -95,11 +105,12 @@ function transaction_rollback_on_error()
     }
 
     /** @test */
-    function invalid_transaction_mode_throws_an_exception()
+    public function invalid_transaction_mode_throws_an_exception()
     {
         $data = range(1, 20);
         $mode = 'invalid';
-        $callback = function () {};
+        $callback = function () {
+        };
 
         $this->expectException(InvalidArgumentException::class);
 

tests/EtlTest.php

@@ -5,7 +5,7 @@
 class EtlTest extends TestCase
 {
     /** @test */
-    function get_and_set_configuration_options()
+    public function get_and_set_configuration_options()
     {
         $this->assertNull(Etl::get('nested.key'));
 
@@ -16,7 +16,7 @@ function get_and_set_configuration_options()
     }
 
     /** @test */
-    function add_connection()
+    public function add_connection()
     {
         $this->assertNull(Etl::get('connections.default'));
         $this->assertNull(Etl::get('connections.mysql'));
@@ -29,6 +29,7 @@ function add_connection()
     }
 }
 
-class Etl extends \Marquine\Etl\Etl {
+class Etl extends \Marquine\Etl\Etl
+{
     protected static $config = [];
 }

tests/Extractors/ArrTest.php

@@ -13,7 +13,7 @@ class ArrTest extends TestCase
     ];
 
     /** @test */
-    function extracts_data_from_an_array()
+    public function extracts_data_from_an_array()
     {
         $extractor = new Arr;
 
@@ -23,7 +23,7 @@ function extracts_data_from_an_array()
     }
 
     /** @test */
-    function extracts_specific_columns_from_an_array()
+    public function extracts_specific_columns_from_an_array()
     {
         $expected = [
             ['id' => 1, 'name' => 'John Doe'],

tests/Extractors/CsvTest.php

@@ -13,7 +13,7 @@ class CsvTest extends TestCase
     ];
 
     /** @test */
-    function extracts_data_from_a_csv_file()
+    public function extracts_data_from_a_csv_file()
     {
         $extractor = new Csv;
 
@@ -23,7 +23,7 @@ function extracts_data_from_a_csv_file()
     }
 
     /** @test */
-    function extracts_data_from_a_csv_file_with_custom_options()
+    public function extracts_data_from_a_csv_file_with_custom_options()
     {
         $extractor = new Csv;
 
@@ -37,7 +37,7 @@ function extracts_data_from_a_csv_file_with_custom_options()
     }
 
     /** @test */
-    function it_extracts_data_from_a_csv_file_without_a_header_line()
+    public function it_extracts_data_from_a_csv_file_without_a_header_line()
     {
         $extractor = new Csv;
 

tests/Extractors/FixedWidthTest.php

@@ -13,7 +13,7 @@ class FixedWidthTest extends TestCase
     ];
 
     /** @test */
-    function extracts_data_from_a_fixed_width_text_file()
+    public function extracts_data_from_a_fixed_width_text_file()
     {
         $extractor = new FixedWidth;
 

tests/Extractors/JsonTest.php

@@ -13,7 +13,7 @@ class JsonTest extends TestCase
     ];
 
     /** @test */
-    function extracts_data_from_a_json_file()
+    public function extracts_data_from_a_json_file()
     {
         $extractor = new Json;
 
@@ -23,7 +23,7 @@ function extracts_data_from_a_json_file()
     }
 
     /** @test */
-    function extracts_data_from_a_json_file_with_custom_attributes_path()
+    public function extracts_data_from_a_json_file_with_custom_attributes_path()
     {
         $extractor = new Json;