Updated test

Author: Pieter Zandbergen

dev/tests/integration/testsuite/Magento/Catalog/Model/ResourceModel/UrlTest.php

@@ -7,6 +7,15 @@
 
 namespace Magento\Catalog\Model\ResourceModel;
 
+use Magento\Catalog\Api\CategoryRepositoryInterface;
+use Magento\Catalog\Test\Fixture\Category as CategoryFixture;
+use Magento\Framework\Exception\CouldNotSaveException;
+use Magento\Framework\Exception\NoSuchEntityException;
+use Magento\Store\Model\StoreManagerInterface;
+use Magento\TestFramework\Fixture\DataFixture;
+use Magento\TestFramework\Fixture\DataFixtureStorage;
+use Magento\TestFramework\Fixture\DataFixtureStorageManager;
+use Magento\TestFramework\Fixture\DbIsolation;
 use Magento\TestFramework\Helper\Bootstrap;
 use PHPUnit\Framework\TestCase;
 
@@ -15,28 +24,71 @@
  */
 class UrlTest extends TestCase
 {
+    private CategoryRepositoryInterface $categoryRepository;
+    private DataFixtureStorage $fixtures;
+    private StoreManagerInterface $storeManager;
     private Url $urlResource;
 
-    /**
-     * @inheritdoc
-     */
     protected function setUp(): void
     {
         $objectManager = Bootstrap::getObjectManager();
+        $this->categoryRepository = $objectManager->create(CategoryRepositoryInterface::class);
+        $this->fixtures = $objectManager->get(DataFixtureStorageManager::class)->getStorage();
+        $this->storeManager = $objectManager->create(StoreManagerInterface::class);
         $this->urlResource = $objectManager->create(Url::class);
     }
 
     /**
      * Test that scope is respected for the is_active flag.
      *
      * @return void
-     * @magentoDataFixture Magento/Catalog/_files/categories_disabled_different_scopes.php
+     * @throws NoSuchEntityException|CouldNotSaveException
      */
+    #[
+        DbIsolation(true),
+        DataFixture(CategoryFixture::class, [
+            'name' => 'Enabled on default scope',
+            'is_active' => '1',
+        ], 'c1'),
+        DataFixture(CategoryFixture::class, [
+            'name' => 'Disabled on default scope',
+            'is_active' => '0',
+        ], 'c2'),
+        DataFixture(CategoryFixture::class, [
+            'name' => 'Enabled on default scope, disabled for store',
+            'is_active' => '1',
+        ], 'c3'),
+        DataFixture(CategoryFixture::class, [
+            'name' => 'Disabled on default scope, enabled for store',
+            'is_active' => '0',
+        ], 'c4'),
+    ]
     public function testIsActiveScope(): void
     {
-        $categories = $this->urlResource->getCategories([3, 4, 5], 1);
-        $this->assertTrue((bool) $categories[3]->getIsActive());
-        $this->assertFalse((bool) $categories[4]->getIsActive());
-        $this->assertFalse((bool) $categories[5]->getIsActive());
+        // Get Store ID
+        $storeId = (int) $this->storeManager->getStore('default')->getId();
+
+        // Get Category IDs
+        $fixtureNames = ['c1', 'c2', 'c3', 'c4'];
+        $categoryIds = array_combine($fixtureNames, array_map(function (string $fixtureName): int {
+            return (int) $this->fixtures->get($fixtureName)->getId();
+        }, $fixtureNames));
+
+        // Disable c3 for store
+        $c3 = $this->categoryRepository->get($categoryIds['c3'], $storeId);
+        $c3->setIsActive(false);
+        $this->categoryRepository->save($c3);
+
+        // Enable c4 for store
+        $c4 = $this->categoryRepository->get($categoryIds['c4'], $storeId);
+        $c4->setIsActive(true);
+        $this->categoryRepository->save($c4);
+
+        // Check categories
+        $categories = $this->urlResource->getCategories($categoryIds, $storeId);
+        $this->assertSame('1', $categories[$categoryIds['c1']]->getIsActive());
+        $this->assertSame('0', $categories[$categoryIds['c2']]->getIsActive());
+        $this->assertSame('0', $categories[$categoryIds['c3']]->getIsActive());
+        $this->assertSame('1', $categories[$categoryIds['c4']]->getIsActive());
     }
 }