Fix PHPStan L2 error "Call to an undefined method Drupal\Tests\WebAssert::{Method}"

Created on 12 April 2023, almost 2 years ago

Problem/Motivation

There is 500+ warnings thrown by PHPStan level 2 analysis in FunctionalJavascript tests that relate to Call to an undefined method Drupal\Tests\WebAssert::{Method}. The main methods I saw thrown were ::waitForField & ::waitForElement

Steps to reproduce

Review https://dispatcher.drupalci.org/job/drupal_patches/157898/consoleFull and see the errors relating to this issue.

Proposed resolution

Change the return type on WebDriverTestBase::assertSession() from WebAssert to the more specific WebDriverWebAssert

Remaining tasks

Create MR & see if tests pass
Discuss if this is the best approach
Commit!

User interface changes

N/A

API changes

WebDriverTestBase::assertSession return type will become WebDriverWebAssert

Data model changes

N/A

Release notes snippet

Is this needed?

📌 Task

Status

Postponed

Version

10.1 ✨

Component

Other →

Last updated 1 day ago

Created by

🇳🇿New Zealand danielveza Brisbane, AU

Live updates comments and jobs are added and updated live.

PHPStan-2

Comments & Activities

Issue created by @danielveza
Status changed to Postponed almost 2 years ago12:50am 12 April 2023
Comment almost 2 years ago →
🇳🇿New Zealand danielveza Brisbane, AU
Ah - Sorry, after opening this I can see that we may be able to fold this into the work done in 📌 Fix PHPStan L2 error in traits Needs work . Postponing based on outcome from that

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024