[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