[11.x] Fix expected/actual argument order for test assertion (#52084)

* Fix expected/actual argument order for test assertion * Fix test * Fix test
laravel · Jul 10, 2024 · 4fafdd9 · 4fafdd9
1 parent 68e88bb
commit 4fafdd9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Illuminate/Foundation/Testing/Concerns/InteractsWithDatabase.php b/src/Illuminate/Foundation/Testing/Concerns/InteractsWithDatabase.php
@@ -199,8 +199,8 @@ public function expectsDatabaseQueryCount($expected, $connection = null)
 
             $this->beforeApplicationDestroyed(function () use (&$actual, $expected, $connectionInstance) {
                 $this->assertSame(
-                    $actual,
                     $expected,
+                    $actual,
                     "Expected {$expected} database queries on the [{$connectionInstance->getName()}] connection. {$actual} occurred."
                 );
             });

diff --git a/tests/Foundation/FoundationInteractsWithDatabaseTest.php b/tests/Foundation/FoundationInteractsWithDatabaseTest.php
@@ -392,7 +392,7 @@ public function testExpectsDatabaseQueryCount()
             $case->tearDown();
             $this->fail();
         } catch (ExpectationFailedException $e) {
-            $this->assertSame("Expected 3 database queries on the [testing] connection. 0 occurred.\nFailed asserting that 3 is identical to 0.", $e->getMessage());
+            $this->assertSame("Expected 3 database queries on the [testing] connection. 0 occurred.\nFailed asserting that 0 is identical to 3.", $e->getMessage());
         }
 
         $case = new class('foo') extends TestingTestCase
@@ -419,7 +419,7 @@ public function testExpectsDatabaseQueryCount()
             $case->tearDown();
             $this->fail();
         } catch (ExpectationFailedException $e) {
-            $this->assertSame("Expected 3 database queries on the [testing] connection. 4 occurred.\nFailed asserting that 3 is identical to 4.", $e->getMessage());
+            $this->assertSame("Expected 3 database queries on the [testing] connection. 4 occurred.\nFailed asserting that 4 is identical to 3.", $e->getMessage());
         }
 
         $case = new class('foo') extends TestingTestCase