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

Tests update #6

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

roxblnfk
Copy link
Contributor

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:
tests

Just run testunit/index.php on php 5 or php 7

Not tested on php <5.3

@roxblnfk roxblnfk mentioned this pull request Jan 20, 2018
@roxblnfk
Copy link
Contributor Author

roxblnfk commented Jan 20, 2018

Support for console mode
image

@simonschaufi
Copy link
Contributor

simonschaufi commented Jan 20, 2018

Please rebase your branch on master to avoid any merge commits and then force push your changes

@roxblnfk
Copy link
Contributor Author

Please rebase your branch on master to avoid any merge commits and then force push your changes

Can i do it from web interface?

@simonschaufi
Copy link
Contributor

Can i do it from web interface?

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.

@roxblnfk
Copy link
Contributor Author

roxblnfk commented Feb 1, 2018

composer test is now available

@Skrol29
Copy link
Owner

Skrol29 commented Feb 13, 2018

I suggest to simply delete the « testunit » and « benches » from this repository.
Those are quite old tools now.
What do you think ?

@simonschaufi
Copy link
Contributor

would you suggest switchting to phpunit? how easy is it to move to phpunit?

@roxblnfk
Copy link
Contributor Author

The modern PHPUnit requires php >= 7.1

In this Pull Request i suggest only that you update SimpleTest without losing compatibility
For php <5.4 will be used old SimpleTest
For php >=5.4 will be used last SimpleTest

When the project will depend on php >=7 then it will be meaningful to rewrite the tests for PHPUnit

@roxblnfk roxblnfk force-pushed the feature/simpleTest-update branch 2 times, most recently from 4156a02 to 1ce7e6b Compare April 29, 2019 08:27
`composer test` available
Added new test case
@roxblnfk
Copy link
Contributor Author

> composer update
> composer test

image

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.

3 participants