Skip to content

Block Hooks API: Harmonize ignoredHookedBlocks metadata injection logic #6550

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
tjcafferkey wants to merge 11 commits into from
Closed

Block Hooks API: Harmonize ignoredHookedBlocks metadata injection logic #6550

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e93c6a0
Update inject_ignored_hooked_blocks_metadata_attributes to handle wp_…
tjcafferkey May 14, 2024
9cd6fdc
Update since annotation for inject_ignored_hooked_blocks_metadata_att…
tjcafferkey May 14, 2024
6eaeba4
Update _build_block_template_object_from_prepared_database_object params
tjcafferkey May 14, 2024
cc50698
Coding standards
tjcafferkey May 14, 2024
b0c920f
Rename function from _build_block_template_object_from_prepared_datab…
tjcafferkey May 15, 2024
691efe7
Unit tests for wp_template objects
tjcafferkey May 15, 2024
d3aa4ad
template part unit tests
tjcafferkey May 15, 2024
a1765bc
isset check on post_type
tjcafferkey May 15, 2024
c8505e0
Remove else statement
tjcafferkey May 15, 2024
be242ab
Update function comment
tjcafferkey May 15, 2024
fa8cfb4
Add default-filters update
tjcafferkey May 15, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
template part unit tests
  • Loading branch information
@tjcafferkey
tjcafferkey committed May 15, 2024
commit d3aa4adc5c38aaadefdb8025a96d2310ad98f3f7
28 changes: 27 additions & 1 deletion tests/phpunit/tests/block-templates/base.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ abstract class WP_Block_Templates_UnitTestCase extends WP_UnitTestCase {
protected static $template_part_post;
protected static $uncustomized_template_db_object;
protected static $customized_template_db_object;
protected static $uncustomized_template_part_db_object;
protected static $customized_template_part_db_object;


public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
Expand Down Expand Up @@ -78,7 +80,6 @@ public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {

// Setup uncustomized template db object.
self::$uncustomized_template_db_object = (object) array(
'post_type' => 'wp_template',
'post_status' => 'publish',
'tax_input' => array(
'wp_theme' => self::TEST_THEME,
Expand All @@ -100,6 +101,31 @@ public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
'post_status' => 'publish',
'post_content' => '<!-- wp:heading {"level":1,"metadata":{"ignoredHookedBlocks":["tests/ignored"]}} --><h1>Template</h1><!-- /wp:heading -->',
);

// Setup uncustomized template part db object.
self::$uncustomized_template_part_db_object = (object) array(
'post_status' => 'publish',
'tax_input' => array(
'wp_theme' => self::TEST_THEME,
'wp_template_part_area' => WP_TEMPLATE_PART_AREA_HEADER,
),
'meta_input' => array(
'origin' => 'theme',
),
'post_content' => '<!-- wp:heading {"level":2,"metadata":{"ignoredHookedBlocks":["tests/ignored"]}} --><h2>Template Part</h2><!-- /wp:heading -->',
'post_type' => 'wp_template_part',
'post_name' => 'my_template_part',
'post_title' => 'My Template Part',
'post_excerpt' => 'Description of my template part',
);

// Setup customised template part db object.
self::$customized_template_part_db_object = (object) array(
'post_name' => 'my_template_part',
'post_title' => 'My Customized Template Part',
'post_status' => 'publish',
'post_content' => '<!-- wp:heading {"level":2,"metadata":{"ignoredHookedBlocks":["tests/ignored"]}} --><h2>Template Customized Part</h2><!-- /wp:heading -->',
);
}

public static function wpTearDownAfterClass() {
Expand Down
35 changes: 35 additions & 0 deletions tests/phpunit/tests/block-templates/buildBlockTemplateObjectFromDatabaseObject.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,4 +61,39 @@ public function test_should_build_template_from_customized_object() {
$this->assertSame( 'Description of my template', $template->description );
$this->assertSame( 'wp_template', $template->type );
}

/**
* @ticket 60759
*/
public function test_should_build_template_part_from_uncustomized_object() {
$template = _build_block_template_object_from_database_object( self::$uncustomized_template_part_db_object );

$this->assertNotWPError( $template );
$this->assertSame( get_stylesheet() . '//my_template_part', $template->id );
$this->assertSame( get_stylesheet(), $template->theme );
$this->assertSame( 'my_template_part', $template->slug );
$this->assertSame( 'publish', $template->status );
$this->assertSame( 'custom', $template->source );
$this->assertSame( 'My Template Part', $template->title );
$this->assertSame( 'Description of my template part', $template->description );
$this->assertSame( 'wp_template_part', $template->type );
}

/**
* @ticket 60759
*/
public function test_should_build_template_part_from_customized_object() {
self::$customized_template_part_db_object->ID = self::$template_part_post->ID;
$template = _build_block_template_object_from_database_object( self::$customized_template_part_db_object );

$this->assertNotWPError( $template );
$this->assertSame( get_stylesheet() . '//my_template_part', $template->id );
$this->assertSame( get_stylesheet(), $template->theme );
$this->assertSame( 'my_template_part', $template->slug );
$this->assertSame( 'publish', $template->status );
$this->assertSame( 'custom', $template->source );
$this->assertSame( 'My Customized Template Part', $template->title );
$this->assertSame( 'Description of my template part', $template->description );
$this->assertSame( 'wp_template_part', $template->type );
}
}