Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add assertResponseContains & assertResponseNotContains methods #48

Merged
merged 4 commits into from Aug 18, 2016

Conversation

goodtune
Copy link
Contributor

A couple of convenience methods to save some typing in test cases.

def test_response_contains(self):
    self.get('hello-world')
    self.assertResponseContains('<p>Hello, World!</p>')

def test_response_not_contains(self):
    self.get('hello-world')
    self.assertResponseNotContains('<p>Hello, Frank!</p>')

Uses the private _which_response method and defaults html to True.

@coveralls
Copy link

coveralls commented Aug 18, 2016

Coverage Status

Coverage decreased (-1.5%) to 95.565% when pulling d9a1ad3 on goodtune:assertResponseContains into b7bd744 on revsys:master.

@coveralls
Copy link

coveralls commented Aug 18, 2016

Coverage Status

Coverage increased (+0.07%) to 97.177% when pulling 5b50303 on goodtune:assertResponseContains into b7bd744 on revsys:master.

@coveralls
Copy link

coveralls commented Aug 18, 2016

Coverage Status

Coverage increased (+0.07%) to 97.177% when pulling ada6db8 on goodtune:assertResponseContains into b7bd744 on revsys:master.

@coveralls
Copy link

coveralls commented Aug 18, 2016

Coverage Status

Coverage increased (+0.07%) to 97.177% when pulling 0d32908 on goodtune:assertResponseContains into b7bd744 on revsys:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 97.177% when pulling 0d32908 on goodtune:assertResponseContains into b7bd744 on revsys:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 97.177% when pulling 0d32908 on goodtune:assertResponseContains into b7bd744 on revsys:master.

@frankwiles
Copy link
Member

Awesome thanks!

@frankwiles frankwiles merged commit 8afb21e into revsys:master Aug 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants