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

Extended bakery test to add Test Scope #919

Closed
wants to merge 6 commits into from
Closed

Extended bakery test to add Test Scope #919

wants to merge 6 commits into from

Conversation

ssnukala
Copy link
Contributor

extended this to have the second argument to be either just the sprinkle name or a full qualified path to the test case and method.
usage php bakery test CRUD,
php bakery test 'CRUD\Tests\Unit\',
php bakery test 'CRUD\Tests\Unit\CRUDTest',
php bakery test 'CRUD\Tests\Unit\CRUDTest::testCRUDMethod'

Added optional Sprinkle name to the usage php bakery test <Sprinkle>
Added fully qualified test scope as an argument for executing tests using bakery test
Fixed user factory and updated the namespace for Account Unit Tests.
@lcharette lcharette self-requested a review December 27, 2018 18:19
@lcharette lcharette self-assigned this Dec 27, 2018
@lcharette lcharette added this to the 4.2.0 milestone Dec 27, 2018
Added Exception block to catch HTTPExceptions. expectException is causing failure so commented that out. Need to troubleshoot this and will add that back later.
@lcharette
Copy link
Member

I manually included your changes in 9c7897b, as this PR had conflicts. Open a new PR for the Docker fix if it's still required.

@lcharette lcharette closed this Jan 10, 2019
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.

2 participants