loadsys / loadsys/loadsys_codesniffer

Add coverage for all sniffs identified by testUntested()

オープン
#39 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

bug help-wanted
主要言語
PHP
スター
0
フォーク
0
PR マージ指標
30日以内にマージされた PR はありません

説明

The test suite now compares the full list of sniffs employed by a coding standard against those in the test sample file `//~` annotations and throws an assertion failure if the sample files fail to "cover" all tests.

This acts as a crude kind of "coverage" metric of the sniffs defined in the ruleset, but right now we're only covering about 31 out of 120+ sniffs.

To resolve this, uncomment the `markTestIncomplete()` in `LoadsysStandardTest::testUntested()`. Then repeatedly run `vendor/bin/phpunit` and add new sample files for each remaining sniff name listed. Ideally, there should be two files for each sniff-- one confirming that incorrect style is caught by our ruleset, and another that correct behavior is _not_ caught. The new files should be arranged in the `tests/files/must/` and `tests/files/must_not/` folders. Read the `## Testing` section of the README for more naming details and goals.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

LoadsysStandardTest::testUntested() から始め、markTestIncomplete() のコメントを解除して vendor/bin/phpunit を実行し、残っている sniff 名を特定します。README の Testing セクションを読み、その後、残っている sniff 用のサンプルファイルを tests/files/must/ と tests/files/must_not/ の下に追加します。完全なルールセットの一覧が網羅され、テストスイートが成功すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
php
領域
testing
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。