loadsys / loadsys/loadsys_codesniffer
Add coverage for all sniffs identified by testUntested()
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 0
- Forks
- 0
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in LoadsysStandardTest::testUntested(), entferne die Auskommentierung von markTestIncomplete() und führe vendor/bin/phpunit aus, um die verbleibenden Sniff-Namen zu identifizieren. Lies den Testing-Abschnitt der README und füge dann Beispieldateien unter tests/files/must/ und tests/files/must_not/ für die verbleibenden Sniffs hinzu; abgeschlossen ist die Aufgabe, wenn die vollständige Regelsatzliste abgedeckt ist und die Testsuite erfolgreich durchläuft.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- testing
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100