pkp#10306 removed direct mocking of job class

tests/jobs/doi/DepositSubmissionTest.php

@@ -20,6 +20,7 @@
 use PHPUnit\Framework\Attributes\CoversClass;
 use APP\submission\Repository as SubmissionRepository;
 use PHPUnit\Framework\Attributes\RunTestsInSeparateProcesses;
+use PKP\context\Context;
 
 #[RunTestsInSeparateProcesses]
 #[CoversClass(DepositSubmission::class)]
@@ -156,26 +157,18 @@ public function testRunSerializedJob(): void
 
         app()->instance(DoiRepository::class, $doiRepoMock);
 
+        /** @var \PKP\context\Context $contextMock */
         $contextMock = Mockery::mock(get_class(Application::getContextDAO()->newDataObject()))
             ->makePartial()
             ->shouldReceive('getData')
             ->getMock();
 
-        $depositSubmissionMock = Mockery::mock(DepositSubmission::class, [
-                0, $contextMock, new \PKP\tests\support\DoiRegistrationAgency
-            ])
-            ->shouldReceive('handle')
-            ->withAnyArgs()
-            ->andReturn(null)
-            ->getMock();
+        $depositSubmissionMock = new DepositSubmission(
+            0, 
+            $contextMock,
+            new \PKP\tests\support\DoiRegistrationAgency
+        );
 
-        /**
-         * @disregard P1013 PHP Intelephense error suppression
-         * @see https://github.com/bmewburn/vscode-intelephense/issues/568
-         * 
-         *  As mock defined it should receive `handle`,
-         *  we will have `handle` method on mock object
-         */
         $depositSubmissionMock->handle();
 
         $this->expectNotToPerformAssertions();