phpcs: 12 failed, 12 total tests

Created on 28 October 2024, about 2 months ago

Problem/Motivation

Running the new test phpcs is allowed to pass, but should be fixed:
https://git.drupalcode.org/project/gnu_terry_pratchett/-/jobs/3178875

Steps to reproduce

Run phpcs

Proposed resolution

Review the tests to evaluate the issues and fix them.

Remaining tasks

Fix the code to allow the tests to pass.
Commit the code
Re-run the test

šŸ› Bug report
Status

Active

Version

9.2

Component

Code

Created by

šŸ‡¬šŸ‡§United Kingdom the_g_bomb

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

Merge Requests

Comments & Activities

  • Issue created by @the_g_bomb
  • šŸ‡¬šŸ‡§United Kingdom the_g_bomb
  • Hi @the_g_bomb,

    I have verified the phpcs issues on my terminal, the commits you made on MR!4 can be applied successfully and after applying it, confirmed most issues were fixed, however, one error still persisted. Please see below:

     gnu_terry_pratchett git:(main) āœ— curl https://git.drupalcode.org/project/gnu_terry_pratchett/-/merge_requests/4.diff | patch -p1
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100  2951    0  2951    0     0   8990      0 --:--:-- --:--:-- --:--:--  9338
    patching file gnu_terry_pratchett.module
    patching file gnu_terry_pratchett.services.yml
    patching file src/EventSubscriber/XClacksSubscriber.php
    patching file src/Tests/Headers/XClacksTest.php
    āžœ  gnu_terry_pratchett git:(main) āœ— ..
    āžœ  contrib git:(main) āœ— phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig gnu_terry_pratchett
    
    FILE: .../web/modules/contrib/gnu_terry_pratchett/src/Tests/Headers/XClacksTest.php
    --------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    --------------------------------------------------------------------------------
     32 | ERROR | Public method name "XClacksTest::testXClacksHeaders" is not in
        |       | lowerCamel format
    --------------------------------------------------------------------------------
    
    Time: 135ms; Memory: 10MB

    Kindly check

    Thanks,
    Jake

  • šŸ‡®šŸ‡³India sayan_k_dutta

    Working on it.

  • šŸ‡®šŸ‡³India sayan_k_dutta

    Added 1 commit to the MR, please check.

  • Hi sayan_k_dutta,

    Confirmed the commit you added to the MR fixed the reported issue.

    gnu_terry_pratchett git:(main) āœ— curl https://git.drupalcode.org/project/gnu_terry_pratchett/-/merge_requests/4.diff | patch -p1
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100  3290    0  3290    0     0   9154      0 --:--:-- --:--:-- --:--:--  9426
    patching file gnu_terry_pratchett.module
    patching file gnu_terry_pratchett.services.yml
    patching file src/EventSubscriber/XClacksSubscriber.php
    patching file src/Tests/Headers/XClacksTest.php
    āžœ  gnu_terry_pratchett git:(main) āœ— ..
    āžœ  contrib git:(main) āœ— phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig gnu_terry_pratchett
    āžœ  contrib git:(main) āœ—

    Will now move this to RTBC

    Thanks,
    Jake

  • Pipeline finished with Failed
    about 1 month ago
    #338137
  • Pipeline finished with Failed
    about 1 month ago
    #338138
  • Pipeline finished with Success
    about 1 month ago
    Total: 142s
    #338139
  • Hi @everyone,

    I have also enabled and ran the phpcs job confirmed passed.

    Thanks,
    Jake

  • Pipeline finished with Success
    about 1 month ago
    Total: 136s
    #338144
Production build 0.71.5 2024