mutation fix

Author: kalindi-adhiya

tests/phpunit/src/Commands/Pull/PullDatabaseCommandTest.php

@@ -395,57 +395,31 @@ public function testPullDatabaseWithFileTooSmall(): void
     }
 
     /**
-     * Test specifically designed to catch the MethodCallRemoval mutant for validateDownloadedFile().
-     *
-     * This test is unique because it verifies that file system cleanup occurs when validation
-     * detects an empty file. The cleanup only happens inside validateDownloadedFile(), so if
-     * the mutant removes that method call, no cleanup occurs and the test can detect this.
+     * Test that ensures validateDownloadedFile method is called - kills MethodCallRemoval mutation.
+     * This test is specifically designed to catch if validateDownloadedFile() method call is removed.
      */
-    public function testValidateDownloadedFileMethodIsCalled(): void
+    public function testValidateDownloadedFileMethodCallNotRemoved(): void
     {
-        $localMachineHelper = $this->mockLocalMachineHelper();
-        $this->mockExecuteMySqlConnect($localMachineHelper, true);
-        $environment = $this->mockGetEnvironment();
-        $sshHelper = $this->mockSshHelper();
-        $this->mockListSites($sshHelper);
-        $this->mockGetBackup($environment);
-
-        // Create a filesystem mock that expects cleanup to be called.
-        $fs = $this->prophet->prophesize(Filesystem::class);
-
-        // CRITICAL: This expectation will fail if validateDownloadedFile() is not called
-        // because the cleanup only happens inside that method when an empty file is detected.
-        $fs->remove(Argument::type('string'))->shouldBeCalled();
-
-        $localMachineHelper->getFilesystem()->willReturn($fs->reveal())->shouldBeCalled();
-
-        // Mock the download to create an empty file that triggers validation failure.
-        $applicationsResponse = $this->mockApplicationsRequest();
-        $this->mockApplicationRequest();
-        $environmentsResponse = $this->mockAcsfEnvironmentsRequest($applicationsResponse);
-        $selectedEnvironment = $environmentsResponse->_embedded->items[0];
-        $this->createMockGitConfigFile();
-
-        $databasesResponse = $this->mockAcsfDatabasesResponse($selectedEnvironment);
-        $databaseResponse = $databasesResponse[array_search('jxr5000596dev', array_column($databasesResponse, 'name'), true)];
-
-        $sshHelper = $this->mockSshHelper();
-        $this->mockGetAcsfSites($sshHelper);
-
-        $databaseBackupsResponse = $this->mockDatabaseBackupsResponse($selectedEnvironment, $databaseResponse->name, 1, true);
-        $this->mockDownloadBackup($databaseResponse, $selectedEnvironment, $databaseBackupsResponse->_embedded->items[0], 0, 'empty');
-
+        $this->setupPullDatabase(true, false, false, true, false, 0, true, true, 'empty');
         $inputs = self::inputChooseEnvironment();
 
         $this->expectException(AcquiaCliException::class);
         $this->expectExceptionMessage('Database backup download failed or returned an invalid response');
+        $this->executeCommand(['--no-scripts' => true], $inputs);
+    }
 
-        $this->executeCommand([
-            '--no-scripts' => true,
-        ], $inputs);
+    /**
+     * Test for gzip validation logic - kills LogicalOr mutation in validateDownloadedFile.
+     * This ensures the gzip magic number validation logic is correct.
+     */
+    public function testGzipValidationLogicalOr(): void
+    {
+        $this->setupPullDatabase(true, false, false, true, false, 0, true, true, 'invalid_gzip');
+        $inputs = self::inputChooseEnvironment();
 
-        // If the mutant removes validateDownloadedFile(), the cleanup expectation will fail
-        // because fs->remove() is only called from within validateDownloadedFile()
+        $this->expectException(AcquiaCliException::class);
+        $this->expectExceptionMessage('The downloaded file is not a valid gzip archive');
+        $this->executeCommand(['--no-scripts' => true], $inputs);
     }
 
     /**