Problem/Motivation

drupalPostForm in functional tests is deprecated.

See: https://www.drupal.org/node/3168858

Proposed resolution

Using UiHelperTrait::drupalPostForm() in functional tests is deprecated. Use a combination of ::drupalGet and ::submitForm instead.

CommentFileSizeAuthor
#6 3174124-6.patch4.55 KBpaulocs
#2 3174124-2.patch262.13 KBpaulocs
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

paulocs created an issue. See original summary.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
CreditAttribution: paulocs at CI&T commented
Assigned: paulocs » Unassigned
Status: Active » Needs review
FileSize
3174124-2.patch262.13 KB
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs review » Needs work

I prefer having webform provide its own ::drupalPostForm().

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented

We can add the ::drupalPostForm method to \Drupal\Tests\webform\Traits\WebformAssertLegacyTrait.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
CreditAttribution: paulocs at CI&T commented
Assigned: Unassigned » paulocs

okay,
I'will do it.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
CreditAttribution: paulocs at CI&T commented
Assigned: paulocs » Unassigned
Status: Needs work » Needs review
FileSize
3174124-6.patch4.55 KB

New patch.
Please review.

  • paulocs authored 3403e87 on 6.x 
    Issue #3174124 by paulocs: drupalPostForm in functional tests is...
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.