A few raw-handling API unit tests fail in the native test environment

[fluiddot]

Description

The following unit tests of raw-handling API fail when running them in the native test environment:

• html-formatting-remover
• phrasing-content-reducer
• ms-list-converter
• figure-content-reducer
• special-comment-converter
• normalise-blocks
• image-corrector

We currently don't run them as part of the mobile unit test suite (reference), but it would be recommended as the logic covered is used in the content pasting functionality used in the native version.

gutenberg/packages/blocks/src/api/raw-handling/paste-handler.js

Lines 191 to 222 in 820b135

	const filters = [
	googleDocsUIDRemover,
	msListConverter,
	headRemover,
	listReducer,
	imageCorrector,
	phrasingContentReducer,
	specialCommentConverter,
	commentRemover,
	iframeRemover,
	figureContentReducer,
	blockquoteNormaliser,
	divNormaliser,
	];
	const schema = {
	...blockContentSchema,
	// Keep top-level phrasing content, normalised by `normaliseBlocks`.
	...phrasingContentSchema,
	};
	piece = deepFilterHTML( piece, filters, blockContentSchema );
	piece = removeInvalidHTML( piece, schema );
	piece = normaliseBlocks( piece );
	piece = deepFilterHTML(
	piece,
	[ htmlFormattingRemover, brRemover, emptyParagraphRemover ],
	blockContentSchema
	);
	// Allows us to ask for this information when we get a report.
	console.log( 'Processed HTML piece:\n\n', piece );

In fact, I found that three of them produce a crash in the editor (only referencing iOS issues but we should expect the same on Android):

• ms-list-converter: RCTFatalException: Unhandled JS Exception: TypeError: undefined is not a function, js engine: hermes: Unhandled JS Exception: TypeError: undefined is not a function, js engine: hermes, stack: wordpress-mobile/gutenberg-mobile#6411 (solved in [RNMobile] Fixes a crash on pasting MS Word list markup #56653)
• special-comment-converter: RCTFatalException: Unhandled JS Exception: TypeError: Cannot set property 'block' of undefined, js engine: hermes: Unhandled JS Exception: TypeError: Cannot set property 'block' of undefined, js engine: hermes, s... wordpress-mobile/gutenberg-mobile#6229 (solved in [Mobile] - Crash fixes when pasting HTML content #60676)
• image-corrector: EXC_BREAKPOINT: canLoadImageURL: wordpress-mobile/gutenberg-mobile#6332 (solved in [Mobile] - Crash fixes when pasting HTML content #60676)

Step-by-step reproduction instructions

⚠️ : When copying HTML into the editor, use the button ) located in the top-right corner of the HTML content in GitHub.

ms-list-converter

1. Create/open a post in the app.
2. Copy the following HTML content:

<p style="mso-list:l0 level1 lfo1"><span style="mso-list:Ignore">* </span>test</p>

3. Paste the HTML content into a Paragraph block.
4. Observe the crash.

special-comment-converter

1. Create/open a post in the app.
2. Copy the following HTML content:

<p><!--more--></p>

3. Paste the HTML content into a Paragraph block.
4. Observe the crash.

image-corrector

1. Create/open a post in the app.
2. Copy the following HTML content.

<div><img src="file:LOW-RES.png"></div>

3. Paste the HTML content into a Paragraph block.
4. Observe the crash.

Expected behaviour

• raw-handling unit tests should pass when running them in the native test environment.
• Pasting content into the editor shouldn't produce a crash.

Actual behaviour

• A few of the raw-handling unit tests fail when running them in the native test environment.
• Pasting content into the editor produces a crash in some cases.

Screenshots or screen recording (optional)

N/A

WordPress information

• WordPress version: N/A
• Gutenberg version: N/A
• Are all plugins except Gutenberg deactivated? N/A
• Are you using a default theme (e.g. Twenty Twenty-One)? N/A

Device information

• Device: iPhone 15 (simulator)
• Operating system: iOS 17.0
• WordPress app version: 23.5

[fluiddot]

The unit tests related to ms-list-converter will be enabled via #56653.

[fluiddot]

ℹ️ Related to this issue, in #56723 we discovered that several test cases related to processing raw blocks fail in the native version. As part of resolving this issue, we should enable/uncomment the test cases from test/native/integration/blocks-raw-handling.native.js and provide fixes to address the failures.