Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

PLAT-23_behat_tests added create user test #47

Merged
merged 13 commits into from
Mar 23, 2016

Conversation

wagg-matt
Copy link
Contributor

Fixes https://jira.comicrelief.com/browse/

Changes proposed in this pull request

  • change...
  • change...

Deployment instructions

  • No manual actions required for deployment

drush_driver: "drush"
drupal:
drupal_root: "/Users/mattw/sites/campaign-platform/campaign"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wagg-matt think this went in accidentally - we should probably gitignore behat.yml as it's user specific

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damnit, hang on.

@pvhee
Copy link
Contributor

pvhee commented Mar 17, 2016

@wagg-matt are you OK for us to merge this branch - we can reopen new tickets for new tests

@bimsonz @Saphyel rev & merge pls?

@api
Scenario: Create news-article
Given "article" content:
| title |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wagg-matt probably best to add some more fields as well, not just title?

@bimsonz
Copy link
Contributor

bimsonz commented Mar 17, 2016

Happy to merge although Peter's right, probably should trial a few more of the fields! That can be done when he gets some more time to work on this though right ?

@pvhee
Copy link
Contributor

pvhee commented Mar 17, 2016

Also this could be useful @wagg-matt: behat tests from agov, could draw some inspiration https://github.com/previousnext/agov/tree/8.x-1.x/tests/behat/features

@bimsonz
Copy link
Contributor

bimsonz commented Mar 21, 2016

@wagg-matt be good to get this one wrapped up and merged as well :)

@wagg-matt
Copy link
Contributor Author

@bimsonz yes, when I'm back in the office.

This will be on-going so will continually being opened and closed but didn't have anytime to do it last week.

@wagg-matt
Copy link
Contributor Author

@bimsonz once this passes Travis, can you please merge? I'll open a new PR with the rest of the work in afterwards

@bimsonz bimsonz merged commit 05c7acd into develop Mar 23, 2016
@bimsonz bimsonz deleted the feature/PLAT-23_behat_tests branch March 23, 2016 14:16
@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants