Problem/Motivation

HEAD of 4.x tests have been failing since around D9.x something
review this now that we're onto D10 and beyond.

Steps to reproduce

review test results

Proposed resolution

TBD

Remaining tasks

TBD

User interface changes

TBD

API changes

TBD

Data model changes

TBD

CommentFileSizeAuthor
#12 panels_ipe-3383474-12.patch1.53 KBjoseph.olstad
#10 panels_ipe-3383474-10.patch1.56 KBjoseph.olstad
#6 panels_ipe-explicit_support-3383474-06.patch4.54 KBjoseph.olstad
#2 panels_ipe-explicit_support-3383474-02.patch825 bytesjoseph.olstad
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joseph.olstad created an issue. See original summary.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
FileSize
panels_ipe-explicit_support-3383474-02.patch825 bytes

see if this helps

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Needs work » Needs review
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

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

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

drupalPostForm() in functional tests is deprecated

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
FileSize
panels_ipe-explicit_support-3383474-06.patch4.54 KB
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Title: Fix HEAD test failures for 4.x branch possibly related to recent core or ctools changes » Fix HEAD test failures - drupalPostForm() in functional tests is deprecated
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Title: Fix HEAD test failures - drupalPostForm() in functional tests is deprecated » Fix HEAD test failures for 4.x
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
FileSize
panels_ipe-3383474-10.patch1.56 KB

Oops that was 3x, here's 4x

I was confused

Status: Needs review » Needs work

The last submitted patch, 10: panels_ipe-3383474-10.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Needs work » Needs review
FileSize
panels_ipe-3383474-12.patch1.53 KB

encore mieux

Status: Needs review » Needs work

The last submitted patch, 12: panels_ipe-3383474-12.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

While it is incomplete, I think it's worthy of commit as it's improving the results.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented
Status: Needs work » Needs review

Thanks for taking on the test failures @joseph.olstad. I've committed this WIP.

I'll leave this open in case you have some ideas on the IPE errors.