update tests reorganisation

solid-contrib / web-access-control-tests

Tests if a Solid server implements web access control correctly

MIT License

2 stars 5 forks source link

update tests reorganisation #18

Closed bourgeoa closed 3 years ago

bourgeoa commented 3 years ago

Due to failing recursive delete it was needed to :

add intermediate containers at describe level
reorder the tests to have success tests before failing tests

NSS now passes all update tests with 3 changes to NSS :

revise DELETE in middleware
PATCH in handlers acl:Write for DELETE with sparl inlieu of acl:Read and acl:Write
adding clearAclCache[url] in ldp.js (delete and put)

bourgeoa commented 3 years ago

I left it as a work in progress, because I found that this PR was a fake solution. The reoreding of the tests was masking the fact that some acls were wrong. see PR #19