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.

Sign up for GitHub

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

Jump to bottom

Test enhancement #1

Merged

ankitjain28may merged 1 commit into ankitjain28may:master from peter279k:test_enhancement

Nov 30, 2018

Contributor

peter279k commented Nov 29, 2018

Changed log

Using DataProvider to collect test cases and split these from test method.
The assertSame arguments should be assertSame($expected, $actual).


          Test enhancement

aa62ce8

ankitjain28may approved these changes

View reviewed changes

Owner

ankitjain28may left a comment

Great :)

ankitjain28may merged commit d0cdbc5 into ankitjain28may:master

Owner

ankitjain28may commented Nov 30, 2018

Thanks for the contribution.

peter279k deleted the test_enhancement branch

November 30, 2018 09:53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment