Skip to content

Commit fae913d

Browse files
committed
Just a minor commit
1 parent 8b0e8e7 commit fae913d

File tree

2 files changed

+43
-5
lines changed

2 files changed

+43
-5
lines changed

lib/ldp-middleware.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const del = require('./handlers/delete')
1010
const patch = require('./handlers/patch')
1111
const index = require('./handlers/index')
1212
const copy = require('./handlers/copy')
13-
const validate = require('./handlers/validate')
13+
// const validate = require('./handlers/validate')
1414

1515
function LdpMiddleware (corsSettings) {
1616
const router = express.Router('/')
@@ -24,9 +24,9 @@ function LdpMiddleware (corsSettings) {
2424

2525
router.copy('/*', allow('Write'), copy)
2626
router.get('/*', index, allow('Read'), header.addPermissions, get)
27-
router.post('/*', allow('Append'), validate, post)
28-
router.patch('/*', allow('Append'), validate, patch)
29-
router.put('/*', allow('Write'), validate, put)
27+
router.post('/*', allow('Append'), post)
28+
router.patch('/*', allow('Append'), patch)
29+
router.put('/*', allow('Write'), put)
3030
router.delete('/*', allow('Write'), del)
3131

3232
return router

test/utils.js

Lines changed: 39 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,8 +88,46 @@ function createServer (options) {
8888
return ldnode.createServer(options)
8989
}
9090

91-
exports.setupSupertestServer = setupSuperServer
91+
exports.setupSuperServer = setupSuperServer
9292
function setupSuperServer (options) {
9393
const ldpServer = createServer(options)
9494
return supertest(ldpServer)
9595
}
96+
97+
/**
98+
* Creates a new test basic container via an LDP POST
99+
* (located in `test/resources/{containerName}`)
100+
* @method createTestContainer
101+
* @param server {supertest} An instance of a supertest server
102+
* @param containerName {String} Container name used as slug, no leading `/`
103+
* @return {Promise} Promise obj, for use with Mocha's `before()` etc
104+
*/
105+
exports.createTestContainer = function createTestContainer (server, containerName) {
106+
return new Promise(function (resolve, reject) {
107+
server.post('/')
108+
.set('content-type', 'text/turtle')
109+
.set('slug', containerName)
110+
.set('link', '<http://www.w3.org/ns/ldp#BasicContainer>; rel="type"')
111+
.set('content-type', 'text/turtle')
112+
.end(function (error, res) {
113+
error ? reject(error) : resolve(res)
114+
})
115+
})
116+
}
117+
118+
/**
119+
* Creates a new turtle test resource via an LDP PUT
120+
* (located in `test/resources/{resourceName}`)
121+
* @method createTestResource
122+
* @param resourceName {String} Resource name (should have a leading `/`)
123+
* @return {Promise} Promise obj, for use with Mocha's `before()` etc
124+
*/
125+
exports.createTestResource = function createTestResource (server, resourceName) {
126+
return new Promise(function (resolve, reject) {
127+
server.put(resourceName)
128+
.set('content-type', 'text/turtle')
129+
.end(function (error, res) {
130+
error ? reject(error) : resolve(res)
131+
})
132+
})
133+
}

0 commit comments

Comments
 (0)