Skip to content
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

Allow use of ReferenceExpectation in expect calls (-PHP4) #7

Merged
merged 1 commit into from
Jul 27, 2015

Conversation

mal
Copy link
Contributor

@mal mal commented Jul 27, 2015

Make ReferenceExpectation inherit from EqualException so it will be
recognised by SimpleExpectation::isExpectation.

Alternate approach to #5 that doesn't maintain PHP4 compat.

Make ReferenceExpectation inherit from EqualException so it will be
recognised by SimpleExpectation::isExpectation.
@mal mal changed the title Allow use of ReferenceExpectation in expect calls Allow use of ReferenceExpectation in expect calls (-PHP4) Jul 27, 2015
jakoch added a commit that referenced this pull request Jul 27, 2015
Allow use of ReferenceExpectation in expect calls
@jakoch jakoch merged commit 597330b into simpletest:master Jul 27, 2015
@mal
Copy link
Contributor Author

mal commented Jul 27, 2015

Any speculation as to when this might find it's way into a composered release?

@jakoch
Copy link
Member

jakoch commented Jul 27, 2015

I would like to see the tests go green again, before i tag a new release.
Please try using "dev-master" meanwhile.

Some additional cleanups need to be done on the repository to get it in a better shape for distribution with Composer. At the moment a lot of unnecessary stuff (like the website) is distributed with each composer install.

@mal mal deleted the fix-reference-expectation-class branch July 27, 2015 17:03
@mal
Copy link
Contributor Author

mal commented Jul 27, 2015

Wow, I didn't appreciate that, but it makes sense. Thanks for being so responsive and keeping the project chugging along after all these years! 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants