test: change to container paths to end in / on POST only

CommunitySolidServer · Feb 21, 2022 · 07b8e3c · 07b8e3c
1 parent 472caea
commit 07b8e3c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/test/integration/LdpHandlerWithoutAuth.test.ts b/test/integration/LdpHandlerWithoutAuth.test.ts
@@ -265,14 +265,14 @@ describe.each(stores)('An LDP handler allowing all requests %s', (name, { storeC
   });
 
   it('can create a container with a diamond identifier in the data.', async(): Promise<void> => {
-    const slug = 'my-container';
+    const slug = 'my-container/';
 
     const body = '<> <http://www.w3.org/2000/01/rdf-schema#label> "My Container" .';
     let response = await postResource(baseUrl, { isContainer: true, contentType: 'text/turtle', slug, body });
-    expect(response.headers.get('location')).toBe(`${baseUrl}${slug}/`);
+    expect(response.headers.get('location')).toBe(`${baseUrl}${slug}`);
 
     // GET
-    const containerUrl = `${baseUrl}${slug}/`;
+    const containerUrl = `${baseUrl}${slug}`;
     response = await getResource(containerUrl);
 
     await expectQuads(response, [

diff --git a/test/integration/ServerFetch.test.ts b/test/integration/ServerFetch.test.ts
@@ -75,7 +75,6 @@ describe('A Solid server', (): void => {
       method: 'PUT',
       headers: {
         'content-type': 'text/turtle',
-        'link': `<${LDP.Container}>; rel="type"`,
       },
       body: '<a:b> <a:b> <a:b>.',
     });
@@ -141,7 +140,6 @@ describe('A Solid server', (): void => {
       method: 'PUT',
       headers: {
         'content-type': 'text/turtle',
-        'link': `<${LDP.Container}>; rel="type"`,
       },
       body: '<a:b> <a:b> <a:b>.',
     });
@@ -187,7 +185,6 @@ describe('A Solid server', (): void => {
       method: 'PUT',
       headers: {
         'content-type': 'text/turtle',
-        'link': `<${LDP.Container}>; rel="type"`,
       },
       body: '<a:b> <a:b> <a:b>.',
     });

diff --git a/test/unit/storage/DataAccessorBasedStore.test.ts b/test/unit/storage/DataAccessorBasedStore.test.ts
@@ -419,6 +419,7 @@ describe('A DataAccessorBasedStore', (): void => {
     it('will error if the ending slash does not match its resource type.', async(): Promise<void> => {
       const resourceID = { path: `${root}resource` };
       representation.metadata.add(RDF.type, LDP.terms.Container);
+      representation.metadata.add(SOLID_HTTP.slug, `${root}resource/`);
       await expect(store.setRepresentation(resourceID, representation)).rejects.toThrow(
         new BadRequestHttpError('Containers should have a `/` at the end of their path, resources should not.'),
       );