From 0ca59a1f974ac9c40ab7224270b1ff5f78229374 Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Sat, 14 Sep 2024 10:09:04 +0530
Subject: [PATCH 01/13] feat: Add CoreHeadingTest

---
 tests/unit/CoreHeadingTest.php | 115 +++++++++++++++++++++++++++++++++
 1 file changed, 115 insertions(+)
 create mode 100644 tests/unit/CoreHeadingTest.php

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
new file mode 100644
index 00000000..62cbfc4d
--- /dev/null
+++ b/tests/unit/CoreHeadingTest.php
@@ -0,0 +1,115 @@
+<?php
+
+namespace WPGraphQL\ContentBlocks\Unit;
+
+final class CoreHeadingTest extends PluginTestCase
+{
+    public $instance;
+    public $post_id;
+
+    public function setUp(): void
+    {
+        parent::setUp();
+        global $wpdb;
+
+        $this->post_id = wp_insert_post(
+            array(
+                'post_title' => 'Post with Heading',
+                'post_content' => preg_replace(
+                    '/\s+/',
+                    ' ',
+                    trim(
+                        '
+                        <!-- wp:heading {"level":2,"textAlign":"center","style":{"typography":{"fontSize":"28px","fontStyle":"normal","fontWeight":"700"}}} -->
+                        <h2 class="wp-block-heading has-text-align-center" style="font-size:28px;font-style:normal;font-weight:700">Sample Heading</h2>
+                        <!-- /wp:heading -->
+                        '
+                    )
+                ),
+                'post_status' => 'publish',
+            )
+        );
+    }
+
+    public function tearDown(): void
+    {
+        parent::tearDown();
+        wp_delete_post($this->post_id, true);
+    }
+
+    public function test_retrieve_core_heading_attributes()
+    {
+        $query = '
+          fragment CoreHeadingBlockFragment on CoreHeading {
+            attributes {
+              content
+              level
+              textAlign
+              style
+            }
+          }
+
+          query GetPosts {
+            posts(first: 1) {
+              nodes {
+                databaseId
+                editorBlocks {
+                  name
+                  ...CoreHeadingBlockFragment
+                }
+              }
+            }
+          }
+        ';
+
+        $actual = graphql(array('query' => $query));
+        $node = $actual['data']['posts']['nodes'][0];
+
+        // Verify that the ID of the first post matches the one we just created.
+        $this->assertEquals($this->post_id, $node['databaseId']);
+
+        // There should be only one block using that query when not using flat: true
+        $this->assertEquals(count($node['editorBlocks']), 1);
+        $this->assertEquals($node['editorBlocks'][0]['name'], 'core/heading');
+
+        $this->assertEquals(
+            $node['editorBlocks'][0]['attributes'],
+            [
+                'content' => 'Sample Heading',
+                'level' => 2,
+                'textAlign' => 'center',
+                'style' => [
+                    'typography' => [
+                        'fontSize' => '28px',
+                        'fontStyle' => 'normal',
+                        'fontWeight' => '700',
+                    ],
+                ],
+            ]
+        );
+    }
+
+    public function test_retrieve_core_heading_content()
+    {
+        $query = '
+          fragment CoreHeadingBlockFragment on CoreHeading {
+            content
+          }
+
+          query GetPosts {
+            posts(first: 1) {
+              nodes {
+                editorBlocks {
+                  ...CoreHeadingBlockFragment
+                }
+              }
+            }
+          }
+        ';
+
+        $actual = graphql(array('query' => $query));
+        $node = $actual['data']['posts']['nodes'][0];
+
+        $this->assertEquals($node['editorBlocks'][0]['content'], 'Sample Heading');
+    }
+}

From 039a0d102bb0ac3204c512da70c2ecf61a5c43ed Mon Sep 17 00:00:00 2001
From: David Levine <david@axepress.dev>
Date: Sun, 15 Sep 2024 19:26:17 +0000
Subject: [PATCH 02/13] chore: refactor CoreHeadingTest

---
 tests/unit/CoreHeadingTest.php | 259 +++++++++++++++++++--------------
 1 file changed, 149 insertions(+), 110 deletions(-)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index 62cbfc4d..f992b241 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -2,114 +2,153 @@
 
 namespace WPGraphQL\ContentBlocks\Unit;
 
-final class CoreHeadingTest extends PluginTestCase
-{
-    public $instance;
-    public $post_id;
-
-    public function setUp(): void
-    {
-        parent::setUp();
-        global $wpdb;
-
-        $this->post_id = wp_insert_post(
-            array(
-                'post_title' => 'Post with Heading',
-                'post_content' => preg_replace(
-                    '/\s+/',
-                    ' ',
-                    trim(
-                        '
-                        <!-- wp:heading {"level":2,"textAlign":"center","style":{"typography":{"fontSize":"28px","fontStyle":"normal","fontWeight":"700"}}} -->
-                        <h2 class="wp-block-heading has-text-align-center" style="font-size:28px;font-style:normal;font-weight:700">Sample Heading</h2>
-                        <!-- /wp:heading -->
-                        '
-                    )
-                ),
-                'post_status' => 'publish',
-            )
-        );
-    }
-
-    public function tearDown(): void
-    {
-        parent::tearDown();
-        wp_delete_post($this->post_id, true);
-    }
-
-    public function test_retrieve_core_heading_attributes()
-    {
-        $query = '
-          fragment CoreHeadingBlockFragment on CoreHeading {
-            attributes {
-              content
-              level
-              textAlign
-              style
-            }
-          }
-
-          query GetPosts {
-            posts(first: 1) {
-              nodes {
-                databaseId
-                editorBlocks {
-                  name
-                  ...CoreHeadingBlockFragment
-                }
-              }
-            }
-          }
-        ';
-
-        $actual = graphql(array('query' => $query));
-        $node = $actual['data']['posts']['nodes'][0];
-
-        // Verify that the ID of the first post matches the one we just created.
-        $this->assertEquals($this->post_id, $node['databaseId']);
-
-        // There should be only one block using that query when not using flat: true
-        $this->assertEquals(count($node['editorBlocks']), 1);
-        $this->assertEquals($node['editorBlocks'][0]['name'], 'core/heading');
-
-        $this->assertEquals(
-            $node['editorBlocks'][0]['attributes'],
-            [
-                'content' => 'Sample Heading',
-                'level' => 2,
-                'textAlign' => 'center',
-                'style' => [
-                    'typography' => [
-                        'fontSize' => '28px',
-                        'fontStyle' => 'normal',
-                        'fontWeight' => '700',
-                    ],
-                ],
-            ]
-        );
-    }
-
-    public function test_retrieve_core_heading_content()
-    {
-        $query = '
-          fragment CoreHeadingBlockFragment on CoreHeading {
-            content
-          }
-
-          query GetPosts {
-            posts(first: 1) {
-              nodes {
-                editorBlocks {
-                  ...CoreHeadingBlockFragment
-                }
-              }
-            }
-          }
-        ';
-
-        $actual = graphql(array('query' => $query));
-        $node = $actual['data']['posts']['nodes'][0];
-
-        $this->assertEquals($node['editorBlocks'][0]['content'], 'Sample Heading');
-    }
+final class CoreHeadingTest extends PluginTestCase {
+	public $post_id;
+
+	public function setUp(): void {
+		parent::setUp();
+
+		$this->post_id = wp_insert_post(
+			[
+				'post_title'   => 'Post with Heading',
+				'post_content' => '',
+				'post_status'  => 'publish',
+			]
+		);
+	}
+
+	public function tearDown(): void {
+		parent::tearDown();
+
+		wp_delete_post( $this->post_id, true );
+	}
+
+	public function query(): string {
+		return '
+			fragment CoreHeadingBlockFragment on CoreHeading {
+				attributes {
+					align
+					anchor
+					backgroundColor
+					className
+					content
+					cssClassName
+					fontFamily
+					fontSize
+					gradient
+					level
+					lock
+					# metadata
+					placeholder
+					style
+					textAlign
+					textColor
+				}
+			}
+
+			query Post( $id: ID! ) {
+				post(id: $id, idType: DATABASE_ID) {
+					databaseId
+					editorBlocks {
+						apiVersion
+						blockEditorCategoryName
+						clientId
+						cssClassNames
+						innerBlocks {
+							name
+						}
+						isDynamic
+						name
+						parentClientId
+						renderedHtml
+						... on BlockWithSupportsAnchor {
+							anchor
+						}
+						...CoreHeadingBlockFragment
+					}
+				}
+			}
+		';
+	}
+
+	public function test_retrieve_core_heading_attributes() {
+		$block_content = '
+			<!-- wp:heading {"level":2,"textAlign":"center","style":{"typography":{"fontSize":"28px","fontStyle":"normal","fontWeight":"700"}}} -->
+			<h2 class="wp-block-heading has-text-align-center" style="font-size:28px;font-style:normal;font-weight:700">Sample Heading</h2>
+			<!-- /wp:heading -->
+		';
+
+		// Update the post content with the block content.
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$query     = $this->query();
+		$variables = [
+			'id' => $this->post_id,
+		];
+
+		// Test the query.
+
+		$actual = graphql( compact( 'query', 'variables' ) );
+
+		error_log( print_r( $actual, true ) );
+
+		$this->assertArrayNotHasKey( 'errors', $actual, 'There should not be any errors' );
+		$this->assertArrayHasKey( 'data', $actual, 'The data key should be present' );
+		$this->assertArrayHasKey( 'post', $actual['data'], 'The post key should be present' );
+
+		// Verify that the ID of the first post matches the one we just created.
+		$this->assertEquals( $this->post_id, $actual['data']['post']['databaseId'], 'The post ID should match' );
+
+		// There should be only one block using that query when not using flat: true
+		$this->assertEquals( 1, count( $actual['data']['post']['editorBlocks'] ) );
+
+		// Verify the block data.
+		error_log( print_r( $actual['data']['post']['editorBlocks'][0], true ) );
+		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['apiVersion'], 'The apiVersion should be present' );
+		$this->assertEquals( 'text', $actual['data']['post']['editorBlocks'][0]['blockEditorCategoryName'],  'The blockEditorCategoryName should be text' );
+		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['clientId'], 'The clientId should be present' );
+
+		// @todo this is not working
+		// $this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['cssClassNames'], 'The cssClassNames should be present' );
+
+		$this->assertEmpty( $actual['data']['post']['editorBlocks'][0]['innerBlocks'], 'There should be no inner blocks' );
+		$this->assertEquals( 'core/heading', $actual['data']['post']['editorBlocks'][0]['name'], 'The block name should be core/heading' );
+		$this->assertEmpty( $actual['data']['post']['editorBlocks'][0]['parentClientId'], 'There should be no parentClientId' );
+		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['renderedHtml'], 'The renderedHtml should be present' );
+
+		// Verify the attributes.
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'Sample Heading',
+				'cssClassName'    => 'wp-block-heading has-text-align-center',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 2.0, // @todo this should be an integer
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => wp_json_encode(
+					[
+						'typography' => [
+							'fontSize'   => '28px',
+							'fontStyle'  => 'normal',
+							'fontWeight' => '700',
+						],
+					],
+				),
+				'textAlign'       => 'center',
+				'textColor'       => null,
+			],
+			$actual['data']['post']['editorBlocks'][0]['attributes'],
+		);
+	}
 }

From 92e7ae61441a4043f30de07e3e2fce5b9b01fb4e Mon Sep 17 00:00:00 2001
From: David Levine <david@axepress.dev>
Date: Sun, 15 Sep 2024 19:41:28 +0000
Subject: [PATCH 03/13] chore: cleanup

---
 tests/unit/CoreHeadingTest.php | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index f992b241..3ca5e058 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -15,12 +15,16 @@ public function setUp(): void {
 				'post_status'  => 'publish',
 			]
 		);
+
+		\WPGraphQL::clear_schema();
 	}
 
 	public function tearDown(): void {
 		parent::tearDown();
 
 		wp_delete_post( $this->post_id, true );
+
+		\WPGraphQL::clear_schema();
 	}
 
 	public function query(): string {
@@ -95,8 +99,6 @@ public function test_retrieve_core_heading_attributes() {
 
 		$actual = graphql( compact( 'query', 'variables' ) );
 
-		error_log( print_r( $actual, true ) );
-
 		$this->assertArrayNotHasKey( 'errors', $actual, 'There should not be any errors' );
 		$this->assertArrayHasKey( 'data', $actual, 'The data key should be present' );
 		$this->assertArrayHasKey( 'post', $actual['data'], 'The post key should be present' );
@@ -108,7 +110,6 @@ public function test_retrieve_core_heading_attributes() {
 		$this->assertEquals( 1, count( $actual['data']['post']['editorBlocks'] ) );
 
 		// Verify the block data.
-		error_log( print_r( $actual['data']['post']['editorBlocks'][0], true ) );
 		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['apiVersion'], 'The apiVersion should be present' );
 		$this->assertEquals( 'text', $actual['data']['post']['editorBlocks'][0]['blockEditorCategoryName'],  'The blockEditorCategoryName should be text' );
 		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['clientId'], 'The clientId should be present' );

From cdec9ee302ae34473c3f7eae5f5636b867732dfa Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Wed, 18 Sep 2024 20:04:25 +0530
Subject: [PATCH 04/13] feat: Add tests for additional snippets

- cover the attributes that are null
---
 tests/unit/CoreHeadingTest.php | 223 ++++++++++++++++++++++++++++++---
 1 file changed, 206 insertions(+), 17 deletions(-)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index 3ca5e058..d8f03259 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -49,7 +49,6 @@ className
 					textColor
 				}
 			}
-
 			query Post( $id: ID! ) {
 				post(id: $id, idType: DATABASE_ID) {
 					databaseId
@@ -82,7 +81,6 @@ public function test_retrieve_core_heading_attributes() {
 			<!-- /wp:heading -->
 		';
 
-		// Update the post content with the block content.
 		wp_update_post(
 			[
 				'ID'           => $this->post_id,
@@ -95,34 +93,31 @@ public function test_retrieve_core_heading_attributes() {
 			'id' => $this->post_id,
 		];
 
-		// Test the query.
-
 		$actual = graphql( compact( 'query', 'variables' ) );
 
 		$this->assertArrayNotHasKey( 'errors', $actual, 'There should not be any errors' );
 		$this->assertArrayHasKey( 'data', $actual, 'The data key should be present' );
 		$this->assertArrayHasKey( 'post', $actual['data'], 'The post key should be present' );
 
-		// Verify that the ID of the first post matches the one we just created.
 		$this->assertEquals( $this->post_id, $actual['data']['post']['databaseId'], 'The post ID should match' );
 
-		// There should be only one block using that query when not using flat: true
 		$this->assertEquals( 1, count( $actual['data']['post']['editorBlocks'] ) );
 
-		// Verify the block data.
-		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['apiVersion'], 'The apiVersion should be present' );
-		$this->assertEquals( 'text', $actual['data']['post']['editorBlocks'][0]['blockEditorCategoryName'],  'The blockEditorCategoryName should be text' );
-		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['clientId'], 'The clientId should be present' );
+		$block = $actual['data']['post']['editorBlocks'][0];
+
+		$this->assertNotEmpty( $block['apiVersion'], 'The apiVersion should be present' );
+		$this->assertEquals( 'text', $block['blockEditorCategoryName'],  'The blockEditorCategoryName should be text' );
+		$this->assertNotEmpty( $block['clientId'], 'The clientId should be present' );
 
 		// @todo this is not working
-		// $this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['cssClassNames'], 'The cssClassNames should be present' );
+		// $this->assertNotEmpty( $block['cssClassNames'], 'The cssClassNames should be present' );
 
-		$this->assertEmpty( $actual['data']['post']['editorBlocks'][0]['innerBlocks'], 'There should be no inner blocks' );
-		$this->assertEquals( 'core/heading', $actual['data']['post']['editorBlocks'][0]['name'], 'The block name should be core/heading' );
-		$this->assertEmpty( $actual['data']['post']['editorBlocks'][0]['parentClientId'], 'There should be no parentClientId' );
-		$this->assertNotEmpty( $actual['data']['post']['editorBlocks'][0]['renderedHtml'], 'The renderedHtml should be present' );
+		$this->assertEmpty( $block['innerBlocks'], 'There should be no inner blocks' );
+		$this->assertEquals( 'core/heading', $block['name'], 'The block name should be core/heading' );
+		$this->assertEmpty( $block['parentClientId'], 'There should be no parentClientId' );
+		$this->assertNotEmpty( $block['renderedHtml'], 'The renderedHtml should be present' );
 
-		// Verify the attributes.
+		$attributes = $block['attributes'];
 		$this->assertEquals(
 			[
 				'align'           => null,
@@ -149,7 +144,201 @@ public function test_retrieve_core_heading_attributes() {
 				'textAlign'       => 'center',
 				'textColor'       => null,
 			],
-			$actual['data']['post']['editorBlocks'][0]['attributes'],
+			$attributes,
+		);
+	}
+
+	public function test_retrieve_core_heading_with_colors_and_alignment() {
+		$block_content = '
+			<!-- wp:heading {"level":3,"textAlign":"right","align":"wide","style":{"color":{"background":"#cf2e2e","text":"#ffffff"}}} -->
+			<h3 class="wp-block-heading has-text-align-right has-text-color has-background alignwide" style="background-color:#cf2e2e;color:#ffffff">Colored Heading</h3>
+			<!-- /wp:heading -->
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql([
+			'query' => $this->query(),
+			'variables' => ['id' => $this->post_id],
+		]);
+
+		$block = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals( 'Colored Heading', $attributes['content'] );
+		$this->assertEquals( 3.0, $attributes['level'] );
+		$this->assertEquals( 'right', $attributes['textAlign'] );
+		$this->assertEquals( 'wide', $attributes['align'] );
+		
+		$style = json_decode($attributes['style'], true);
+		$this->assertEquals('#cf2e2e', $style['color']['background']);
+		$this->assertEquals('#ffffff', $style['color']['text']);
+	}
+
+	public function test_retrieve_core_heading_with_font_and_anchor() {
+		$block_content = '
+			<!-- wp:heading {"anchor":"custom-id","style":{"typography":{"fontFamily":"Arial","fontSize":"32px"}}} -->
+			<h2 class="wp-block-heading" id="custom-id" style="font-family:Arial;font-size:32px">Custom Font Heading</h2>
+			<!-- /wp:heading -->
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql([
+			'query' => $this->query(),
+			'variables' => ['id' => $this->post_id],
+		]);
+
+		$block = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals( 'Custom Font Heading', $attributes['content'] );
+		$this->assertEquals( 'custom-id', $attributes['anchor'] );
+		
+		$style = json_decode($attributes['style'], true);
+		$this->assertEquals('Arial', $style['typography']['fontFamily']);
+		$this->assertEquals('32px', $style['typography']['fontSize']);
+	}
+
+	public function test_retrieve_core_heading_with_gradient() {
+		$block_content = '
+			<!-- wp:heading {"style":{"color":{"gradient":"linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)"}}} -->
+			<h2 class="wp-block-heading has-background" style="background:linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)">Gradient Heading</h2>
+			<!-- /wp:heading -->
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql([
+			'query' => $this->query(),
+			'variables' => ['id' => $this->post_id],
+		]);
+
+		$block = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals( 'Gradient Heading', $attributes['content'] );
+		
+		$style = json_decode($attributes['style'], true);
+		$this->assertEquals('linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)', $style['color']['gradient']);
+	}
+
+	public function test_retrieve_core_heading_with_background_color() {
+		$block_content = '
+			<!-- wp:heading {"backgroundColor":"vivid-red-background-color"} -->
+			<h2 class="wp-block-heading has-vivid-red-background-color has-background">Heading with Background Color</h2>
+			<!-- /wp:heading>
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql([
+			'query' => $this->query(),
+			'variables' => ['id' => $this->post_id],
+		]);
+
+		$block = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals('Heading with Background Color', $attributes['content']);
+		$this->assertEquals('vivid-red-background-color', $attributes['backgroundColor']);
+	}
+
+	public function test_retrieve_core_heading_with_text_color() {
+		$block_content = '
+			<!-- wp:heading {"textColor":"vivid-red"} -->
+			<h2 class="wp-block-heading has-vivid-red-color has-text-color">Heading with Text Color</h2>
+			<!-- /wp:heading>
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql([
+			'query' => $this->query(),
+			'variables' => ['id' => $this->post_id],
+		]);
+
+		$block = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals('Heading with Text Color', $attributes['content']);
+		$this->assertEquals('vivid-red', $attributes['textColor']);
+	}
+
+	public function test_retrieve_core_heading_with_font_size() {
+		$block_content = '
+			<!-- wp:heading {"fontSize":"large"} -->
+			<h2 class="wp-block-heading has-large-font-size">Heading with Font Size</h2>
+			<!-- /wp:heading>
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql([
+			'query' => $this->query(),
+			'variables' => ['id' => $this->post_id],
+		]);
+
+		$block = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals('Heading with Font Size', $attributes['content']);
+		$this->assertEquals('large', $attributes['fontSize']);
+	}
+
+	public function test_retrieve_core_heading_with_class_name() {
+		$block_content = '
+			<!-- wp:heading {"className":"custom-class"} -->
+			<h2 class="wp-block-heading custom-class">Heading with Custom Class</h2>
+			<!-- /wp:heading>
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
 		);
+
+		$actual = graphql([
+			'query' => $this->query(),
+			'variables' => ['id' => $this->post_id],
+		]);
+
+		$block = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals('Heading with Custom Class', $attributes['content']);
+		$this->assertEquals('custom-class', $attributes['className']);
 	}
 }

From eaf44ca755a790aaea2c53261a9c048252ccef41 Mon Sep 17 00:00:00 2001
From: David Levine <david@axepress.dev>
Date: Wed, 18 Sep 2024 15:05:44 +0000
Subject: [PATCH 05/13] chore: phpcbf

---
 tests/unit/CoreHeadingTest.php | 124 ++++++++++++++++++---------------
 1 file changed, 69 insertions(+), 55 deletions(-)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index d8f03259..a5c79f06 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -106,7 +106,7 @@ public function test_retrieve_core_heading_attributes() {
 		$block = $actual['data']['post']['editorBlocks'][0];
 
 		$this->assertNotEmpty( $block['apiVersion'], 'The apiVersion should be present' );
-		$this->assertEquals( 'text', $block['blockEditorCategoryName'],  'The blockEditorCategoryName should be text' );
+		$this->assertEquals( 'text', $block['blockEditorCategoryName'], 'The blockEditorCategoryName should be text' );
 		$this->assertNotEmpty( $block['clientId'], 'The clientId should be present' );
 
 		// @todo this is not working
@@ -162,22 +162,24 @@ public function test_retrieve_core_heading_with_colors_and_alignment() {
 			]
 		);
 
-		$actual = graphql([
-			'query' => $this->query(),
-			'variables' => ['id' => $this->post_id],
-		]);
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
 
-		$block = $actual['data']['post']['editorBlocks'][0];
+		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
 		$this->assertEquals( 'Colored Heading', $attributes['content'] );
 		$this->assertEquals( 3.0, $attributes['level'] );
 		$this->assertEquals( 'right', $attributes['textAlign'] );
 		$this->assertEquals( 'wide', $attributes['align'] );
-		
-		$style = json_decode($attributes['style'], true);
-		$this->assertEquals('#cf2e2e', $style['color']['background']);
-		$this->assertEquals('#ffffff', $style['color']['text']);
+
+		$style = json_decode( $attributes['style'], true );
+		$this->assertEquals( '#cf2e2e', $style['color']['background'] );
+		$this->assertEquals( '#ffffff', $style['color']['text'] );
 	}
 
 	public function test_retrieve_core_heading_with_font_and_anchor() {
@@ -194,20 +196,22 @@ public function test_retrieve_core_heading_with_font_and_anchor() {
 			]
 		);
 
-		$actual = graphql([
-			'query' => $this->query(),
-			'variables' => ['id' => $this->post_id],
-		]);
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
 
-		$block = $actual['data']['post']['editorBlocks'][0];
+		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
 		$this->assertEquals( 'Custom Font Heading', $attributes['content'] );
 		$this->assertEquals( 'custom-id', $attributes['anchor'] );
-		
-		$style = json_decode($attributes['style'], true);
-		$this->assertEquals('Arial', $style['typography']['fontFamily']);
-		$this->assertEquals('32px', $style['typography']['fontSize']);
+
+		$style = json_decode( $attributes['style'], true );
+		$this->assertEquals( 'Arial', $style['typography']['fontFamily'] );
+		$this->assertEquals( '32px', $style['typography']['fontSize'] );
 	}
 
 	public function test_retrieve_core_heading_with_gradient() {
@@ -224,18 +228,20 @@ public function test_retrieve_core_heading_with_gradient() {
 			]
 		);
 
-		$actual = graphql([
-			'query' => $this->query(),
-			'variables' => ['id' => $this->post_id],
-		]);
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
 
-		$block = $actual['data']['post']['editorBlocks'][0];
+		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
 		$this->assertEquals( 'Gradient Heading', $attributes['content'] );
-		
-		$style = json_decode($attributes['style'], true);
-		$this->assertEquals('linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)', $style['color']['gradient']);
+
+		$style = json_decode( $attributes['style'], true );
+		$this->assertEquals( 'linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)', $style['color']['gradient'] );
 	}
 
 	public function test_retrieve_core_heading_with_background_color() {
@@ -252,16 +258,18 @@ public function test_retrieve_core_heading_with_background_color() {
 			]
 		);
 
-		$actual = graphql([
-			'query' => $this->query(),
-			'variables' => ['id' => $this->post_id],
-		]);
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
 
-		$block = $actual['data']['post']['editorBlocks'][0];
+		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals('Heading with Background Color', $attributes['content']);
-		$this->assertEquals('vivid-red-background-color', $attributes['backgroundColor']);
+		$this->assertEquals( 'Heading with Background Color', $attributes['content'] );
+		$this->assertEquals( 'vivid-red-background-color', $attributes['backgroundColor'] );
 	}
 
 	public function test_retrieve_core_heading_with_text_color() {
@@ -278,16 +286,18 @@ public function test_retrieve_core_heading_with_text_color() {
 			]
 		);
 
-		$actual = graphql([
-			'query' => $this->query(),
-			'variables' => ['id' => $this->post_id],
-		]);
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
 
-		$block = $actual['data']['post']['editorBlocks'][0];
+		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals('Heading with Text Color', $attributes['content']);
-		$this->assertEquals('vivid-red', $attributes['textColor']);
+		$this->assertEquals( 'Heading with Text Color', $attributes['content'] );
+		$this->assertEquals( 'vivid-red', $attributes['textColor'] );
 	}
 
 	public function test_retrieve_core_heading_with_font_size() {
@@ -304,16 +314,18 @@ public function test_retrieve_core_heading_with_font_size() {
 			]
 		);
 
-		$actual = graphql([
-			'query' => $this->query(),
-			'variables' => ['id' => $this->post_id],
-		]);
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
 
-		$block = $actual['data']['post']['editorBlocks'][0];
+		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals('Heading with Font Size', $attributes['content']);
-		$this->assertEquals('large', $attributes['fontSize']);
+		$this->assertEquals( 'Heading with Font Size', $attributes['content'] );
+		$this->assertEquals( 'large', $attributes['fontSize'] );
 	}
 
 	public function test_retrieve_core_heading_with_class_name() {
@@ -330,15 +342,17 @@ public function test_retrieve_core_heading_with_class_name() {
 			]
 		);
 
-		$actual = graphql([
-			'query' => $this->query(),
-			'variables' => ['id' => $this->post_id],
-		]);
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
 
-		$block = $actual['data']['post']['editorBlocks'][0];
+		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals('Heading with Custom Class', $attributes['content']);
-		$this->assertEquals('custom-class', $attributes['className']);
+		$this->assertEquals( 'Heading with Custom Class', $attributes['content'] );
+		$this->assertEquals( 'custom-class', $attributes['className'] );
 	}
 }

From 9be31c6d6509a55a7220c523398df8af5e1b4fab Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Wed, 18 Sep 2024 23:07:36 +0530
Subject: [PATCH 06/13] fix: Update new tests so assertEquals()

---
 tests/unit/CoreHeadingTest.php | 184 ++++++++++++++++++++++++++++-----
 1 file changed, 156 insertions(+), 28 deletions(-)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index a5c79f06..c7975c9f 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -139,12 +139,12 @@ public function test_retrieve_core_heading_attributes() {
 							'fontStyle'  => 'normal',
 							'fontWeight' => '700',
 						],
-					],
+					]
 				),
 				'textAlign'       => 'center',
 				'textColor'       => null,
 			],
-			$attributes,
+			$attributes
 		);
 	}
 
@@ -172,14 +172,31 @@ public function test_retrieve_core_heading_with_colors_and_alignment() {
 		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals( 'Colored Heading', $attributes['content'] );
-		$this->assertEquals( 3.0, $attributes['level'] );
-		$this->assertEquals( 'right', $attributes['textAlign'] );
-		$this->assertEquals( 'wide', $attributes['align'] );
-
-		$style = json_decode( $attributes['style'], true );
-		$this->assertEquals( '#cf2e2e', $style['color']['background'] );
-		$this->assertEquals( '#ffffff', $style['color']['text'] );
+		$this->assertEquals(
+			[
+				'align'           => 'wide', // Previously untested
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'Colored Heading',
+				'cssClassName'    => 'wp-block-heading has-text-align-right has-text-color has-background alignwide',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 3.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => wp_json_encode([
+					'color' => [
+						'background' => '#cf2e2e',
+						'text'       => '#ffffff',
+					],
+				]),
+				'textAlign'       => 'right',
+				'textColor'       => null,
+			],
+			$attributes
+		);
 	}
 
 	public function test_retrieve_core_heading_with_font_and_anchor() {
@@ -206,12 +223,31 @@ public function test_retrieve_core_heading_with_font_and_anchor() {
 		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals( 'Custom Font Heading', $attributes['content'] );
-		$this->assertEquals( 'custom-id', $attributes['anchor'] );
-
-		$style = json_decode( $attributes['style'], true );
-		$this->assertEquals( 'Arial', $style['typography']['fontFamily'] );
-		$this->assertEquals( '32px', $style['typography']['fontSize'] );
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => 'custom-id',
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'Custom Font Heading',
+				'cssClassName'    => 'wp-block-heading',
+				'fontFamily'      => null, // Previously untested
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => wp_json_encode([
+					'typography' => [
+						'fontFamily' => 'Arial',
+						'fontSize'   => '32px',
+					],
+				]),
+				'textAlign'       => null,
+				'textColor'       => null,
+			],
+			$attributes
+		);
 	}
 
 	public function test_retrieve_core_heading_with_gradient() {
@@ -238,10 +274,30 @@ public function test_retrieve_core_heading_with_gradient() {
 		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals( 'Gradient Heading', $attributes['content'] );
-
-		$style = json_decode( $attributes['style'], true );
-		$this->assertEquals( 'linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)', $style['color']['gradient'] );
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'Gradient Heading',
+				'cssClassName'    => 'wp-block-heading has-background',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null, // Previously untested
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => wp_json_encode([
+					'color' => [
+						'gradient' => 'linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)',
+					],
+				]),
+				'textAlign'       => null,
+				'textColor'       => null,
+			],
+			$attributes
+		);
 	}
 
 	public function test_retrieve_core_heading_with_background_color() {
@@ -268,8 +324,26 @@ public function test_retrieve_core_heading_with_background_color() {
 		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals( 'Heading with Background Color', $attributes['content'] );
-		$this->assertEquals( 'vivid-red-background-color', $attributes['backgroundColor'] );
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => 'vivid-red-background-color',
+				'className'       => null,
+				'content'         => 'Heading with Background Color',
+				'cssClassName'    => 'wp-block-heading has-vivid-red-background-color has-background',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => null,
+				'textColor'       => null,
+			],
+			$attributes
+		);
 	}
 
 	public function test_retrieve_core_heading_with_text_color() {
@@ -296,8 +370,26 @@ public function test_retrieve_core_heading_with_text_color() {
 		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals( 'Heading with Text Color', $attributes['content'] );
-		$this->assertEquals( 'vivid-red', $attributes['textColor'] );
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'Heading with Text Color',
+				'cssClassName'    => 'wp-block-heading has-vivid-red-color has-text-color',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => null,
+				'textColor'       => 'vivid-red',
+			],
+			$attributes
+		);
 	}
 
 	public function test_retrieve_core_heading_with_font_size() {
@@ -324,8 +416,26 @@ public function test_retrieve_core_heading_with_font_size() {
 		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals( 'Heading with Font Size', $attributes['content'] );
-		$this->assertEquals( 'large', $attributes['fontSize'] );
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'Heading with Font Size',
+				'cssClassName'    => 'wp-block-heading has-large-font-size',
+				'fontFamily'      => null,
+				'fontSize'        => 'large',
+				'gradient'        => null,
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => null,
+				'textColor'       => null,
+			],
+			$attributes
+		);
 	}
 
 	public function test_retrieve_core_heading_with_class_name() {
@@ -352,7 +462,25 @@ public function test_retrieve_core_heading_with_class_name() {
 		$block      = $actual['data']['post']['editorBlocks'][0];
 		$attributes = $block['attributes'];
 
-		$this->assertEquals( 'Heading with Custom Class', $attributes['content'] );
-		$this->assertEquals( 'custom-class', $attributes['className'] );
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => 'custom-class',
+				'content'         => 'Heading with Custom Class',
+				'cssClassName'    => 'wp-block-heading custom-class',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => null,
+				'textColor'       => null,
+			],
+			$attributes
+		);
 	}
 }

From 7301ed9321f8801204f38a1be815ecc9dacb64eb Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Wed, 18 Sep 2024 23:15:49 +0530
Subject: [PATCH 07/13] feat: Add a changeset for patch

---
 .changeset/four-shoes-fix.md | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 .changeset/four-shoes-fix.md

diff --git a/.changeset/four-shoes-fix.md b/.changeset/four-shoes-fix.md
new file mode 100644
index 00000000..d36e85e5
--- /dev/null
+++ b/.changeset/four-shoes-fix.md
@@ -0,0 +1,5 @@
+---
+"@wpengine/wp-graphql-content-blocks": patch
+---
+
+tests: Add tests for CoreHeading block

From 530233b05c3c0873d262ccee76cae650190610d7 Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Thu, 19 Sep 2024 20:02:26 +0530
Subject: [PATCH 08/13] fix: Fix the remaining annotations and phpcbf

---
 tests/unit/CoreHeadingTest.php | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index c7975c9f..9f0bf4a3 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -176,7 +176,7 @@ public function test_retrieve_core_heading_with_colors_and_alignment() {
 			[
 				'align'           => 'wide', // Previously untested
 				'anchor'          => null,
-				'backgroundColor' => null,
+				'backgroundColor' => null, // @todo: This is not returning correctly.
 				'className'       => null,
 				'content'         => 'Colored Heading',
 				'cssClassName'    => 'wp-block-heading has-text-align-right has-text-color has-background alignwide',
@@ -193,7 +193,7 @@ public function test_retrieve_core_heading_with_colors_and_alignment() {
 					],
 				]),
 				'textAlign'       => 'right',
-				'textColor'       => null,
+				'textColor'       => null, // @todo: This is not returning correctly.
 			],
 			$attributes
 		);
@@ -226,21 +226,21 @@ public function test_retrieve_core_heading_with_font_and_anchor() {
 		$this->assertEquals(
 			[
 				'align'           => null,
-				'anchor'          => 'custom-id',
+				'anchor'          => 'custom-id', // Previously untested
 				'backgroundColor' => null,
 				'className'       => null,
 				'content'         => 'Custom Font Heading',
 				'cssClassName'    => 'wp-block-heading',
-				'fontFamily'      => null, // Previously untested
-				'fontSize'        => null,
+				'fontFamily' => null, // @todo: This is not returning correctly.
+				'fontSize' => null, // @todo: This is not returning correctly.
 				'gradient'        => null,
 				'level'           => 2.0,
 				'lock'            => null,
 				'placeholder'     => null,
 				'style'           => wp_json_encode([
 					'typography' => [
-						'fontFamily' => 'Arial',
-						'fontSize'   => '32px',
+						'fontFamily' => 'Arial', // Previously untested
+						'fontSize'   => '32px', // Previously untested
 					],
 				]),
 				'textAlign'       => null,
@@ -284,7 +284,7 @@ public function test_retrieve_core_heading_with_gradient() {
 				'cssClassName'    => 'wp-block-heading has-background',
 				'fontFamily'      => null,
 				'fontSize'        => null,
-				'gradient'        => null, // Previously untested
+				'gradient' => null, // @todo: This is not returning correctly.
 				'level'           => 2.0,
 				'lock'            => null,
 				'placeholder'     => null,
@@ -386,7 +386,7 @@ public function test_retrieve_core_heading_with_text_color() {
 				'placeholder'     => null,
 				'style'           => null,
 				'textAlign'       => null,
-				'textColor'       => 'vivid-red',
+				'textColor'       => 'vivid-red', // Previously untested
 			],
 			$attributes
 		);
@@ -425,7 +425,7 @@ public function test_retrieve_core_heading_with_font_size() {
 				'content'         => 'Heading with Font Size',
 				'cssClassName'    => 'wp-block-heading has-large-font-size',
 				'fontFamily'      => null,
-				'fontSize'        => 'large',
+				'fontSize'        => 'large', // Previously untested
 				'gradient'        => null,
 				'level'           => 2.0,
 				'lock'            => null,
@@ -467,7 +467,7 @@ public function test_retrieve_core_heading_with_class_name() {
 				'align'           => null,
 				'anchor'          => null,
 				'backgroundColor' => null,
-				'className'       => 'custom-class',
+				'className'       => 'custom-class', // Previously untested
 				'content'         => 'Heading with Custom Class',
 				'cssClassName'    => 'wp-block-heading custom-class',
 				'fontFamily'      => null,

From a474a469c269d28740e0aec1b11e83ecee00cacc Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Tue, 24 Sep 2024 01:19:54 +0530
Subject: [PATCH 09/13] fix: Add test coverage for preset color and font

---
 phpcs.xml                      | 148 +++++++++++++++++++++++++
 tests/unit/CoreHeadingTest.php | 193 ++++++++++++++++++++++++++++-----
 2 files changed, 315 insertions(+), 26 deletions(-)
 create mode 100644 phpcs.xml

diff --git a/phpcs.xml b/phpcs.xml
new file mode 100644
index 00000000..7d6aadfb
--- /dev/null
+++ b/phpcs.xml
@@ -0,0 +1,148 @@
+<?xml version="1.0"?>
+<ruleset name="WordPress Coding Standards for WPGraphQL Plugins" xsi:noNamespaceSchemaLocation="https://raw.githubusercontent.com/squizlabs/PHP_CodeSniffer/master/phpcs.xsd">
+	<description>Sniffs for WPGraphQL Content Blocks</description>
+
+	<!-- What to scan: include any root-level PHP files, and the /includes folder -->
+	<file>./wp-graphql-content-blocks.php</file>
+	<file>./includes/</file>
+	<exclude-pattern>/vendor/</exclude-pattern>
+	<exclude-pattern>/node_modules/</exclude-pattern>
+	<!-- <exclude-pattern>**/tests/**</exclude-pattern> -->
+
+	<!-- How to scan: include CLI args so you don't need to pass them manually -->
+	<!-- Usage instructions: https://github.com/squizlabs/PHP_CodeSniffer/wiki/Usage -->
+	<!-- Annotated ruleset: https://github.com/squizlabs/PHP_CodeSniffer/wiki/Annotated-ruleset.xml -->
+
+	<!-- Show sniff and progress -->
+	<arg value="sp"/>
+	<!-- Strip the file paths down to the relevant bit -->
+	<arg name="basepath" value="./"/>
+	<!-- Enable colors in report -->
+	<arg name="colors"/>
+	<!-- Only lint php files by default -->
+	<arg name="extensions" value="php"/>
+	<!-- Whenever possible, cache the scan results and re-use those for unchanged files on the next scan. -->
+	<arg name="cache" value="tests/_output/cache.json" />
+	<!-- Enables parallel processing when available for faster results. -->
+	<arg name="parallel" value="20"/>
+	<!-- Set severity to 1 to see everything that isn't effectively turned off. -->
+	<arg name="severity" value="1" />
+
+	<!-- Ruleset Config: set these to match your project constraints-->
+
+	<!--
+		Tests for PHP version compatibility.
+		https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards#Recomended-additional-rulesets
+	-->
+	<config name="testVersion" value="7.4-"/>
+
+	<!--
+		Tests for WordPress version compatibility.
+		https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/wiki/Customizable-sniff-properties
+	-->
+	<config name="minimum_supported_wp_version" value="5.3"/>
+
+	<!-- Rules: WPGraphQL Coding Standards -->
+	<!-- https://github.com/AxeWP/WPGraphQL-Coding-Standards/WPGraphQL/ruleset.xml -->
+	<rule ref="WPGraphQL">
+		<!-- The is the only thing missing from WPEngine-Strict -->
+		<exclude name="Squiz.Commenting.InlineComment.InvalidEndChar" />
+		<!-- This should be excluded upstream in the ruleset-->
+		<exclude name="WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase" />
+		<!-- These maybe should be added back later -->
+		<exclude name="SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingTraversableTypeHintSpecification" />
+		<exclude name="SlevomatCodingStandard.TypeHints.PropertyTypeHint.MissingTraversableTypeHintSpecification" />
+		<exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingTraversableTypeHintSpecification" />
+		<exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingAnyTypeHint" />
+		<exclude name="WordPressVIPMinimum.Functions.RestrictedFunctions.wp_remote_get_wp_remote_get" />
+	</rule>
+
+	<!-- Individual rule configuration -->
+	<rule ref="WordPress.NamingConventions.PrefixAllGlobals">
+		<properties>
+			<!-- Value: replace the function, class, and variable prefixes used. Separate multiple prefixes with a comma. -->
+			<property name="prefixes" type="array">
+				<element value="wpgraphql\content_blocks" />
+				<element value="WPGRAPHQL\CONTENT_BLOCKS" />
+				<element value="WPGraphQL\ContentBlocks" />
+				<element value="wpgraphql_content_blocks_" />
+				<element value="WPGraphQLContentBlocks" />
+			</property>
+		</properties>
+	</rule>
+
+	<rule ref="WordPress.WP.I18n">
+		<properties>
+			<!-- Value: replace the text domain used. -->
+			<property name="text_domain" type="array" value="wp-graphql-content-blocks" />
+		</properties>
+	</rule>
+
+	
+	<!-- Exclude rules that break PHP Unit test conventions or result in redundant comments. -->
+	<!-- Test functions and properties are self-describing. -->
+	<!-- @todo These shouldnt be necessary, since tests are excluded -->
+	<rule ref="Generic.Commenting.DocComment.MissingShort">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Test file names are self-describing. -->
+	<rule ref="Squiz.Commenting.FileComment">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Test class names are self-describing. -->
+	<rule ref="Squiz.Commenting.ClassComment.Missing">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Test function names are self-describing. -->
+	<rule ref="Squiz.Commenting.FunctionComment">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Test variables should be self-describing. -->
+	<rule ref="Squiz.Commenting.VariableComment.Missing">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Test functions are not public so do not require @throws docblock annotations. -->
+	<rule ref="Squiz.Commenting.FunctionCommentThrowTag.Missing">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Multiple assignments are used as part of test setup. -->
+	<rule ref="Squiz.PHP.DisallowMultipleAssignments.Found">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<rule ref="WordPress.NamingConventions">
+		<exclude-pattern>/includes/utilities/DomHelpers.php</exclude-pattern>
+	</rule>
+
+	<!-- We follow the Codeception class name convention instead of the WordPress one. -->
+	<rule ref="WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedClassFound">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Test globals don't require prefixes. -->
+	<rule ref="WordPress.NamingConventions.PrefixAllGlobals">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- `json_encode()` is fine for tests. -->
+	<rule ref="WordPress.WP.AlternativeFunctions.json_encode_json_encode">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Allow queries, DROP and general database manipulation in tests. -->
+	<rule ref="WordPress.DB">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+	<!-- Suppress warnings about the _before() test methods used in Codeception tests. -->
+	<rule ref="PSR2.Methods.MethodDeclaration.Underscore">
+		<exclude-pattern>/tests/</exclude-pattern>
+	</rule>
+
+</ruleset>
diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index 9f0bf4a3..126f9fa7 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -3,7 +3,7 @@
 namespace WPGraphQL\ContentBlocks\Unit;
 
 final class CoreHeadingTest extends PluginTestCase {
-	public $post_id;
+	public int $post_id;
 
 	public function setUp(): void {
 		parent::setUp();
@@ -109,9 +109,6 @@ public function test_retrieve_core_heading_attributes() {
 		$this->assertEquals( 'text', $block['blockEditorCategoryName'], 'The blockEditorCategoryName should be text' );
 		$this->assertNotEmpty( $block['clientId'], 'The clientId should be present' );
 
-		// @todo this is not working
-		// $this->assertNotEmpty( $block['cssClassNames'], 'The cssClassNames should be present' );
-
 		$this->assertEmpty( $block['innerBlocks'], 'There should be no inner blocks' );
 		$this->assertEquals( 'core/heading', $block['name'], 'The block name should be core/heading' );
 		$this->assertEmpty( $block['parentClientId'], 'There should be no parentClientId' );
@@ -176,7 +173,7 @@ public function test_retrieve_core_heading_with_colors_and_alignment() {
 			[
 				'align'           => 'wide', // Previously untested
 				'anchor'          => null,
-				'backgroundColor' => null, // @todo: This is not returning correctly.
+				'backgroundColor' => null,
 				'className'       => null,
 				'content'         => 'Colored Heading',
 				'cssClassName'    => 'wp-block-heading has-text-align-right has-text-color has-background alignwide',
@@ -186,14 +183,62 @@ public function test_retrieve_core_heading_with_colors_and_alignment() {
 				'level'           => 3.0,
 				'lock'            => null,
 				'placeholder'     => null,
-				'style'           => wp_json_encode([
-					'color' => [
-						'background' => '#cf2e2e',
-						'text'       => '#ffffff',
-					],
-				]),
+				'style'           => wp_json_encode(
+					[
+						'color' => [
+							'background' => '#cf2e2e',
+							'text'       => '#ffffff',
+						],
+					]
+				),
 				'textAlign'       => 'right',
-				'textColor'       => null, // @todo: This is not returning correctly.
+				'textColor'       => null,
+			],
+			$attributes
+		);
+	}
+
+	public function test_retrieve_core_heading_with_background_text_color() {
+		$block_content = '
+		<!-- wp:heading {"textAlign":"right","level":3,"align":"wide","backgroundColor":"accent-4","textColor":"accent-3"} -->
+		<h3 class="wp-block-heading alignwide has-text-align-right has-accent-3-color has-accent-4-background-color has-text-color has-background">Colored Heading</h3>
+		<!-- /wp:heading -->
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
+
+		$block      = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals(
+			[
+				'align'           => 'wide',
+				'anchor'          => null,
+				'backgroundColor' => 'accent-4', // Previously untested
+				'className'       => null,
+				'content'         => 'Colored Heading',
+				'cssClassName'    => 'wp-block-heading alignwide has-text-align-right has-accent-3-color has-accent-4-background-color has-text-color has-background',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 3.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => 'right',
+				'textColor'       => 'accent-3', // Previously untested
 			],
 			$attributes
 		);
@@ -231,18 +276,20 @@ public function test_retrieve_core_heading_with_font_and_anchor() {
 				'className'       => null,
 				'content'         => 'Custom Font Heading',
 				'cssClassName'    => 'wp-block-heading',
-				'fontFamily' => null, // @todo: This is not returning correctly.
-				'fontSize' => null, // @todo: This is not returning correctly.
+				'fontFamily'      => null,
+				'fontSize'        => null,
 				'gradient'        => null,
 				'level'           => 2.0,
 				'lock'            => null,
 				'placeholder'     => null,
-				'style'           => wp_json_encode([
-					'typography' => [
-						'fontFamily' => 'Arial', // Previously untested
-						'fontSize'   => '32px', // Previously untested
-					],
-				]),
+				'style'           => wp_json_encode(
+					[
+						'typography' => [
+							'fontFamily' => 'Arial', // Previously untested
+							'fontSize'   => '32px', // Previously untested
+						],
+					]
+				),
 				'textAlign'       => null,
 				'textColor'       => null,
 			],
@@ -250,7 +297,99 @@ public function test_retrieve_core_heading_with_font_and_anchor() {
 		);
 	}
 
+	public function test_retrieve_core_heading_with_font_family_and_size() {
+		$block_content = '
+		<!-- wp:heading {"className":"is-style-default","backgroundColor":"accent","textColor":"contrast-2","fontSize":"xx-large","fontFamily":"system-sans-serif"} -->
+<h2 class="wp-block-heading is-style-default has-contrast-2-color has-accent-background-color has-text-color has-background has-system-sans-serif-font-family has-xx-large-font-size">hurrah</h2>
+<!-- /wp:heading -->
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
+
+		$block      = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => 'accent',
+				'className'       => 'is-style-default',
+				'content'         => 'hurrah',
+				'cssClassName'    => 'wp-block-heading is-style-default has-contrast-2-color has-accent-background-color has-text-color has-background has-system-sans-serif-font-family has-xx-large-font-size',
+				'fontFamily'      => 'system-sans-serif', // Previously untested
+				'fontSize'        => 'xx-large', // Previously untested
+				'gradient'        => null,
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => null,
+				'textColor'       => 'contrast-2',
+			],
+			$attributes
+		);
+	}
+
 	public function test_retrieve_core_heading_with_gradient() {
+		$block_content = '
+		<!-- wp:heading {"gradient":"gradient-3","fontSize":"medium"} -->
+		<h2 class="wp-block-heading has-gradient-3-gradient-background has-background has-medium-font-size">hello</h2>
+		<!-- /wp:heading -->
+		';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
+
+		$block      = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'hello',
+				'cssClassName'    => 'wp-block-heading has-gradient-3-gradient-background has-background has-medium-font-size',
+				'fontFamily'      => null,
+				'fontSize'        => 'medium',
+				'gradient'        => 'gradient-3', // Previously untested
+				'level'           => 2.0,
+				'lock'            => null,
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => null,
+				'textColor'       => null,
+			],
+			$attributes
+		);
+	}
+
+	public function test_retrieve_core_heading_with_custom_gradient() {
 		$block_content = '
 			<!-- wp:heading {"style":{"color":{"gradient":"linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)"}}} -->
 			<h2 class="wp-block-heading has-background" style="background:linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)">Gradient Heading</h2>
@@ -284,15 +423,17 @@ public function test_retrieve_core_heading_with_gradient() {
 				'cssClassName'    => 'wp-block-heading has-background',
 				'fontFamily'      => null,
 				'fontSize'        => null,
-				'gradient' => null, // @todo: This is not returning correctly.
+				'gradient'        => null,
 				'level'           => 2.0,
 				'lock'            => null,
 				'placeholder'     => null,
-				'style'           => wp_json_encode([
-					'color' => [
-						'gradient' => 'linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)',
-					],
-				]),
+				'style'           => wp_json_encode(
+					[
+						'color' => [
+							'gradient' => 'linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)',
+						],
+					]
+				),
 				'textAlign'       => null,
 				'textColor'       => null,
 			],

From ba1e53284a2163cf2432cc07e5a548cb92e90949 Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Tue, 24 Sep 2024 02:01:17 +0530
Subject: [PATCH 10/13] fix: Remove phpcs.xml

---
 phpcs.xml | 148 ------------------------------------------------------
 1 file changed, 148 deletions(-)
 delete mode 100644 phpcs.xml

diff --git a/phpcs.xml b/phpcs.xml
deleted file mode 100644
index 7d6aadfb..00000000
--- a/phpcs.xml
+++ /dev/null
@@ -1,148 +0,0 @@
-<?xml version="1.0"?>
-<ruleset name="WordPress Coding Standards for WPGraphQL Plugins" xsi:noNamespaceSchemaLocation="https://raw.githubusercontent.com/squizlabs/PHP_CodeSniffer/master/phpcs.xsd">
-	<description>Sniffs for WPGraphQL Content Blocks</description>
-
-	<!-- What to scan: include any root-level PHP files, and the /includes folder -->
-	<file>./wp-graphql-content-blocks.php</file>
-	<file>./includes/</file>
-	<exclude-pattern>/vendor/</exclude-pattern>
-	<exclude-pattern>/node_modules/</exclude-pattern>
-	<!-- <exclude-pattern>**/tests/**</exclude-pattern> -->
-
-	<!-- How to scan: include CLI args so you don't need to pass them manually -->
-	<!-- Usage instructions: https://github.com/squizlabs/PHP_CodeSniffer/wiki/Usage -->
-	<!-- Annotated ruleset: https://github.com/squizlabs/PHP_CodeSniffer/wiki/Annotated-ruleset.xml -->
-
-	<!-- Show sniff and progress -->
-	<arg value="sp"/>
-	<!-- Strip the file paths down to the relevant bit -->
-	<arg name="basepath" value="./"/>
-	<!-- Enable colors in report -->
-	<arg name="colors"/>
-	<!-- Only lint php files by default -->
-	<arg name="extensions" value="php"/>
-	<!-- Whenever possible, cache the scan results and re-use those for unchanged files on the next scan. -->
-	<arg name="cache" value="tests/_output/cache.json" />
-	<!-- Enables parallel processing when available for faster results. -->
-	<arg name="parallel" value="20"/>
-	<!-- Set severity to 1 to see everything that isn't effectively turned off. -->
-	<arg name="severity" value="1" />
-
-	<!-- Ruleset Config: set these to match your project constraints-->
-
-	<!--
-		Tests for PHP version compatibility.
-		https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards#Recomended-additional-rulesets
-	-->
-	<config name="testVersion" value="7.4-"/>
-
-	<!--
-		Tests for WordPress version compatibility.
-		https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/wiki/Customizable-sniff-properties
-	-->
-	<config name="minimum_supported_wp_version" value="5.3"/>
-
-	<!-- Rules: WPGraphQL Coding Standards -->
-	<!-- https://github.com/AxeWP/WPGraphQL-Coding-Standards/WPGraphQL/ruleset.xml -->
-	<rule ref="WPGraphQL">
-		<!-- The is the only thing missing from WPEngine-Strict -->
-		<exclude name="Squiz.Commenting.InlineComment.InvalidEndChar" />
-		<!-- This should be excluded upstream in the ruleset-->
-		<exclude name="WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase" />
-		<!-- These maybe should be added back later -->
-		<exclude name="SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingTraversableTypeHintSpecification" />
-		<exclude name="SlevomatCodingStandard.TypeHints.PropertyTypeHint.MissingTraversableTypeHintSpecification" />
-		<exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingTraversableTypeHintSpecification" />
-		<exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingAnyTypeHint" />
-		<exclude name="WordPressVIPMinimum.Functions.RestrictedFunctions.wp_remote_get_wp_remote_get" />
-	</rule>
-
-	<!-- Individual rule configuration -->
-	<rule ref="WordPress.NamingConventions.PrefixAllGlobals">
-		<properties>
-			<!-- Value: replace the function, class, and variable prefixes used. Separate multiple prefixes with a comma. -->
-			<property name="prefixes" type="array">
-				<element value="wpgraphql\content_blocks" />
-				<element value="WPGRAPHQL\CONTENT_BLOCKS" />
-				<element value="WPGraphQL\ContentBlocks" />
-				<element value="wpgraphql_content_blocks_" />
-				<element value="WPGraphQLContentBlocks" />
-			</property>
-		</properties>
-	</rule>
-
-	<rule ref="WordPress.WP.I18n">
-		<properties>
-			<!-- Value: replace the text domain used. -->
-			<property name="text_domain" type="array" value="wp-graphql-content-blocks" />
-		</properties>
-	</rule>
-
-	
-	<!-- Exclude rules that break PHP Unit test conventions or result in redundant comments. -->
-	<!-- Test functions and properties are self-describing. -->
-	<!-- @todo These shouldnt be necessary, since tests are excluded -->
-	<rule ref="Generic.Commenting.DocComment.MissingShort">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Test file names are self-describing. -->
-	<rule ref="Squiz.Commenting.FileComment">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Test class names are self-describing. -->
-	<rule ref="Squiz.Commenting.ClassComment.Missing">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Test function names are self-describing. -->
-	<rule ref="Squiz.Commenting.FunctionComment">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Test variables should be self-describing. -->
-	<rule ref="Squiz.Commenting.VariableComment.Missing">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Test functions are not public so do not require @throws docblock annotations. -->
-	<rule ref="Squiz.Commenting.FunctionCommentThrowTag.Missing">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Multiple assignments are used as part of test setup. -->
-	<rule ref="Squiz.PHP.DisallowMultipleAssignments.Found">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<rule ref="WordPress.NamingConventions">
-		<exclude-pattern>/includes/utilities/DomHelpers.php</exclude-pattern>
-	</rule>
-
-	<!-- We follow the Codeception class name convention instead of the WordPress one. -->
-	<rule ref="WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedClassFound">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Test globals don't require prefixes. -->
-	<rule ref="WordPress.NamingConventions.PrefixAllGlobals">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- `json_encode()` is fine for tests. -->
-	<rule ref="WordPress.WP.AlternativeFunctions.json_encode_json_encode">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Allow queries, DROP and general database manipulation in tests. -->
-	<rule ref="WordPress.DB">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-	<!-- Suppress warnings about the _before() test methods used in Codeception tests. -->
-	<rule ref="PSR2.Methods.MethodDeclaration.Underscore">
-		<exclude-pattern>/tests/</exclude-pattern>
-	</rule>
-
-</ruleset>

From 110103c0515ac07a949a5ac6abfee8b6cbc51c48 Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Tue, 24 Sep 2024 03:30:37 +0530
Subject: [PATCH 11/13] fix: Add doc-blocks, remove duplicate tests

---
 tests/unit/CoreHeadingTest.php | 290 ++++++---------------------------
 1 file changed, 50 insertions(+), 240 deletions(-)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index 126f9fa7..4af5c7f1 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -3,8 +3,16 @@
 namespace WPGraphQL\ContentBlocks\Unit;
 
 final class CoreHeadingTest extends PluginTestCase {
-	public int $post_id;
-
+	/**
+	 * The ID of the post created for the test.
+	 *
+	 * @var int
+	 */
+	public $post_id;
+
+	/**
+	 * Set up the test environment.
+	 */
 	public function setUp(): void {
 		parent::setUp();
 
@@ -19,6 +27,9 @@ public function setUp(): void {
 		\WPGraphQL::clear_schema();
 	}
 
+	/**
+	 * Tear down the test environment.
+	 */
 	public function tearDown(): void {
 		parent::tearDown();
 
@@ -27,6 +38,11 @@ public function tearDown(): void {
 		\WPGraphQL::clear_schema();
 	}
 
+	/**
+	 * Provide the GraphQL query for testing.
+	 *
+	 * @return string The GraphQL query.
+	 */
 	public function query(): string {
 		return '
 			fragment CoreHeadingBlockFragment on CoreHeading {
@@ -74,6 +90,11 @@ className
 		';
 	}
 
+	/**
+	 * Test the retrieval of core/heading block attributes.
+	 *
+	 * @return void
+	 */
 	public function test_retrieve_core_heading_attributes() {
 		$block_content = '
 			<!-- wp:heading {"level":2,"textAlign":"center","style":{"typography":{"fontSize":"28px","fontStyle":"normal","fontWeight":"700"}}} -->
@@ -145,6 +166,11 @@ public function test_retrieve_core_heading_attributes() {
 		);
 	}
 
+	/**
+	 * Test retrieval of core/heading block with colors and alignment.
+	 *
+	 * @return void
+	 */
 	public function test_retrieve_core_heading_with_colors_and_alignment() {
 		$block_content = '
 			<!-- wp:heading {"level":3,"textAlign":"right","align":"wide","style":{"color":{"background":"#cf2e2e","text":"#ffffff"}}} -->
@@ -198,6 +224,11 @@ public function test_retrieve_core_heading_with_colors_and_alignment() {
 		);
 	}
 
+	/**
+	 * Test retrieval of core/heading block with background and text color.
+	 *
+	 * @return void
+	 */
 	public function test_retrieve_core_heading_with_background_text_color() {
 		$block_content = '
 		<!-- wp:heading {"textAlign":"right","level":3,"align":"wide","backgroundColor":"accent-4","textColor":"accent-3"} -->
@@ -244,6 +275,11 @@ public function test_retrieve_core_heading_with_background_text_color() {
 		);
 	}
 
+	/**
+	 * Test retrieval of core/heading block with font and anchor.
+	 *
+	 * @return void
+	 */
 	public function test_retrieve_core_heading_with_font_and_anchor() {
 		$block_content = '
 			<!-- wp:heading {"anchor":"custom-id","style":{"typography":{"fontFamily":"Arial","fontSize":"32px"}}} -->
@@ -297,6 +333,11 @@ public function test_retrieve_core_heading_with_font_and_anchor() {
 		);
 	}
 
+	/**
+	 * Test retrieval of core/heading block with className, font family and font size.
+	 *
+	 * @return void
+	 */
 	public function test_retrieve_core_heading_with_font_family_and_size() {
 		$block_content = '
 		<!-- wp:heading {"className":"is-style-default","backgroundColor":"accent","textColor":"contrast-2","fontSize":"xx-large","fontFamily":"system-sans-serif"} -->
@@ -326,7 +367,7 @@ public function test_retrieve_core_heading_with_font_family_and_size() {
 				'align'           => null,
 				'anchor'          => null,
 				'backgroundColor' => 'accent',
-				'className'       => 'is-style-default',
+				'className'       => 'is-style-default', // Previously untested
 				'content'         => 'hurrah',
 				'cssClassName'    => 'wp-block-heading is-style-default has-contrast-2-color has-accent-background-color has-text-color has-background has-system-sans-serif-font-family has-xx-large-font-size',
 				'fontFamily'      => 'system-sans-serif', // Previously untested
@@ -343,6 +384,11 @@ public function test_retrieve_core_heading_with_font_family_and_size() {
 		);
 	}
 
+	/**
+	 * Test retrieval of core/heading block with font size & gradient.
+	 *
+	 * @return void
+	 */
 	public function test_retrieve_core_heading_with_gradient() {
 		$block_content = '
 		<!-- wp:heading {"gradient":"gradient-3","fontSize":"medium"} -->
@@ -376,7 +422,7 @@ public function test_retrieve_core_heading_with_gradient() {
 				'content'         => 'hello',
 				'cssClassName'    => 'wp-block-heading has-gradient-3-gradient-background has-background has-medium-font-size',
 				'fontFamily'      => null,
-				'fontSize'        => 'medium',
+				'fontSize'        => 'medium', // Previously untested
 				'gradient'        => 'gradient-3', // Previously untested
 				'level'           => 2.0,
 				'lock'            => null,
@@ -388,240 +434,4 @@ public function test_retrieve_core_heading_with_gradient() {
 			$attributes
 		);
 	}
-
-	public function test_retrieve_core_heading_with_custom_gradient() {
-		$block_content = '
-			<!-- wp:heading {"style":{"color":{"gradient":"linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)"}}} -->
-			<h2 class="wp-block-heading has-background" style="background:linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)">Gradient Heading</h2>
-			<!-- /wp:heading -->
-		';
-
-		wp_update_post(
-			[
-				'ID'           => $this->post_id,
-				'post_content' => $block_content,
-			]
-		);
-
-		$actual = graphql(
-			[
-				'query'     => $this->query(),
-				'variables' => [ 'id' => $this->post_id ],
-			]
-		);
-
-		$block      = $actual['data']['post']['editorBlocks'][0];
-		$attributes = $block['attributes'];
-
-		$this->assertEquals(
-			[
-				'align'           => null,
-				'anchor'          => null,
-				'backgroundColor' => null,
-				'className'       => null,
-				'content'         => 'Gradient Heading',
-				'cssClassName'    => 'wp-block-heading has-background',
-				'fontFamily'      => null,
-				'fontSize'        => null,
-				'gradient'        => null,
-				'level'           => 2.0,
-				'lock'            => null,
-				'placeholder'     => null,
-				'style'           => wp_json_encode(
-					[
-						'color' => [
-							'gradient' => 'linear-gradient(135deg,rgb(6,147,227) 0%,rgb(155,81,224) 100%)',
-						],
-					]
-				),
-				'textAlign'       => null,
-				'textColor'       => null,
-			],
-			$attributes
-		);
-	}
-
-	public function test_retrieve_core_heading_with_background_color() {
-		$block_content = '
-			<!-- wp:heading {"backgroundColor":"vivid-red-background-color"} -->
-			<h2 class="wp-block-heading has-vivid-red-background-color has-background">Heading with Background Color</h2>
-			<!-- /wp:heading>
-		';
-
-		wp_update_post(
-			[
-				'ID'           => $this->post_id,
-				'post_content' => $block_content,
-			]
-		);
-
-		$actual = graphql(
-			[
-				'query'     => $this->query(),
-				'variables' => [ 'id' => $this->post_id ],
-			]
-		);
-
-		$block      = $actual['data']['post']['editorBlocks'][0];
-		$attributes = $block['attributes'];
-
-		$this->assertEquals(
-			[
-				'align'           => null,
-				'anchor'          => null,
-				'backgroundColor' => 'vivid-red-background-color',
-				'className'       => null,
-				'content'         => 'Heading with Background Color',
-				'cssClassName'    => 'wp-block-heading has-vivid-red-background-color has-background',
-				'fontFamily'      => null,
-				'fontSize'        => null,
-				'gradient'        => null,
-				'level'           => 2.0,
-				'lock'            => null,
-				'placeholder'     => null,
-				'style'           => null,
-				'textAlign'       => null,
-				'textColor'       => null,
-			],
-			$attributes
-		);
-	}
-
-	public function test_retrieve_core_heading_with_text_color() {
-		$block_content = '
-			<!-- wp:heading {"textColor":"vivid-red"} -->
-			<h2 class="wp-block-heading has-vivid-red-color has-text-color">Heading with Text Color</h2>
-			<!-- /wp:heading>
-		';
-
-		wp_update_post(
-			[
-				'ID'           => $this->post_id,
-				'post_content' => $block_content,
-			]
-		);
-
-		$actual = graphql(
-			[
-				'query'     => $this->query(),
-				'variables' => [ 'id' => $this->post_id ],
-			]
-		);
-
-		$block      = $actual['data']['post']['editorBlocks'][0];
-		$attributes = $block['attributes'];
-
-		$this->assertEquals(
-			[
-				'align'           => null,
-				'anchor'          => null,
-				'backgroundColor' => null,
-				'className'       => null,
-				'content'         => 'Heading with Text Color',
-				'cssClassName'    => 'wp-block-heading has-vivid-red-color has-text-color',
-				'fontFamily'      => null,
-				'fontSize'        => null,
-				'gradient'        => null,
-				'level'           => 2.0,
-				'lock'            => null,
-				'placeholder'     => null,
-				'style'           => null,
-				'textAlign'       => null,
-				'textColor'       => 'vivid-red', // Previously untested
-			],
-			$attributes
-		);
-	}
-
-	public function test_retrieve_core_heading_with_font_size() {
-		$block_content = '
-			<!-- wp:heading {"fontSize":"large"} -->
-			<h2 class="wp-block-heading has-large-font-size">Heading with Font Size</h2>
-			<!-- /wp:heading>
-		';
-
-		wp_update_post(
-			[
-				'ID'           => $this->post_id,
-				'post_content' => $block_content,
-			]
-		);
-
-		$actual = graphql(
-			[
-				'query'     => $this->query(),
-				'variables' => [ 'id' => $this->post_id ],
-			]
-		);
-
-		$block      = $actual['data']['post']['editorBlocks'][0];
-		$attributes = $block['attributes'];
-
-		$this->assertEquals(
-			[
-				'align'           => null,
-				'anchor'          => null,
-				'backgroundColor' => null,
-				'className'       => null,
-				'content'         => 'Heading with Font Size',
-				'cssClassName'    => 'wp-block-heading has-large-font-size',
-				'fontFamily'      => null,
-				'fontSize'        => 'large', // Previously untested
-				'gradient'        => null,
-				'level'           => 2.0,
-				'lock'            => null,
-				'placeholder'     => null,
-				'style'           => null,
-				'textAlign'       => null,
-				'textColor'       => null,
-			],
-			$attributes
-		);
-	}
-
-	public function test_retrieve_core_heading_with_class_name() {
-		$block_content = '
-			<!-- wp:heading {"className":"custom-class"} -->
-			<h2 class="wp-block-heading custom-class">Heading with Custom Class</h2>
-			<!-- /wp:heading>
-		';
-
-		wp_update_post(
-			[
-				'ID'           => $this->post_id,
-				'post_content' => $block_content,
-			]
-		);
-
-		$actual = graphql(
-			[
-				'query'     => $this->query(),
-				'variables' => [ 'id' => $this->post_id ],
-			]
-		);
-
-		$block      = $actual['data']['post']['editorBlocks'][0];
-		$attributes = $block['attributes'];
-
-		$this->assertEquals(
-			[
-				'align'           => null,
-				'anchor'          => null,
-				'backgroundColor' => null,
-				'className'       => 'custom-class', // Previously untested
-				'content'         => 'Heading with Custom Class',
-				'cssClassName'    => 'wp-block-heading custom-class',
-				'fontFamily'      => null,
-				'fontSize'        => null,
-				'gradient'        => null,
-				'level'           => 2.0,
-				'lock'            => null,
-				'placeholder'     => null,
-				'style'           => null,
-				'textAlign'       => null,
-				'textColor'       => null,
-			],
-			$attributes
-		);
-	}
 }

From 33d60f5252de4264f2f348be541e577dd3e31eef Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Tue, 24 Sep 2024 03:34:21 +0530
Subject: [PATCH 12/13] fix: Add a lock attribue test

---
 tests/unit/CoreHeadingTest.php | 56 ++++++++++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index 4af5c7f1..bffc150c 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -434,4 +434,60 @@ public function test_retrieve_core_heading_with_gradient() {
 			$attributes
 		);
 	}
+
+	/**
+	 * Test retrieval of core/heading block with lock attribute.
+	 *
+	 * @return void
+	 */
+	public function test_retrieve_core_heading_with_lock() {
+		$block_content = '
+        <!-- wp:heading {"lock":{"move":true,"remove":true},"level":2} -->
+        <h2 class="wp-block-heading">Locked Heading</h2>
+        <!-- /wp:heading>
+        ';
+
+		wp_update_post(
+			[
+				'ID'           => $this->post_id,
+				'post_content' => $block_content,
+			]
+		);
+
+		$actual = graphql(
+			[
+				'query'     => $this->query(),
+				'variables' => [ 'id' => $this->post_id ],
+			]
+		);
+
+		$block      = $actual['data']['post']['editorBlocks'][0];
+		$attributes = $block['attributes'];
+
+		$this->assertEquals(
+			[
+				'align'           => null,
+				'anchor'          => null,
+				'backgroundColor' => null,
+				'className'       => null,
+				'content'         => 'Locked Heading',
+				'cssClassName'    => 'wp-block-heading',
+				'fontFamily'      => null,
+				'fontSize'        => null,
+				'gradient'        => null,
+				'level'           => 2.0,
+				'lock'            => wp_json_encode(
+					[
+						'move'   => true,
+						'remove' => true,
+					]
+				),
+				'placeholder'     => null,
+				'style'           => null,
+				'textAlign'       => null,
+				'textColor'       => null,
+			],
+			$attributes
+		);
+	}
 }

From 9bc6fe9328f1943c469a92aa47b8e353938f731f Mon Sep 17 00:00:00 2001
From: Ashutosh Gautam <ashutoshgautam10b11@gmail.com>
Date: Tue, 24 Sep 2024 03:47:33 +0530
Subject: [PATCH 13/13] docs: Add explicit list of "previously untested"
 attributes

---
 tests/unit/CoreHeadingTest.php | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/tests/unit/CoreHeadingTest.php b/tests/unit/CoreHeadingTest.php
index bffc150c..e6796548 100644
--- a/tests/unit/CoreHeadingTest.php
+++ b/tests/unit/CoreHeadingTest.php
@@ -93,6 +93,14 @@ className
 	/**
 	 * Test the retrieval of core/heading block attributes.
 	 *
+	 * This test covers the basic functionality of retrieving and validating
+	 * core/heading block attributes through GraphQL. It checks:
+	 *    - content
+	 *    - cssClassName
+	 *    - level
+	 *    - style (typography settings)
+	 *    - textAlign
+	 *
 	 * @return void
 	 */
 	public function test_retrieve_core_heading_attributes() {