Problem/Motivation

When running ParagraphsAccessTest.php, WebAssert::responseContains should have just one argument.

Steps to reproduce

Proposed resolution

Remove one extra argument to ::responseContains.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 3229999-responseContains-argument-3.patch1.72 KBtaniachque
#2 3229999-responseContains-argument-2.patch2.64 KBtaniachque
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

taniachque created an issue. See original summary.

taniachque’s picture

taniachque CreditAttribution: taniachque at MD Systems GmbH commented
Status: Active » Needs review
FileSize
3229999-responseContains-argument-2.patch2.64 KB

Updating changes in ParagraphsAccessTest.php.

taniachque’s picture

taniachque CreditAttribution: taniachque at MD Systems GmbH commented
FileSize
3229999-responseContains-argument-3.patch1.72 KB

Uploading the right patch.

The last submitted patch, 2: 3229999-responseContains-argument-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Thanks, committed.

  • Berdir committed 5408c7d on 8.x-1.x authored by taniachque 
    Issue #3229999 by taniachque: ::responseContains should have one...

Status: Fixed » Closed (fixed)

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