fixed todo and added .meta test in patch-test

nodeSolidServer · Mar 25, 2024 · e4bb2e9 · e4bb2e9
1 parent d56cdf0
commit e4bb2e9
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 8 deletions.
diff --git a/test/integration/http-test.js b/test/integration/http-test.js
@@ -891,13 +891,6 @@ describe('HTTP APIs', function () {
         .set('content-type', 'text/turtle')
         .expect(403, done)
     })
-    it('should not error with 400 if slug contains invalid suffix', function (done) { // TODO find better name
-      server.post('/post-tests/')
-        .set('slug', 'put-resource.acl.ttl')
-        .send(postRequest1Body)
-        .set('content-type', 'text-turtle')
-        .expect(201, done)
-    })
     it('should error with 400 if the body is empty and no content type is provided', function (done) {
       server.post('/post-tests/')
         .set('slug', 'post-resource-empty-fail')
@@ -921,6 +914,15 @@ describe('HTTP APIs', function () {
           .expect(hasHeader('acl', suffixAcl))
           .expect(201, done)
       })
+    it('should create new resource even if slug contains invalid suffix', function (done) {
+      server.post('/post-tests/')
+        .set('slug', 'put-resource.acl.ttl')
+        .send(postRequest1Body)
+        .set('content-type', 'text-turtle')
+        .expect(hasHeader('describedBy', suffixMeta))
+        .expect(hasHeader('acl', suffixAcl))
+        .expect(201, done)
+    })
     it('should fail return 404 if no parent container found', function (done) {
       server.post('/hello.html/')
         .send(postRequest1Body)

diff --git a/test/integration/patch-test.js b/test/integration/patch-test.js
@@ -131,7 +131,7 @@ describe('PATCH through text/n3', () => {
       result: '@prefix : </new.n3#>.\n@prefix tim: </>.\n\ntim:x tim:y tim:z.\n\n'
     }))
 
-    describe('on an N3 file that has an invalid uri', describePatch({
+    describe('on an N3 file that has an invalid uri (*.acl)', describePatch({
       path: '/foo/bar.acl/test.n3',
       exists: false,
       patch: `<> a solid:InsertDeletePatch;
@@ -141,6 +141,16 @@ describe('PATCH through text/n3', () => {
       text: 'contained reserved suffixes in path'
     }))
 
+    describe('on an N3 file that has an invalid uri (*.meta)', describePatch({
+      path: '/foo/bar/xyz.meta/test.n3',
+      exists: false,
+      patch: `<> a solid:InsertDeletePatch;
+                 solid:insers { <x> <y> <z>. }.`
+    }, {
+      status: 400,
+      text: 'contained reserved suffixes in path'
+    }))
+
     describe('on a resource with read-only access', describePatch({
       path: '/read-only.ttl',
       patch: `<> a solid:InsertDeletePatch;