JavaScriptSolidServer
diff --git a/‎src/utils/keyHelpers/acl.ts‎
Lines changed: 5 additions & 9 deletions b/‎src/utils/keyHelpers/acl.ts‎
Lines changed: 5 additions & 9 deletions
diff --git a/‎test/unit/utils/keyHelpers/acl.test.ts‎
Lines changed: 23 additions & 4 deletions b/‎test/unit/utils/keyHelpers/acl.test.ts‎
Lines changed: 23 additions & 4 deletions
@@ -19,19 +19,15 @@ export async function setAcl (keyDoc: string, aclBody: string) {
     throw new Error('Key ACL doc not found!')
   }
 
-  // delete READ only keyAclDoc. This is possible if the webId is an owner
   try {
-    const response = await store.fetcher.webOperation('DELETE', keyAclDoc.value) // this may fail if webId is not an owner
-    debug.log('delete ' + keyAclDoc.value + ' ' + response.status) // should test 404 and 2xx
+    await store.fetcher.webOperation('PUT', keyAclDoc.value, {
+      data: aclBody,
+      contentType: 'text/turtle'
+    })
   } catch (err) {
-    if (err.response.status !== 404) { throw new Error(err) }
+    if (err?.response?.status !== 404) { throw new Error(err) }
     debug.log('delete ' + keyAclDoc.value + ' ' + err.response.status) // should test 404 and 2xx
   }
-
-  const aclResponse = await store.fetcher.webOperation('PUT', keyAclDoc.value, {
-    data: aclBody,
-    contentType: 'text/turtle'
-  })
 }
 
 /**
 
@@ -3,17 +3,36 @@ import { setAcl, keyAclBody } from '../../../../src/utils/keyHelpers/acl'
 
 store.fetcher.load = jest.fn().mockImplementation(() => {})
 store.any = jest.fn()
+store.fetcher.webOperation = jest.fn()
+
+const keyDoc = 'https://alice.solidcommunity.net/profile/keys/publicKey.ttl'
+const keyAclDoc = 'https://alice.solidcommunity.net/profile/keys/publicKey.ttl'
+const error404 = {
+  response: {
+    status: 404
+  }
+}
 
 describe('ACL Helpers', () => {
   describe('setAcl', () => {
     it('throws an error if an ACL document is NOT found', async () => {
       /* @ts-ignore */
-      store.any.mockReturnValue('')
-      const keyDoc = 'https://alice.solidcommunity.net/profile/keys/publicKey.ttl'
+      store.any.mockReturnValueOnce('')
       await expect(setAcl(keyDoc, keyAclBody(keyDoc, ''))).rejects.toThrowError('Key ACL doc not found!')
     })
-    it.skip('want to find out why we delete and then re-add', () => {
-
+    it('throws error if acl can not be written', async () => {
+      /* @ts-ignore */
+      store.any.mockReturnValueOnce(keyAclDoc)
+      /* @ts-ignore */
+      store.fetcher.webOperation.mockRejectedValueOnce()
+      await expect(setAcl(keyDoc, keyAclBody(keyDoc, ''))).rejects.toThrowError('')
+    })
+    it('continues if error is because acl doc can not be found.', async () => {
+      /* @ts-ignore */
+      store.any.mockReturnValueOnce(keyAclDoc)
+      /* @ts-ignore */
+      store.fetcher.webOperation.mockRejectedValueOnce(error404)
+      await expect(setAcl(keyDoc, keyAclBody(keyDoc, ''))).resolves.toBeUndefined
     })
   })
 })