heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

pushing so paroneayea can help me resolve a conflict within the tests.

Browse Source
This commit is contained in:
tilly-Q 2013-08-23 16:14:36 -04:00
parent 2c901db023
commit e46fb71c1d
2 changed files with 16 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
mediagoblin/tests/test_modelmethods.py
View File

@ -153,16 +153,14 @@ class TestMediaEntrySlugs(object):
class TestUserHasPrivilege: class TestUserHasPrivilege:
def _setup(self): def _setup(self):
self.natalie_user = fixture_add_user(u'natalie') fixture_add_user(u'natalie',
self.aeva_user = fixture_add_user(u'aeva') privileges=[u'admin',u'moderator',u'active'])
self.natalie_user.all_privileges += [ fixture_add_user(u'aeva',
Privilege.query.filter( privileges=[u'moderator',u'active'])
Privilege.privilege_name == u'admin').one(), self.natalie_user = User.query.filter(
Privilege.query.filter( User.username==u'natalie').first()
Privilege.privilege_name == u'moderator').one()] self.aeva_user = User.query.filter(
self.aeva_user.all_privileges += [ User.username==u'aeva').first()
Privilege.query.filter(
Privilege.privilege_name == u'moderator').one()]
def test_privilege_added_correctly(self, test_app): def test_privilege_added_correctly(self, test_app):
self._setup() self._setup()
@ -177,8 +175,8 @@ class TestUserHasPrivilege:
self._setup() self._setup()
# then test out the user.has_privilege method for one privilege # then test out the user.has_privilege method for one privilege
assert not natalie_user.has_privilege(u'commenter') assert not self.natalie_user.has_privilege(u'commenter')
assert aeva_user.has_privilege(u'active') assert self.aeva_user.has_privilege(u'active')
def test_user_has_privileges_multiple(self, test_app): def test_user_has_privileges_multiple(self, test_app):
@ -186,9 +184,9 @@ class TestUserHasPrivilege:
# when multiple args are passed to has_privilege, the method returns # when multiple args are passed to has_privilege, the method returns
# True if the user has ANY of the privileges # True if the user has ANY of the privileges
assert natalie_user.has_privilege(u'admin',u'commenter') assert self.natalie_user.has_privilege(u'admin',u'commenter')
assert aeva_user.has_privilege(u'moderator',u'active') assert self.aeva_user.has_privilege(u'moderator',u'active')
assert not natalie_user.has_privilege(u'commenter',u'uploader') assert not self.natalie_user.has_privilege(u'commenter',u'uploader')

6
mediagoblin/tests/test_submission.py
View File

@ -24,7 +24,7 @@ import pytest
from mediagoblin.tests.tools import fixture_add_user from mediagoblin.tests.tools import fixture_add_user
from mediagoblin import mg_globals from mediagoblin import mg_globals
from mediagoblin.db.models import MediaEntry from mediagoblin.db.models import MediaEntry, User
from mediagoblin.tools import template from mediagoblin.tools import template
from mediagoblin.media_types.image import ImageMediaManager from mediagoblin.media_types.image import ImageMediaManager
from mediagoblin.media_types.pdf.processing import check_prerequisites as pdf_check_prerequisites from mediagoblin.media_types.pdf.processing import check_prerequisites as pdf_check_prerequisites
@ -46,9 +46,9 @@ class TestSubmission:
# TODO: Possibly abstract into a decorator like: # TODO: Possibly abstract into a decorator like:
# @as_authenticated_user('chris') # @as_authenticated_user('chris')
test_user = fixture_add_user(privileges=[u'active',u'uploader']) fixture_add_user(privileges=[u'active',u'uploader'])
self.test_user = test_user self.test_user = User.query.filter(User.username==u'chris').first()
self.login() self.login()