Follow-up to #2752659: [META] Convert SimpleTest Tests to PHPUnit

Core is deprecating SimpleTest for Drupal 9 and converting all the tests #2735005: Convert all Simpletest web tests to BrowserTestBase (or UnitTestBase/KernelTestBase).

We should also do this.

CommentFileSizeAuthor
#8 ParagraphsPreviewTest_2753887-8.patch2.68 KBjmuzz
#2 ParagraphsPreviewTest_2753887-1.patch5.91 KBbalagan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

GoZ created an issue. See original summary.

balagan’s picture

balagan CreditAttribution: balagan commented
FileSize
ParagraphsPreviewTest_2753887-1.patch5.91 KB

I tried to work on this at devdays Milan, and ended up with a file that does not fail nor succeeds. I attach it as a reference, not sure what is wrong.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Active » Needs review

You need to switch to needs review to trigger the test bot processing it.

yobottehg’s picture

yobottehg CreditAttribution: yobottehg commented
Assigned: Unassigned » yobottehg
Status: Needs review » Needs work

This tests wont work.

I'll take a stab at this.

The last submitted patch, 2: ParagraphsPreviewTest_2753887-1.patch, failed testing.

The last submitted patch, 2: ParagraphsPreviewTest_2753887-1.patch, failed testing.

The last submitted patch, 2: ParagraphsPreviewTest_2753887-1.patch, failed testing.

jmuzz’s picture

jmuzz CreditAttribution: jmuzz commented
Status: Needs work » Postponed
Related issues: +#2784537: Add legacy assertFieldByXPath()/assertNoFieldByXPath() method for browser tests, +#2794347: Convert web tests to browser tests for field_ui module
FileSize
ParagraphsPreviewTest_2753887-8.patch2.68 KB

This passes for me. I am setting to postponed because it depends on patches in the core issues I am relating.

Lendude’s picture

Lendude
Primary language Dutch
Location Amsterdam
CreditAttribution: Lendude as a volunteer and at ezCompany commented
Status: Postponed » Closed (outdated)

This is now being done in #2753889: Convert ParagraphsTestBase and the easy-to-convert tests extending it to BrowserTestBase