Travis-ci integration? about facebook-php-business-sdk

[paulbain]

We have considered it but as you highlighted, the problem is having to use a real ad account and app.
It's something we definitely want to do so we'll work something out.

[paulbain]

@pruno, any thoughts here?

[pruno]

Hi @bjeavons,

1 - unfortunately is not feasible for security reasons
2 - the majority of tests are integration tests, this is a design choice made to grant an always up to date SDK. I feel that considering the number of unit tests, running them as part of CI won't have relevance in ensuring the stability of the SDK

[bjeavons]

I hear you that it technically doesn't make a lot of sense or provides a lot of value at the moment. However, from my perspective, having transparency into the testing process and results gives me (and others) better assurance and greater confidence in the code and dev changes. For example, several phpunit tests for 2.2.0 are failing for me (I've got to investigate further because in some cases running a test individually shows it passing). But, if I could see a travis-ci badge that says master or tag 2.2.0 are "OK" then I feel better about updating my app and testing it out.

[brucehazan]

Thanks @bjeavons - you have a valid point, I have asked the team to look into a solution for running tests continuously for the reasons you mention. It might not be with Travis but we'll figure something out. Thanks for your support.

[bjeavons]

Nice! 9a023d9

[pruno]

;)