-
Notifications
You must be signed in to change notification settings - Fork 18
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 update #6
base: master
Are you sure you want to change the base?
Tests update #6
Conversation
Please rebase your branch on master to avoid any merge commits and then force push your changes |
Can i do it from web interface? |
81396d2
to
7989a1a
Compare
unfortunatelly not. If you collaborate with others on one pull request you should avoid that as it rewrites your commits. Do that only if you are the only one working on that pull request. Thats why github doesn't support that on the platform. |
7989a1a
to
250a9ff
Compare
c1f3332
to
f1d532d
Compare
|
I suggest to simply delete the « testunit » and « benches » from this repository. |
would you suggest switchting to phpunit? how easy is it to move to phpunit? |
The modern PHPUnit requires php >= 7.1 In this Pull Request i suggest only that you update SimpleTest without losing compatibility When the project will depend on php >=7 then it will be meaningful to rewrite the tests for PHPUnit |
4156a02
to
1ce7e6b
Compare
`composer test` available Added new test case
simpleTest 1.1 added into
composer.json
Old simpleTest was not deleted (for compatibility tests with php <5.4)
The results of running tests without plugins on different php versions below:
Just run
testunit/index.php
on php 5 or php 7Not tested on php <5.3