[copr] bkabrda-workspace: Make the tests more adaptable to possible changes in fixtures (b174ef1)
bkabrda at fedorahosted.org
bkabrda at fedorahosted.org
Wed Nov 21 08:37:21 UTC 2012
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : bkabrda-workspace
>---------------------------------------------------------------
commit b174ef1a294dae1a9b97bbe1915cc4cad7740950
Author: Bohuslav Kabrda <bkabrda at redhat.com>
Date: Wed Nov 21 09:37:05 2012 +0100
Make the tests more adaptable to possible changes in fixtures
>---------------------------------------------------------------
.../tests/test_views/test_coprs_ns/test_general.py | 14 ++++++++------
1 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/wsgi/tests/test_views/test_coprs_ns/test_general.py b/wsgi/tests/test_views/test_coprs_ns/test_general.py
index f995b7c..55cfe3e 100644
--- a/wsgi/tests/test_views/test_coprs_ns/test_general.py
+++ b/wsgi/tests/test_views/test_coprs_ns/test_general.py
@@ -63,27 +63,29 @@ class TestCoprNew(CoprsTestCase):
assert "New entry was successfully posted" in r.data
def test_copr_new_exists_for_another_user(self, f_users, f_coprs):
+ name = self.c1.name
with self.tc as c:
with c.session_transaction() as s:
s['openid'] = self.u3.openid_name
- foocoprs = len(self.models.Copr.query.filter(self.models.Copr.name == 'foocopr').all())
+ foocoprs = len(self.models.Copr.query.filter(self.models.Copr.name == name).all())
assert foocoprs > 0
- r = c.post('/coprs/new/', data = {'name': 'foocopr', 'release': 'fedora-rawhide', 'arches': ['i386']}, follow_redirects = True)
- assert len(self.models.Copr.query.filter(self.models.Copr.name == 'foocopr').all()) == foocoprs + 1
+ r = c.post('/coprs/new/', data = {'name': name, 'release': 'fedora-rawhide', 'arches': ['i386']}, follow_redirects = True)
+ assert len(self.models.Copr.query.filter(self.models.Copr.name == name).all()) == foocoprs + 1
assert "New entry was successfully posted" in r.data
def test_copr_new_exists_for_this_user(self, f_users, f_coprs):
+ name = self.c1.name
with self.tc as c:
with c.session_transaction() as s:
s['openid'] = self.u1.openid_name
- foocoprs = len(self.models.Copr.query.filter(self.models.Copr.name == 'foocopr').all())
+ foocoprs = len(self.models.Copr.query.filter(self.models.Copr.name == name).all())
assert foocoprs > 0
- r = c.post('/coprs/new/', data = {'name': 'foocopr', 'release': 'fedora-rawhide', 'arches': ['i386']}, follow_redirects = True)
- assert len(self.models.Copr.query.filter(self.models.Copr.name == 'foocopr').all()) == foocoprs
+ r = c.post('/coprs/new/', data = {'name': name, 'release': 'fedora-rawhide', 'arches': ['i386']}, follow_redirects = True)
+ assert len(self.models.Copr.query.filter(self.models.Copr.name == name).all()) == foocoprs
assert "You already have copr named" in r.data
class TestCoprDetail(CoprsTestCase):
More information about the copr-devel
mailing list