Gallery block fixtures in phpunit tests need updating

[glendaviesnz]

In 5.9 the Gallery block was updated to use nested image blocks, and to be wrapped in a figure tag, but the block fixtures still us the old ul and internal image format.

The fixtures should be updated to the new block format.

The Gallery block unit tests still use the old format, so this PR updates it to the new format that uses nested Image blocks.
---
Fixes: https://core.trac.wordpress.org/ticket/55571
## To test

Run:

• npm run test:php -- --filter Tests_Blocks_wpBlockParser
• npm run test:php -- --filter Tests_Blocks_Render

This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See ​GitHub Pull Requests for Code Review in the Core Handbook for more details.

[gziolo]

In 53261:

Tests: Update Gallery block unit tests to new gallery format

The Gallery block unit tests still use the old format, so this updates it to the new format that uses nested Image blocks.

Props glendaviesnz.
Fixes #55571.

Committed with https://core.trac.wordpress.org/changeset/53261.

[peterwilsoncc]

I missed this earlier, sorry.

I think it would be good to maintain tests for both the updated and legacy format. That will ensure that any code changes don't break the legacy format on sites that haven't upgraded.

Will putting the fixture back as core__gallery-legacy.* cause problems?

I'll reopen this for now while it's discussed.

[glendaviesnz]

Will putting the fixture back as coregallery-legacy.* cause problems?

Good question, @gziolo do you know what the process has been in the past for deprecated block formats in these fixtures - the only other deprecated fixtures here are for columns and paragraphs, and these only have a single deprecation, but there have been many other deprecations that aren't covered at all in these fixtures?

[gziolo]

I didn't think about keeping both versions, it makes a lot of sense!

I would have to double-check the implementation in the core, but in Gutenberg, we have multiple versions per block, including a few deprecated versions so what @peterwilsoncc suggested would work.

[peterwilsoncc]

Thanks folks, let's go with __deprecated-1 rather than -legacy so the naming convention is consistent between repositories.

Tests can go in at any time so this can wait until after RC 1 if you have other higher priority tasks to do.

[glendaviesnz]

Ok, will take a look at adding these back sometime in the next few days.

[hellofromTonya]

Though tests can go in at anytime, moving this to 6.1 milestone to clear 6.0 for RC1 (which is tomorrow).

[desrosj]

I came across this one scrubbing tickets today, and I'm wondering. Would this help us find and prevent backwards compatibility problems such as the ones described in #53617? Or am I misunderstanding what the tests check for?

[desrosj]

RC1 is in a few hours. Going to punt this to 6.2.

This reverts commit 3cc94be23b31d8eb22c81a615b749f82bd6de2ad.

And then will do more.

https://core.trac.wordpress.org/ticket/55571

[peterwilsoncc]

In the ​linked pull request I have:

• added all the deprecated gallery tests included in the Gutenberg repo to duplicate them in wp-develop
• generated the additional .server.html files used in wp-dev but not the gutenberg repo
• modified the server versions so they pass, however I don't think they are correct as I've ​noted on the pull request.

[gziolo]

In 55439:

Editor: Only add layout classes to inner wrapper if block is a container

Backport of ​https://github.com/WordPress/gutenberg/issues/48606, which fixes an issue found in adding fixtures for deprecated Gallery block versions in #55571.

See #55571.
Fixes #57831.
Props isabel_brison, Mamaduka, dasnitesh780.

I think the null blocks come from end of file empty lines.
I tried using trim to remove them but it broke a bunch of other tests that already had trailing null blocks recorded.

[peterwilsoncc]

In 55471:

Build/Tests Tools: Add unit tests for Gallery blocks.

Introduces unit tests for the following blocks

• Gallery block with caption
• Gallery block, deprecations 1 thru 7

Updates the unit tests for the following blocks to match the counterparts stored in the Gutenberg repository:

• Gallery block
• Gallery block with columns

Modifies Tests_Blocks_Render::test_do_block_output() to ignore white space at the end of lines to account for whitespace equivalence in HTML.

Props peterwilsoncc, isabel_brison, gziolo.
Fixes #55571.

Merged in r55471 / ​https://github.com/WordPress/wordpress-develop/commit/4d3627fd2eab274c55ca4fb22dbdc0de64b17112