Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : bkabrda-workspace
commit 2e0ada3e4b7efb072487c4db4a4d4718ddd65318 Author: Bohuslav Kabrda bkabrda@redhat.com Date: Mon Nov 26 12:34:13 2012 +0100
Refresh test fixtures, fix one logics function
wsgi/coprs/logic/coprs_logic.py | 3 ++- wsgi/tests/coprs_test_case.py | 7 ++++--- 2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/wsgi/coprs/logic/coprs_logic.py b/wsgi/coprs/logic/coprs_logic.py index 1d57372..98371e0 100644 --- a/wsgi/coprs/logic/coprs_logic.py +++ b/wsgi/coprs/logic/coprs_logic.py @@ -1,5 +1,6 @@ from coprs import db from coprs import exceptions +from coprs import helpers from coprs import models
class CoprsLogic(object): @@ -34,7 +35,7 @@ class CoprsLogic(object): elif user_relation == 'allowed': aliased_user = db.aliased(models.User) query = query.join(models.CoprPermission, models.Copr.copr_permissions).\ - filter(models.CoprPermission.copr_builder == True).\ + filter(models.CoprPermission.copr_builder == helpers.PermissionEnum.num('Approved')).\ join(aliased_user, models.CoprPermission.user).\ filter(aliased_user.openid_name == models.User.openidize_name(username)) return query diff --git a/wsgi/tests/coprs_test_case.py b/wsgi/tests/coprs_test_case.py index 8aad8d8..1ddd094 100644 --- a/wsgi/tests/coprs_test_case.py +++ b/wsgi/tests/coprs_test_case.py @@ -5,6 +5,7 @@ import pytest
import coprs
+from coprs import helpers from coprs import models
class CoprsTestCase(object): @@ -56,9 +57,9 @@ class CoprsTestCase(object):
@pytest.fixture def f_copr_permissions(self): - self.cp1 = models.CoprPermission(copr = self.c2, user = self.u1, copr_builder = True, copr_admin = False) - self.cp2 = models.CoprPermission(copr = self.c3, user = self.u3, copr_builder = False, copr_admin = False) - self.cp3 = models.CoprPermission(copr = self.c3, user = self.u1, copr_builder = False, copr_admin = True) + self.cp1 = models.CoprPermission(copr = self.c2, user = self.u1, copr_builder = helpers.PermissionEnum.num('Approved'), copr_admin = helpers.PermissionEnum.num('No Action')) + self.cp2 = models.CoprPermission(copr = self.c3, user = self.u3, copr_builder = helpers.PermissionEnum.num('No Action'), copr_admin = helpers.PermissionEnum.num('No Action')) + self.cp3 = models.CoprPermission(copr = self.c3, user = self.u1, copr_builder = helpers.PermissionEnum.num('Requested'), copr_admin = helpers.PermissionEnum.num('Approved'))
self.db.session.add_all([self.cp1, self.cp2, self.cp3]) self.db.session.commit()
copr-devel@lists.fedorahosted.org