Fix CI failures: e2e name conflicts and PHP test issues

E2E tests: The menuitemradio name 'Edit' matched 3 elements (our
intent choice plus Visual/Code editor modes). Switch to regex
patterns (e.g., /^Suggest/, /^View\s+Read-only/) to uniquely match
intent switcher items by their label + info text. Also add waitFor
on the menu item in switchIntent for timing resilience.

PHP tests: Allow empty note content when _wp_suggestion meta is
present (matches the existing pattern for resolution notes). Fix
test_editor_can_update_note_on_own_post by setting comment_approved.
Fix test_suggestion_status_ignores_invalid_value to check the stored
value rather than expecting a 400 (WP meta enum validation silently
drops invalid values).

Refs #73411

Author: adamsilverstein

test/e2e/specs/editor/various/suggestion-mode.spec.js

@@ -7,9 +7,11 @@ async function switchIntent( page, intentLabel ) {
 	await page.click(
 		'role=region[name="Editor top bar"i] >> role=button[name="Options"i]'
 	);
-	await page
-		.getByRole( 'menuitemradio', { name: intentLabel, exact: true } )
-		.click();
+	const menuItem = page.getByRole( 'menuitemradio', {
+		name: new RegExp( `^${ intentLabel }` ),
+	} );
+	await menuItem.waitFor( { state: 'visible', timeout: 10000 } );
+	await menuItem.click();
 }
 
 test.describe( 'Suggestion mode', () => {