MC-21542: Category query does not handle disabled children properly

- added test function to trace back active status of it's parents back to root category being queried

Author: cpartica

app/code/Magento/CatalogGraphQl/Model/Resolver/Category/Image.php

@@ -41,7 +41,7 @@ public function resolve(
         array $args = null
     ) {
         if (!isset($value['model'])) {
-            return null;
+            throw new LocalizedException(__('"model" value should be specified'));
         }
         /** @var \Magento\Catalog\Model\Category $category */
         $category = $value['model'];

app/code/Magento/CatalogGraphQl/Model/Resolver/Products/DataProvider/ExtractDataFromCategoryTree.php

@@ -48,16 +48,18 @@ public function __construct(
     public function execute(\Iterator $iterator): array
     {
         $tree = [];
+        /** @var CategoryInterface $rootCategory */
+        $rootCategory = $iterator->current();
         while ($iterator->valid()) {
-            /** @var CategoryInterface $category */
-            $category = $iterator->current();
+            /** @var CategoryInterface $currentCategory */
+            $currentCategory = $iterator->current();
             $iterator->next();
-            if ($category->getIsActive()) {
-                $pathElements = explode("/", $category->getPath());
+            if ($this->areParentsActive($currentCategory, $rootCategory, (array)$iterator)) {
+                $pathElements = explode("/", $currentCategory->getPath());
                 if (empty($tree)) {
                     $this->startCategoryFetchLevel = count($pathElements) - 1;
                 }
-                $this->iteratingCategory = $category;
+                $this->iteratingCategory = $currentCategory;
                 $currentLevelTree = $this->explodePathToArray($pathElements, $this->startCategoryFetchLevel);
                 if (empty($tree)) {
                     $tree = $currentLevelTree;
@@ -68,6 +70,34 @@ public function execute(\Iterator $iterator): array
         return $tree;
     }
 
+
+    /**
+     * Test that all parents of the current category are active
+     *
+     * Assumes that $categoriesArray are key-pair values and key is the ID of the category
+     *
+     * @param CategoryInterface $currentCategory
+     * @param CategoryInterface $rootCategory
+     * @param $categoriesArray
+     */
+    private function areParentsActive(
+        CategoryInterface $currentCategory,
+        CategoryInterface $rootCategory,
+        array $categoriesArray
+    ): bool {
+        if ($currentCategory === $rootCategory) {
+            return true;
+        } elseif (array_key_exists($currentCategory->getParentId(), $categoriesArray)) {
+            return $this->areParentsActive(
+                $categoriesArray[$currentCategory->getParentId()],
+                $rootCategory,
+                $categoriesArray
+            );
+        } else {
+            return false;
+        }
+    }
+
     /**
      * Merge together complex categories trees
      *