-
Notifications
You must be signed in to change notification settings - Fork 2
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
A proper test suite #1
Conversation
…onality is improved
|
||
class Wrapper | ||
{ | ||
protected $output = ''; | ||
protected $currentPosition = 0; | ||
|
||
protected static $overrideWrappers = array('http', 'https'); | ||
public static $overrideWrappers = array('http', 'https'); |
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.
remind me why this is public again?
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.
It is intended to allow the user more flexibility in what they choose to mock. I will implement a getter and setter for this instead.
Tests looked solid, made a few minor comments. Take a look before I merge. |
There are a few small changes to the actual Spray code, but this pull is mostly to give Spray a proper test suite