Skip to content

Commit

Permalink
Test that premissions also get set on already existing repofolders.
Browse files Browse the repository at this point in the history
  • Loading branch information
Niklaus Johner committed Jan 25, 2022
1 parent 907ed3d commit b012833
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions opengever/bundle/tests/test_os_migration.py
Original file line number Diff line number Diff line change
Expand Up @@ -1459,7 +1459,7 @@ def test_repository_migrator_move_updates_blocked_permissions_correctly(self):
['user:fa_inbox_users', 'user:kathi.barfuss'],
self.get_allowed_users(self.dossier))

def test_repository_migrator_create_sets_permissions_correctly(self):
def test_repository_migrator_sets_permissions_correctly(self):
self.login(self.manager)
migration_file = resource_filename('opengever.bundle.tests', 'assets/os_migration/os_test_create.xlsx')
analysis_file = resource_filename('opengever.bundle.tests', 'assets/os_migration/test_analysis.xlsx')
Expand Down Expand Up @@ -1522,9 +1522,11 @@ def test_repository_migrator_create_sets_permissions_correctly(self):

# Check that permissions were not changed on branch_repofolder
self.assertEqual(
False, getattr(self.branch_repofolder, '__ac_local_roles_block__', False))
True, getattr(self.branch_repofolder, '__ac_local_roles_block__', True))

self.assertDictEqual(
{'faivel.fruhling': ['DossierManager'],
{u'group1': ['Contributor', 'Reader'],
u'group2': ['Reader'],
'nicole.kohler': ['Owner']},
self.branch_repofolder.__ac_local_roles__)

Expand Down

0 comments on commit b012833

Please sign in to comment.