loadsys / loadsys/loadsys_codesniffer
Add coverage for all sniffs identified by testUntested()
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in LoadsysStandardTest::testUntested(), decommenta markTestIncomplete() ed esegui vendor/bin/phpunit per identificare i nomi degli sniff rimanenti. Leggi la sezione Testing del README, quindi aggiungi file di esempio in tests/files/must/ e tests/files/must_not/ per gli sniff rimanenti; il lavoro è completato quando l’intero elenco delle regole è coperto e la suite di test ha esito positivo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- testing
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 35/100