B2B-2530: Unskip GraphQL cache tests skipped due to DEVOPS-4924

- Fixed static issue and other new review comments

Author: avattam06

dev/tests/api-functional/testsuite/Magento/GraphQl/PageCache/CacheTagTest.php

@@ -13,7 +13,7 @@
 use Magento\TestFramework\ObjectManager;
 
 /**
- * Test the cache invalidation for products and categories
+ * Test the cache works properly for products and categories
  */
 class CacheTagTest extends GraphQLPageCacheAbstract
 {
@@ -107,7 +107,10 @@ public function testCacheInvalidationForCategoriesWithProduct()
 
         // cache-debug header value should be MISS after  updating product1 and reloading the Category
         $responseMissCategoryAfterProductUpdate = $this->graphQlQueryWithResponseHeaders($categoryQuery);
-        $this->assertArrayHasKey(CacheIdCalculator::CACHE_ID_HEADER, $responseMissCategoryAfterProductUpdate['headers']);
+        $this->assertArrayHasKey(
+            CacheIdCalculator::CACHE_ID_HEADER,
+            $responseMissCategoryAfterProductUpdate['headers']
+        );
         $cacheId = $responseMissCategoryAfterProductUpdate['headers'][CacheIdCalculator::CACHE_ID_HEADER];
         // Verify we obtain a cache MISS the first time we search the cache using this X-Magento-Cache-Id
         $this->assertCacheMissAndReturnResponse($categoryQuery, [CacheIdCalculator::CACHE_ID_HEADER => $cacheId]);

dev/tests/api-functional/testsuite/Magento/GraphQl/PageCache/Cms/BlockCacheTest.php

@@ -14,7 +14,7 @@
 use Magento\TestFramework\Helper\Bootstrap;
 
 /**
- * Test the cache invalidation for CMS Blocks
+ * Test the cache works properly for CMS Blocks
  */
 class BlockCacheTest extends GraphQLPageCacheAbstract
 {

dev/tests/api-functional/testsuite/Magento/GraphQl/PageCache/Cms/PageCacheTest.php

@@ -14,7 +14,7 @@
 use Magento\TestFramework\Helper\Bootstrap;
 
 /**
- * Test the cache invalidation for CMS Pages
+ * Test the cache works properly for CMS Pages
  */
 class PageCacheTest extends GraphQLPageCacheAbstract
 {
@@ -130,7 +130,8 @@ public function testCacheIsInvalidatedOnPageUpdate()
 
         $page100ResponseHitAfterUpdate = $this->graphQlQueryWithResponseHeaders($page100Query);
         $this->assertArrayHasKey(CacheIdCalculator::CACHE_ID_HEADER, $page100ResponseHitAfterUpdate['headers']);
-        $cacheIdPage100ResponseHitAfterUpdate = $page100ResponseHitAfterUpdate['headers'][CacheIdCalculator::CACHE_ID_HEADER];
+        $cacheIdPage100ResponseHitAfterUpdate =
+            $page100ResponseHitAfterUpdate['headers'][CacheIdCalculator::CACHE_ID_HEADER];
         $this->assertCacheHitAndReturnResponse(
             $page100Query,
             [CacheIdCalculator::CACHE_ID_HEADER => $cacheIdPage100ResponseHitAfterUpdate]

dev/tests/api-functional/testsuite/Magento/GraphQl/PageCache/UrlRewrite/UrlResolverCacheTest.php

@@ -14,7 +14,7 @@
 use Magento\TestFramework\Helper\Bootstrap;
 
 /**
- * Test cache works for url resolver.
+ * Test cache works properly for url resolver.
  */
 class UrlResolverCacheTest extends GraphQLPageCacheAbstract
 {