AC-15635: PHPUnit 12 Upgrade | Fefactor helper files

Author: manishranjan-adobe

…/Helper/AbstractCollectionTestHelper.php …/Helper/AbstractCollectionTestHelper.phpapp/code/Magento/CatalogUrlRewrite/Test/Unit/Helper/AbstractCollectionTestHelper.php renamed to app/code/Magento/Catalog/Test/Unit/Helper/AbstractCollectionTestHelper.php app/code/Magento/CatalogUrlRewrite/Test/Unit/Helper/AbstractCollectionTestHelper.php renamed to app/code/Magento/Catalog/Test/Unit/Helper/AbstractCollectionTestHelper.php

@@ -5,7 +5,7 @@
  */
 declare(strict_types=1);
 
-namespace Magento\CatalogUrlRewrite\Test\Unit\Helper;
+namespace Magento\Catalog\Test\Unit\Helper;
 
 use Magento\Catalog\Model\ResourceModel\Collection\AbstractCollection;
 
@@ -49,3 +49,4 @@ protected function _construct(): void
         // Mock implementation
     }
 }
+

app/code/Magento/Catalog/Test/Unit/Helper/CategoryInterfaceTestHelper.php

@@ -1,65 +1,198 @@
 <?php
 /**
- * Copyright 2017 Adobe
+ * Copyright 2015 Adobe
  * All Rights Reserved.
  */
 declare(strict_types=1);
 
 namespace Magento\Catalog\Test\Unit\Helper;
 
-use Magento\Catalog\Model\Category;
+use Magento\Catalog\Api\Data\CategoryInterface;
+use Magento\Framework\Api\CustomAttributesDataInterface;
 
 /**
- * Test helper for CategoryInterface
- *
- * This helper extends the concrete Category class to provide
- * test-specific functionality without dependency injection issues.
+ * Mock class for CategoryInterface with additional methods
  * 
  * @SuppressWarnings(PHPMD.UnusedFormalParameter)
  */
-class CategoryInterfaceTestHelper extends Category
+class CategoryInterfaceTestHelper implements CategoryInterface, CustomAttributesDataInterface
 {
+    /**
+     * @var mixed
+     */
+    private $resource = null;
+
     /**
      * @var mixed
      */
     private $isAnchor = null;
 
     /**
-     * Constructor that skips parent initialization
+     * Mock method for getResource
+     *
+     * @return mixed
+    */
+    public function getResource()
+    {
+        return $this->resource;
+    }
+
+    /**
+     * Set the resource
+     *
+     * @param mixed $resource
+     * @return $this
      */
-    public function __construct()
+    public function setResource($resource)
     {
-        // Skip parent constructor to avoid dependency injection issues
+        $this->resource = $resource;
+        return $this;
     }
 
     /**
      * Mock method for getIsAnchor
      *
-     * @return bool|null
+     * @return mixed
     */
     public function getIsAnchor()
     {
         return $this->isAnchor;
     }
 
     /**
-     * Set the isAnchor value
+     * Set is anchor
      *
-     * @param bool|null $value
+     * @param mixed $isAnchor
      * @return $this
      */
-    public function setIsAnchor($value)
+    public function setIsAnchor($isAnchor)
     {
-        $this->isAnchor = $value;
+        $this->isAnchor = $isAnchor;
         return $this;
     }
 
-    /**
-     * Required method from Category
-     */
-    protected function _construct(): void
+    // Required methods from CategoryInterface
+    public function getId(): ?int
     {
-        // Mock implementation
+        return null;
+    }
+    public function setId($id): CategoryInterface
+    {
+        return $this;
+    }
+    public function getParentId(): ?int
+    {
+        return null;
+    }
+    public function setParentId($parentId): CategoryInterface
+    {
+        return $this;
+    }
+    public function getName(): ?string
+    {
+        return null;
+    }
+    public function setName($name): CategoryInterface
+    {
+        return $this;
+    }
+    public function getIsActive(): ?int
+    {
+        return null;
+    }
+    public function setIsActive($isActive): CategoryInterface
+    {
+        return $this;
+    }
+    public function getPosition(): ?int
+    {
+        return null;
+    }
+    public function setPosition($position): CategoryInterface
+    {
+        return $this;
+    }
+    public function getLevel(): ?int
+    {
+        return null;
+    }
+    public function setLevel($level): CategoryInterface
+    {
+        return $this;
+    }
+    public function getPath(): ?string
+    {
+        return null;
+    }
+    public function setPath($path): CategoryInterface
+    {
+        return $this;
+    }
+    public function getAvailableSortBy(): ?array
+    {
+        return null;
+    }
+    public function setAvailableSortBy($availableSortBy): CategoryInterface
+    {
+        return $this;
+    }
+    public function getIncludeInMenu(): ?int
+    {
+        return null;
+    }
+    public function setIncludeInMenu($includeInMenu): CategoryInterface
+    {
+        return $this;
+    }
+    public function getChildren(): ?string
+    {
+        return null;
+    }
+    public function setChildren($children): CategoryInterface
+    {
+        return $this;
+    }
+    public function getCreatedAt(): ?string
+    {
+        return null;
+    }
+    public function setCreatedAt($createdAt): CategoryInterface
+    {
+        return $this;
+    }
+    public function getUpdatedAt(): ?string
+    {
+        return null;
+    }
+    public function setUpdatedAt($updatedAt): CategoryInterface
+    {
+        return $this;
+    }
+    public function getExtensionAttributes(): ?\Magento\Catalog\Api\Data\CategoryExtensionInterface
+    {
+        return null;
+    }
+    public function setExtensionAttributes(
+        \Magento\Catalog\Api\Data\CategoryExtensionInterface $extensionAttributes
+    ): CategoryInterface {
+        return $this;
     }
-}
 
+    // Required methods from CustomAttributesDataInterface
+    public function getCustomAttributes(): ?array
+    {
+        return null;
+    }
+    public function setCustomAttributes(array $customAttributes): CustomAttributesDataInterface
+    {
+        return $this;
+    }
+    public function getCustomAttribute($attributeCode): ?\Magento\Framework\Api\AttributeInterface
+    {
+        return null;
+    }
+    public function setCustomAttribute($attributeCode, $attributeValue): CustomAttributesDataInterface
+    {
+        return $this;
+    }
+}

app/code/Magento/Catalog/Test/Unit/Helper/CategoryTestHelper.php

@@ -10,31 +10,79 @@
 use Magento\Catalog\Model\Category;
 
 /**
- * TestHelper for Category with dynamic methods
+ * Mock class for Category with URL and product change methods
  */
 class CategoryTestHelper extends Category
 {
-    /** @var array */
-    private $changedProductIds = [];
+    /**
+     * Mock method for getChangedProductIds
+     *
+     * @return array
+     */
+    public function getChangedProductIds()
+    {
+        return $this->getData('changed_product_ids') ?: [];
+    }
 
-    public function __construct()
+    /**
+     * Mock method for getUrlPath
+     *
+     * @return string|null
+     */
+    public function getUrlPath()
     {
-        // Skip parent constructor to avoid complex dependencies
+        return $this->getData('url_path');
     }
 
-    public function getChangedProductIds()
+    /**
+     * Mock method for setUrlPath
+     *
+     * @param string $urlPath
+     * @return $this
+     */
+    public function setUrlPath($urlPath)
+    {
+        return $this->setData('url_path', $urlPath);
+    }
+
+    /**
+     * Mock method for unsUrlPath (unset url_path)
+     *
+     * @return $this
+     */
+    public function unsUrlPath()
+    {
+        return $this->unsetData('url_path');
+    }
+
+    /**
+     * Mock method for getUrlKey
+     *
+     * @return string|null
+     */
+    public function getUrlKey()
     {
-        return $this->changedProductIds;
+        return $this->getData('url_key');
     }
 
-    public function setChangedProductIds($value)
+    /**
+     * Mock method for setUrlKey
+     *
+     * @param string $urlKey
+     * @return $this
+     */
+    public function setUrlKey($urlKey)
     {
-        $this->changedProductIds = $value;
-        return $this;
+        return $this->setData('url_key', $urlKey);
     }
 
-    public function __wakeUp()
+    /**
+     * Initialize resources
+     *
+     * @return void
+     */
+    protected function _construct()
     {
-        // Implementation for __wakeUp method
+        // Mock implementation - no actual resource initialization needed
     }
 }

…t/Helper/ProductCollectionTestHelper.php …t/Helper/ProductCollectionTestHelper.phpapp/code/Magento/CatalogUrlRewrite/Test/Unit/Helper/ProductCollectionTestHelper.php renamed to app/code/Magento/Catalog/Test/Unit/Helper/ProductCollectionTestHelper.php app/code/Magento/CatalogUrlRewrite/Test/Unit/Helper/ProductCollectionTestHelper.php renamed to app/code/Magento/Catalog/Test/Unit/Helper/ProductCollectionTestHelper.php

@@ -5,7 +5,7 @@
  */
 declare(strict_types=1);
 
-namespace Magento\CatalogUrlRewrite\Test\Unit\Helper;
+namespace Magento\Catalog\Test\Unit\Helper;
 
 use Magento\Catalog\Model\ResourceModel\Product\Collection;
 
@@ -63,3 +63,4 @@ protected function _construct(): void
         // Mock implementation
     }
 }
+