Rename mis-named data providers discovered in #3421418

Created on 18 June 2024, 5 months ago
Updated 29 June 2024, 5 months ago

Problem/Motivation

In 📌 Add void return typehints to all test methods Active , we noticed a number of data provider methods that were misleading named testSomethingProvider(), rather than the correct providerTestSomething(). These included:

  • DateFormatAccessControlHandlerTest::testAccessProvider()
  • MenuAccessControlHandlerTest::testAccessProvider()
  • JsonApiDocumentTopLevelNormalizerTest::testCacheableMetadataProvider()

There might be others.

Proposed resolution

Rename the methods so that the word "provider" comes at the beginning rather than the end, and update the @dataProvider entries accordingly.

Remaining tasks

TBD

API changes

A few data providers renamed according to standard practice

Release notes snippet

N/A

📌 Task
Status

Postponed

Version

11.0 🔥

Component
PHPUnit 

Last updated about 1 hour ago

Created by

🇺🇸United States xjm

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024