search

reneoelke / Php53to54

PHP_CodeSniffer standard for compatibility checking of PHP 5.3 to 5.4 migrations.
http://foobugs.com
Other
6 stars 1 forks source link

UnitTests #56

Closed goatherd closed 12 years ago

goatherd commented 12 years ago

Unittests können per phpcs test Bootstrap ausgeführt werden.

Soll ich eine README in /tests anlegen, die potenziellen Nutzern weiterhilft (wer will schon groß darüber nachdenken, wo die bootstrap herumliegt, wenn es nur ums Debugging von Sniffs geht).

Auch sind die Testklassen quasi alle defekt. Dachte, dass es UnitTests hohe Priorität hätten? (zumindest habe ich auf die schnelle kein entsprechendes, offenes Issue gesehen).

goatherd commented 12 years ago

Zusätzliche Einschränkung: CodeSniffer tests funktionieren nicht mit (meinem) phpunit 3.6.11, weil er keine test*-Methode findet.

Auch können die includes (*.inc) nicht geladen werden, weil CodeSniffer diese im eigenen tests/Standards erwartet.

Wie soll weiter vorgegangen werden?

Werde lokal erst einmal AbstractSniffUnitTest patchen . Zusätzlich versuche ich die defekten Testklassen zum Laufen zu bringen. Coding standards für die Tests belasse ich erst einmal auf "willkürlich".

goatherd commented 12 years ago

Klassennamen der UnitTests sind defekt: SniffUnitTest muss UnitTest lauten, da sonst *SniffSniff geladen wird. (aktuelles CodeSniffer 1.3.4).

Fehlende UnitTests für Includes:

Tests, die versagen:

goatherd commented 12 years ago

Cleaned up code, added separate issues.