Skip to content

Commit

Permalink
Feature/638 fix bruno test revoka all over limit (#159)
Browse files Browse the repository at this point in the history
* Bruno test revoke all instance delegation single instance
Fix and negative test for revoke 11 delegations

* Fix test name

* Revrote all test to have a descriptive name for each test step and use assert instead of axpect
Revrote all test testing multiple iterations of delegations to use for loop
Revrote al test to expect to to be in alist to not be vulnerable to difrent sorting from server and not using sorting on client as this would only fix it for one environment but could be difrent in other environment based on the uuid of that user.

* Fixed expected path according to code change as it is instanceId that will make the number of policies handled not ResourceId

* Fix expected after change in code

* Fix error in testdata
  • Loading branch information
lovoll authored Dec 16, 2024
1 parent 9d2a924 commit 8b4488a
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ tests {
assert.equal(data.instanceId, testdata.revokeallbeyondlimitinstanceid, 'InstanceId is incorect');

assert.equal(data.from.value, testdata.org1.partyuuid, 'FromId is inncorect');
assert.equal(data.to.value, testdata.org11.partyuuid, 'ToId is inncorect');
assert.equal(data.to.value, testdata.org10.partyuuid, 'ToId is inncorect');

assert.equal(data.rights[0].status, 'Delegated', 'read right has incorrect status');
assert.equal(data.rights[0].action.value, 'read', 'read right has incorrect action');
Expand Down

0 comments on commit 8b4488a

Please sign in to comment.