Also add extension on POSTed Turtle files.

Author: RubenVerborgh

lib/ldp.js

@@ -17,7 +17,6 @@ var ldpContainer = require('./ldp-container')
 var parse = require('./utils').parse
 
 const DEFAULT_CONTENT_TYPE = 'text/turtle'
-const DEFAULT_EXTENSION = '.ttl'
 
 const RDF_MIME_TYPES = [
   'text/turtle',            // .ttl
@@ -196,8 +195,10 @@ class LDP {
         return
       }
     }
-    // No need to use the default extension, as we know its MIME type already
-    if (extension === DEFAULT_EXTENSION) extension = ''
+    // Containers should not receive an extension
+    if (container) {
+      extension = ''
+    }
     // TODO: possibly package this in ldp.post
     ldp.getAvailablePath(host, containerPath, { slug, extension }).then(resourcePath => {
       debug.handlers('POST -- Will create at: ' + resourcePath)

test/integration/http-test.js

@@ -510,7 +510,7 @@ describe('HTTP APIs', function () {
       server.post('/post-tests/')
         .send(postRequest1Body)
         .set('content-type', 'text/turtle')
-        .set('slug', 'post-resource-1.ttl')
+        .set('slug', 'post-resource-1')
         .expect('location', /\/post-resource-1/)
         .expect(hasHeader('describedBy', suffixMeta))
         .expect(hasHeader('acl', suffixAcl))
@@ -521,7 +521,7 @@ describe('HTTP APIs', function () {
         server.post('')
           .send(postRequest1Body)
           .set('content-type', 'text/turtle')
-          .set('slug', 'post-test-target.ttl')
+          .set('slug', 'post-test-target')
           .expect('location', /\/post-test-target\.ttl/)
           .expect(hasHeader('describedBy', suffixMeta))
           .expect(hasHeader('acl', suffixAcl))
@@ -531,15 +531,15 @@ describe('HTTP APIs', function () {
       server.post('/hello.html/')
         .send(postRequest1Body)
         .set('content-type', 'text/turtle')
-        .set('slug', 'post-test-target2.ttl')
+        .set('slug', 'post-test-target2')
         .expect(404, done)
     })
     it('should create a new slug if there is a resource with the same name',
       function (done) {
         server.post('/post-tests/')
           .send(postRequest1Body)
           .set('content-type', 'text/turtle')
-          .set('slug', 'post-resource-1.ttl')
+          .set('slug', 'post-resource-1')
           .expect(201, done)
       })
     it('should be able to delete newly created resource', function (done) {
@@ -624,9 +624,9 @@ describe('HTTP APIs', function () {
                 .then(res => { response = res })
         )
 
-        it('is assigned an extensionless URL', () => {
+        it('is assigned an URL with the .ttl extension', () => {
           expect(response.headers).to.have.property('location')
-          expect(response.headers.location).to.match(/^\/post-tests\/[^./]+$/)
+          expect(response.headers.location).to.match(/^\/post-tests\/[^./]+\.ttl$/)
         })
       })
 
@@ -639,8 +639,8 @@ describe('HTTP APIs', function () {
                 .then(res => { response = res })
         )
 
-        it('is assigned an extensionless URL', () => {
-          expect(response.headers).to.have.property('location', '/post-tests/slug1')
+        it('is assigned an URL with the .ttl extension', () => {
+          expect(response.headers).to.have.property('location', '/post-tests/slug1.ttl')
         })
       })