Fix styling

tpetry · github-actions[bot] · commit f4dd7b416839 · 2025-02-14T11:43:43.000Z
diff --git a/tests/LaravelQueryTest.php b/tests/LaravelQueryTest.php
@@ -11,7 +11,7 @@
 
 class LaravelQueryTest extends TestCase
 {
-    public function testConstructorValues(): void
+    public function test_constructor_values(): void
     {
         $date = new DateTimeImmutable('@1199699021');
         $query = new LaravelQuery(
@@ -30,7 +30,7 @@ public function testConstructorValues(): void
         $this->assertEquals([1, 1, 0, $date->format(DB::connection('mysql')->getQueryGrammar()->getDateFormat())], $query->getParameters());
     }
 
-    public function testExecuteWithoutParameters(): void
+    public function test_execute_without_parameters(): void
     {
         $query = new LaravelQuery(DB::connection(), "SELECT 'unused'", ['not-used']);
 
@@ -39,7 +39,7 @@ public function testExecuteWithoutParameters(): void
         $this->assertEquals([['val' => 1], ['val' => 2]], $rows);
     }
 
-    public function testExecuteWitParameters(): void
+    public function test_execute_wit_parameters(): void
     {
         $query = new LaravelQuery(DB::connection(), "SELECT 'unused'", [1, 2]);
 
diff --git a/tests/Mixins/EloquentBuilderTest.php b/tests/Mixins/EloquentBuilderTest.php
@@ -10,27 +10,27 @@
 
 class EloquentBuilderTest extends TestCase
 {
-    public function testDdExplainForHumans(): void
+    public function test_dd_explain_for_humans(): void
     {
         $this->markTestSkipped('how to test dump() and not exit PHPUnit on exit() call?');
     }
 
-    public function testDdVisualExplain(): void
+    public function test_dd_visual_explain(): void
     {
         $this->markTestSkipped('how to test dump() and not exit PHPUnit on exit() call?');
     }
 
-    public function testDumpExplainForHumans(): void
+    public function test_dump_explain_for_humans(): void
     {
         $this->markTestSkipped('how to test dump()?');
     }
 
-    public function testDumpVisualExplain(): void
+    public function test_dump_visual_explain(): void
     {
         $this->markTestSkipped('how to test dump()?');
     }
 
-    public function testExplainForHumans(): void
+    public function test_explain_for_humans(): void
     {
         $model = new class extends Model {};
         $builder = $model->newQuery();
@@ -45,7 +45,7 @@ public function testExplainForHumans(): void
         $this->assertEquals('https://dummy-url-A5mhRHJXvC.local/I2aifhDBCO', $url);
     }
 
-    public function testVisualExplain(): void
+    public function test_visual_explain(): void
     {
         $model = new class extends Model {};
         $builder = $model->newQuery();
diff --git a/tests/Mixins/QueryBuilderTest.php b/tests/Mixins/QueryBuilderTest.php
@@ -10,27 +10,27 @@
 
 class QueryBuilderTest extends TestCase
 {
-    public function testDdExplainForHumans(): void
+    public function test_dd_explain_for_humans(): void
     {
         $this->markTestSkipped('how to test dump() and not exit PHPUnit on exit() call?');
     }
 
-    public function testDdVisualExplain(): void
+    public function test_dd_visual_explain(): void
     {
         $this->markTestSkipped('how to test dump() and not exit PHPUnit on exit() call?');
     }
 
-    public function testDumpExplainForHumans(): void
+    public function test_dump_explain_for_humans(): void
     {
         $this->markTestSkipped('how to test dump()?');
     }
 
-    public function testDumpVisualExplain(): void
+    public function test_dump_visual_explain(): void
     {
         $this->markTestSkipped('how to test dump()?');
     }
 
-    public function testExplainForHumans(): void
+    public function test_explain_for_humans(): void
     {
         $builder = DB::table('test123');
         MysqlExplain::shouldReceive('submitBuilder')
@@ -43,7 +43,7 @@ public function testExplainForHumans(): void
         $this->assertEquals('https://dummy-url-f6V7VImZnz.local', $url);
     }
 
-    public function testVisualExplain(): void
+    public function test_visual_explain(): void
     {
         $builder = DB::table('test123');
         MysqlExplain::shouldReceive('submitBuilder')
diff --git a/tests/MysqlExplainTest.php b/tests/MysqlExplainTest.php
@@ -19,7 +19,7 @@
 
 class MysqlExplainTest extends TestCase
 {
-    public function testSubmitBuilderForwardsCall(): void
+    public function test_submit_builder_forwards_call(): void
     {
         $builder = DB::connection()->table('test')->where('col', 1);
         $mysqlExplain = Mockery::mock(MysqlExplain::class.'[submitQuery]')
@@ -34,7 +34,7 @@ public function testSubmitBuilderForwardsCall(): void
         $this->assertEquals('https://dummy-url-LTGq4mDWlo.local/PTzphBiWQW', $url);
     }
 
-    public function testSubmitQueryFailsWhenMariadbFeatureIsDetected(): void
+    public function test_submit_query_fails_when_mariadb_feature_is_detected(): void
     {
         $this->expectException(NotMysqlException::class);
         $this->expectExceptionMessage('Only queries on mysql databases can be analyzed. mariadb query given.');
@@ -49,7 +49,7 @@ public function testSubmitQueryFailsWhenMariadbFeatureIsDetected(): void
         (new MysqlExplain)->submitQuery($connection, 'SELECT 1');
     }
 
-    public function testSubmitQueryFailsWhenPassedNonMysqlConnection(): void
+    public function test_submit_query_fails_when_passed_non_mysql_connection(): void
     {
         $this->expectException(NotMysqlException::class);
         $this->expectExceptionMessage('Only queries on mysql databases can be analyzed. mariadb query given.');
@@ -62,15 +62,15 @@ public function testSubmitQueryFailsWhenPassedNonMysqlConnection(): void
         (new MysqlExplain)->submitQuery($connection, 'SELECT 1');
     }
 
-    public function testSubmitQueryFailsWhenPassedNonPdoConnection(): void
+    public function test_submit_query_fails_when_passed_non_pdo_connection(): void
     {
         $this->expectException(NotMysqlException::class);
         $this->expectExceptionMessage('Only queries on mysql databases can be analyzed. unknown query given.');
 
         (new MysqlExplain)->submitQuery(Mockery::mock(ConnectionInterface::class), 'SELECT 1');
     }
 
-    public function testSubmitQueryReturnsUrl(): void
+    public function test_submit_query_returns_url(): void
     {
         $connection = DB::connection('mysql');
         $sql = 'SELECT * FROM customer WHERE last_name = ?';
diff --git a/tests/NotMysqlExceptionTest.php b/tests/NotMysqlExceptionTest.php
@@ -8,15 +8,15 @@
 
 class NotMysqlExceptionTest extends TestCase
 {
-    public function testCreatesExceptionWithDrivername(): void
+    public function test_creates_exception_with_drivername(): void
     {
         $exception = NotMysqlException::create('mysql');
 
         $this->assertEquals(NotMysqlException::class, $exception::class);
         $this->assertEquals('Only queries on mysql databases can be analyzed. mysql query given.', $exception->getMessage());
     }
 
-    public function testCreatesExceptionWithMissingDrivername(): void
+    public function test_creates_exception_with_missing_drivername(): void
     {
         $exception = NotMysqlException::create(null);
 

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`
`12`	`12`	`class LaravelQueryTest extends TestCase`
`13`	`13`	`{`
`14`		`- public function testConstructorValues(): void`
	`14`	`+ public function test_constructor_values(): void`
`15`	`15`	`{`
`16`	`16`	`$date = new DateTimeImmutable('@1199699021');`
`17`	`17`	`$query = new LaravelQuery(`
`@@ -30,7 +30,7 @@ public function testConstructorValues(): void`
`30`	`30`	`$this->assertEquals([1, 1, 0, $date->format(DB::connection('mysql')->getQueryGrammar()->getDateFormat())], $query->getParameters());`
`31`	`31`	`}`
`32`	`32`
`33`		`- public function testExecuteWithoutParameters(): void`
	`33`	`+ public function test_execute_without_parameters(): void`
`34`	`34`	`{`
`35`	`35`	`$query = new LaravelQuery(DB::connection(), "SELECT 'unused'", ['not-used']);`
`36`	`36`
`@@ -39,7 +39,7 @@ public function testExecuteWithoutParameters(): void`
`39`	`39`	`$this->assertEquals([['val' => 1], ['val' => 2]], $rows);`
`40`	`40`	`}`
`41`	`41`
`42`		`- public function testExecuteWitParameters(): void`
	`42`	`+ public function test_execute_wit_parameters(): void`
`43`	`43`	`{`
`44`	`44`	`$query = new LaravelQuery(DB::connection(), "SELECT 'unused'", [1, 2]);`
`45`	`45`
Original file line number	Diff line number	Diff line change
`@@ -10,27 +10,27 @@`
`10`	`10`
`11`	`11`	`class EloquentBuilderTest extends TestCase`
`12`	`12`	`{`
`13`		`- public function testDdExplainForHumans(): void`
	`13`	`+ public function test_dd_explain_for_humans(): void`
`14`	`14`	`{`
`15`	`15`	`$this->markTestSkipped('how to test dump() and not exit PHPUnit on exit() call?');`
`16`	`16`	`}`
`17`	`17`
`18`		`- public function testDdVisualExplain(): void`
	`18`	`+ public function test_dd_visual_explain(): void`
`19`	`19`	`{`
`20`	`20`	`$this->markTestSkipped('how to test dump() and not exit PHPUnit on exit() call?');`
`21`	`21`	`}`
`22`	`22`
`23`		`- public function testDumpExplainForHumans(): void`
	`23`	`+ public function test_dump_explain_for_humans(): void`
`24`	`24`	`{`
`25`	`25`	`$this->markTestSkipped('how to test dump()?');`
`26`	`26`	`}`
`27`	`27`
`28`		`- public function testDumpVisualExplain(): void`
	`28`	`+ public function test_dump_visual_explain(): void`
`29`	`29`	`{`
`30`	`30`	`$this->markTestSkipped('how to test dump()?');`
`31`	`31`	`}`
`32`	`32`
`33`		`- public function testExplainForHumans(): void`
	`33`	`+ public function test_explain_for_humans(): void`
`34`	`34`	`{`
`35`	`35`	`$model = new class extends Model {};`
`36`	`36`	`$builder = $model->newQuery();`
`@@ -45,7 +45,7 @@ public function testExplainForHumans(): void`
`45`	`45`	`$this->assertEquals('https://dummy-url-A5mhRHJXvC.local/I2aifhDBCO', $url);`
`46`	`46`	`}`
`47`	`47`
`48`		`- public function testVisualExplain(): void`
	`48`	`+ public function test_visual_explain(): void`
`49`	`49`	`{`
`50`	`50`	`$model = new class extends Model {};`
`51`	`51`	`$builder = $model->newQuery();`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@`
`19`	`19`
`20`	`20`	`class MysqlExplainTest extends TestCase`
`21`	`21`	`{`
`22`		`- public function testSubmitBuilderForwardsCall(): void`
	`22`	`+ public function test_submit_builder_forwards_call(): void`
`23`	`23`	`{`
`24`	`24`	`$builder = DB::connection()->table('test')->where('col', 1);`
`25`	`25`	`$mysqlExplain = Mockery::mock(MysqlExplain::class.'[submitQuery]')`
`@@ -34,7 +34,7 @@ public function testSubmitBuilderForwardsCall(): void`
`34`	`34`	`$this->assertEquals('https://dummy-url-LTGq4mDWlo.local/PTzphBiWQW', $url);`
`35`	`35`	`}`
`36`	`36`
`37`		`- public function testSubmitQueryFailsWhenMariadbFeatureIsDetected(): void`
	`37`	`+ public function test_submit_query_fails_when_mariadb_feature_is_detected(): void`
`38`	`38`	`{`
`39`	`39`	`$this->expectException(NotMysqlException::class);`
`40`	`40`	`$this->expectExceptionMessage('Only queries on mysql databases can be analyzed. mariadb query given.');`
`@@ -49,7 +49,7 @@ public function testSubmitQueryFailsWhenMariadbFeatureIsDetected(): void`
`49`	`49`	`(new MysqlExplain)->submitQuery($connection, 'SELECT 1');`
`50`	`50`	`}`
`51`	`51`
`52`		`- public function testSubmitQueryFailsWhenPassedNonMysqlConnection(): void`
	`52`	`+ public function test_submit_query_fails_when_passed_non_mysql_connection(): void`
`53`	`53`	`{`
`54`	`54`	`$this->expectException(NotMysqlException::class);`
`55`	`55`	`$this->expectExceptionMessage('Only queries on mysql databases can be analyzed. mariadb query given.');`
`@@ -62,15 +62,15 @@ public function testSubmitQueryFailsWhenPassedNonMysqlConnection(): void`
`62`	`62`	`(new MysqlExplain)->submitQuery($connection, 'SELECT 1');`
`63`	`63`	`}`
`64`	`64`
`65`		`- public function testSubmitQueryFailsWhenPassedNonPdoConnection(): void`
	`65`	`+ public function test_submit_query_fails_when_passed_non_pdo_connection(): void`
`66`	`66`	`{`
`67`	`67`	`$this->expectException(NotMysqlException::class);`
`68`	`68`	`$this->expectExceptionMessage('Only queries on mysql databases can be analyzed. unknown query given.');`
`69`	`69`
`70`	`70`	`(new MysqlExplain)->submitQuery(Mockery::mock(ConnectionInterface::class), 'SELECT 1');`
`71`	`71`	`}`
`72`	`72`
`73`		`- public function testSubmitQueryReturnsUrl(): void`
	`73`	`+ public function test_submit_query_returns_url(): void`
`74`	`74`	`{`
`75`	`75`	`$connection = DB::connection('mysql');`
`76`	`76`	`$sql = 'SELECT * FROM customer WHERE last_name = ?';`
Original file line number	Diff line number	Diff line change
`@@ -8,15 +8,15 @@`
`8`	`8`
`9`	`9`	`class NotMysqlExceptionTest extends TestCase`
`10`	`10`	`{`
`11`		`- public function testCreatesExceptionWithDrivername(): void`
	`11`	`+ public function test_creates_exception_with_drivername(): void`
`12`	`12`	`{`
`13`	`13`	`$exception = NotMysqlException::create('mysql');`
`14`	`14`
`15`	`15`	`$this->assertEquals(NotMysqlException::class, $exception::class);`
`16`	`16`	`$this->assertEquals('Only queries on mysql databases can be analyzed. mysql query given.', $exception->getMessage());`
`17`	`17`	`}`
`18`	`18`
`19`		`- public function testCreatesExceptionWithMissingDrivername(): void`
	`19`	`+ public function test_creates_exception_with_missing_drivername(): void`
`20`	`20`	`{`
`21`	`21`	`$exception = NotMysqlException::create(null);`
`22`	`22`