From ed4e93b475eb0d0d7d62206a7392379d870bb4dd Mon Sep 17 00:00:00 2001
From: Erin Mullaney <erin.mullaney@torchbox.com>
Date: Thu, 21 Feb 2019 12:35:07 -0500
Subject: [PATCH] #962 simplify test because we already cover the get 403 for
users in another test for non-staff users
---
opentech/apply/review/tests/test_views.py | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/opentech/apply/review/tests/test_views.py b/opentech/apply/review/tests/test_views.py
index 9cf733eb4..8d0c1fe20 100644
--- a/opentech/apply/review/tests/test_views.py
+++ b/opentech/apply/review/tests/test_views.py
@@ -282,10 +282,8 @@ class NonStaffReviewOpinionCase(BaseViewTestCase):
def get_kwargs(self, instance):
return {'pk': instance.id, 'submission_pk': instance.submission.id}
- def test_cant_see_review_or_post_opinion_to_it(self):
+ def nonstaff_cant_post_opinion_to_review(self):
staff = StaffFactory()
review = ReviewFactory(submission=self.submission, author=staff, recommendation_yes=True)
- response = self.get_page(review)
- self.assertEqual(response.status_code, 403)
response = self.post_page(review, {'agree': AGREE})
self.assertEqual(response.status_code, 403)
--
GitLab