Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

[6.0] Add acceptance tests with Behat #637

Open
SammyK opened this issue Aug 19, 2016 · 2 comments
Open

[6.0] Add acceptance tests with Behat #637

SammyK opened this issue Aug 19, 2016 · 2 comments
Milestone

Comments

@SammyK
Copy link
Contributor

SammyK commented Aug 19, 2016

What do you guys think about adding some acceptance tests with Behat? You think it'd be useful? Then we could remove the live calls to Graph from the unit tests. :)

@yguedidi
Copy link
Contributor

Tests are always great! :)
But we should not test the Graph API, only the SDK and its usage

@SammyK
Copy link
Contributor Author

SammyK commented Aug 23, 2016

My thoughts were that Behat wouldn't be testing the Graph API, rather the SDK's interaction with the Graph API ensuring that the expected responses were being processed properly. Since the Graph API changes often (and it's not always documented!), acceptance tests would be a good automated way to ensure that the functionality in the SDK isn't broken. What do you think? :)

@yguedidi yguedidi added this to the 6.0.0 milestone Jun 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants