Skip to content

Commit

Permalink
Fix repository_tool test to expect promiscuous delegation to work
Browse files Browse the repository at this point in the history
A and B both delegating to C works and is OK.

We should probably add more testing for this, covering more
complex situations.

Signed-off-by: Sebastien Awwad <sebastien.awwad@gmail.com>
  • Loading branch information
awwad committed Jan 23, 2018
1 parent 0e0ae80 commit 628b938
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions tests/test_repository_tool.py
Original file line number Diff line number Diff line change
Expand Up @@ -1189,9 +1189,8 @@ def test_delegate(self):
self.assertEqual(self.targets_object.get_delegated_rolenames(),
['tuf'])

# Try to delegate to a role that has already been delegated.
self.assertRaises(securesystemslib.exceptions.Error,
self.targets_object.delegate, rolename, public_keys, paths, threshold,
# Try to delegate to a role that has already been delegated. Expect success.
self.targets_object.delegate(rolename, public_keys, paths, threshold,
terminating=False, list_of_targets=list_of_targets,
path_hash_prefixes=path_hash_prefixes)

Expand Down

0 comments on commit 628b938

Please sign in to comment.