loadsys / loadsys/loadsys_codesniffer
Add coverage for all sniffs identified by testUntested()
まだ誰も着手していません。
- 主要言語
- 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.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- 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