-
Notifications
You must be signed in to change notification settings - Fork 114
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tests PSR compatible #34
Conversation
I'm good with the move to PSR-4. Punting to @mattfarina to weigh in on this. |
@mattfarina what do you think about this? |
use HTML5\Parser\StringInputStream; | ||
use HTML5\Parser\Scanner; | ||
use HTML5\Parser\Tokenizer; | ||
require 'EventStack.php';class TokenizerTest extends \HTML5\Tests\TestCase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The require and class definition should be on different lines.
The change to a PSR-4 autoloader for the tests makes sense. Let's clean up the couple issues and we can merge this in. |
Changes Unknown when pulling 30aa8a6 on goetas:prs-tests into * on Masterminds:master*. |
Fixes the testing errors and makes the tests PSR compatible
I have adapted the test suite to be PSR compatible