Fix test mocks: return int from getCurrentTime, add setCreatedTime to node stub

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: mowens3

modules/mcp_tools_batch/tests/src/Unit/Service/BatchServiceTest.php

@@ -20,6 +20,12 @@
 #[\PHPUnit\Framework\Attributes\Group('mcp_tools_batch')]
 final class BatchServiceTest extends UnitTestCase {
 
+  private function mockTime(): TimeInterface {
+    $time = $this->createMock(TimeInterface::class);
+    $time->method('getCurrentTime')->willReturn(time());
+    return $time;
+  }
+
   private function createService(): BatchService {
     return new class(
       $this->createMock(EntityTypeManagerInterface::class),
@@ -50,7 +56,7 @@ private function createBatchService(array $overrides = []): BatchService {
       $overrides['module_handler'] ?? $this->createMock(ModuleHandlerInterface::class),
       $overrides['access_manager'] ?? $this->createMock(AccessManager::class),
       $overrides['audit_logger'] ?? $this->createMock(AuditLogger::class),
-      $overrides['time'] ?? $this->createMock(TimeInterface::class),
+      $overrides['time'] ?? $this->mockTime(),
     );
   }
 
@@ -124,6 +130,8 @@ public function testCreateMultipleContentCreatesItemsViaStorage(): void {
       public function save(): void {}
       public function id(): int { return 123; }
       public function uuid(): string { return 'uuid-123'; }
+      public function setCreatedTime(int $timestamp): void {}
+      public function setChangedTime(int $timestamp): void {}
       public function toUrl(): object {
         return new class() { public function toString(): string { return '/node/123'; } };
       }

modules/mcp_tools_content/tests/src/Unit/Service/ContentServiceTest.php

@@ -26,6 +26,12 @@
 #[\PHPUnit\Framework\Attributes\Group('mcp_tools_content')]
 class ContentServiceTest extends UnitTestCase {
 
+  protected function mockTime(): TimeInterface {
+    $time = $this->createMock(TimeInterface::class);
+    $time->method('getCurrentTime')->willReturn(time());
+    return $time;
+  }
+
   protected EntityTypeManagerInterface $entityTypeManager;
   protected EntityFieldManagerInterface $entityFieldManager;
   protected AccountProxyInterface $currentUser;
@@ -68,7 +74,7 @@ protected function createContentService(): ContentService {
       $this->currentUser,
       $this->accessManager,
       $this->auditLogger,
-      $this->createMock(TimeInterface::class),
+      $this->mockTime(),
     );
   }
 

modules/mcp_tools_media/tests/src/Unit/Service/MediaServiceTest.php

@@ -22,6 +22,12 @@
 #[\PHPUnit\Framework\Attributes\Group('mcp_tools_media')]
 class MediaServiceTest extends UnitTestCase {
 
+  protected function mockTime(): TimeInterface {
+    $time = $this->createMock(TimeInterface::class);
+    $time->method('getCurrentTime')->willReturn(time());
+    return $time;
+  }
+
   protected EntityTypeManagerInterface $entityTypeManager;
   protected FileSystemInterface $fileSystem;
   protected AccessManager $accessManager;
@@ -62,7 +68,7 @@ protected function createMediaService(): MediaService {
       $this->fileSystem,
       $this->accessManager,
       $this->auditLogger,
-      $this->createMock(TimeInterface::class),
+      $this->mockTime(),
     );
   }