Expand PHPUnit test coverage of the Gettext component

Created on 16 April 2015, over 9 years ago
Updated 2 February 2024, 10 months ago

Problem/Motivation

The Drupal\Component\Gettext component doesn't appear to have much test coverage (PHPUnit or otherwise).

Proposed resolution

Add some PHPUnit tests!

Remaining tasks

User interface changes

API changes

📌 Task
Status

Needs work

Version

11.0 🔥

Component
Base 

Last updated about 1 hour ago

Created by

🇺🇸United States jhedstrom Portland, OR

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • @hardik_patel_12 opened merge request.
  • Status changed to Needs review 10 months ago
  • Status changed to Needs work 10 months ago
  • 🇺🇸United States smustgrave

    All new functions or parameters should be typehinted with returns

  • Status changed to Needs review 10 months ago
  • Status changed to RTBC 10 months ago
  • 🇺🇸United States smustgrave

    Feedback appears to be addressed and I didn't see any other instances of coverage for this.

  • Status changed to Needs work 10 months ago
  • 🇳🇿New Zealand quietone

    I'm adding a reminder that evidence of a code review is expected when setting an issue to RTBC. For this issue, all I see is comment #32 which mentions return types. I would expect to see some comment about the test coverage and the the supplied test cases are sufficient.

    Setting to NW for the comments in the MR.

Production build 0.71.5 2024