Skip to content

Initial commit of behat test setup.#1

Merged
dpslwk merged 3 commits intomasterfrom
behatmeup
Jun 3, 2016
Merged

Initial commit of behat test setup.#1
dpslwk merged 3 commits intomasterfrom
behatmeup

Conversation

@cooperaj
Copy link
Contributor

Behat is configured to use the laravel-behat extension and my slightly different (but IMO useful) way of laying out the test folder structure has been done.

Essentially each context file is mostly empty and all test definitions are broken out into related functionality traits.

I've also taken the liberty of binding the app/HMS folder to the HMS\ namespace (since I have some behat code that lives there)

cooperaj added 2 commits May 23, 2016 13:37
Behat is configured to use the laravel-behat extension and my slightly different (but IMO useful) way of laying out the test folder structure has been done.
… contents of the file but we *do* care that it exists hence it has been committed with the prior commit.
@cooperaj
Copy link
Contributor Author

Worth pointing out that I added a dummy test to the setup so that it's possible to see vaguely how things work. Once actual tests exist then it'll be deleted I imagine.

@geeksareforlife
Copy link
Member

I do not understand most of this, but the bits I do don't seem to conflict with the main system. I'm happy to merge, but would appreciate a second opinion

@cooperaj
Copy link
Contributor Author

cooperaj commented Jun 1, 2016

Seems this covers (at least paritally) issue #12 ?

I'm happy to go through what I've done if you want to point out sticking areas. For the most part this is just integration of the standard behat stuff with the extra laracasts laravel components bolted in.

The only thing thats different from normal IIRC is the way I've organised the test files (which I've done like this to keep things IMO a little easier to parse).

@cooperaj cooperaj added environment dev (Vagrant) / staging / production related testing labels Jun 1, 2016
@cooperaj cooperaj added this to the Base System milestone Jun 1, 2016
@dpslwk
Copy link
Member

dpslwk commented Jun 1, 2016

all looks good to me and covers whats needed to start with

@dpslwk dpslwk merged commit 7caaca5 into master Jun 3, 2016
@dpslwk dpslwk deleted the behatmeup branch June 3, 2016 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

environment dev (Vagrant) / staging / production related testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants