[copr] bkabrda-workspace: Make the tests more adaptable to possible changes in fixtures (b174ef1)

Repository : http://git.fedorahosted.org/cgit/copr.git

On branch  : bkabrda-workspace

> ---------------------------------------------------------------
commit b174ef1a294dae1a9b97bbe1915cc4cad7740950
Author: Bohuslav Kabrda <bkabrda(a)redhat.com&gt;
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):