Fix test for submit sms form. #146

wants to merge 1 commit into

4 participants


This fixes an AttributeError in test for submitting sms form. The test mobile number is not a real number belonging to any person and hence it's safe to be used in the test.


I don't see an error on our Grid setup, likely because we're passing in our own YAML file. @bobsilverberg, @retornam thoughts?


Agreed. The test is likely not passing for you, @rtnpro, because you are not providing it with a path to a valid credentials file on the command line. I have just added one to the repo, so if you pull the latest version from master you will have a credentials.yaml file in the root of the project. Add the option "--credentials=credentials.yaml" (without the quotes) to your command line and this test failure should go away.


As per @bobsilverberg last comment I'll close this pull.
@rtnpro let us know if it works for you now!

@AlinT AlinT closed this Nov 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment