diff --git a/bin/clean_test.sh b/bin/clean_test.sh index ac885d9ec12d1ae0baa0c8ee314cfd32e029b903..ef146e09160cb138cd066cfca7a9d54be05ef29c 100755 --- a/bin/clean_test.sh +++ b/bin/clean_test.sh @@ -4,5 +4,5 @@ basePath=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd ) for i in $(seq 1 6); do printf "use peertube-test%s;\ndb.dropDatabase();" "$i" | mongo - rm -rf "$basePath/../server/test$i" + rm -rf "$basePath/../test$i" done diff --git a/package.json b/package.json index d7e29e0c171b8c95cb894e9e65fbf50a114f65af..cebd5b506f1359ef4f8318b122b63fc10e50dcb3 100644 --- a/package.json +++ b/package.json @@ -34,7 +34,7 @@ "dev": "concurrently \"npm run livereload\" \"npm run client:tsc:watch\" \"npm run client:sass:watch\" \"npm start\"", "livereload": "livereload ./client", "start": "node server", - "test": "standard && mocha tests" + "test": "standard && mocha server/tests" }, "dependencies": { "async": "^1.2.1", @@ -73,6 +73,9 @@ "supertest": "^1.1.0" }, "standard": { + "ignore": [ + "client/" + ], "globals": [ "alert", "confirm", diff --git a/server/controllers/api/v1/index.js b/server/controllers/api/v1/index.js index 07a68ed9def9a343467e6e45ca964ab267b485eb..abbc5eb60420235c66a392d583ec8af4904be89f 100644 --- a/server/controllers/api/v1/index.js +++ b/server/controllers/api/v1/index.js @@ -11,7 +11,14 @@ var videosController = require('./videos') router.use('/pods', podsController) router.use('/remotevideos', remoteVideosController) router.use('/videos', videosController) +router.use('/*', badRequest) // --------------------------------------------------------------------------- module.exports = router + +// --------------------------------------------------------------------------- + +function badRequest (req, res, next) { + res.sendStatus(400) +} diff --git a/server/helpers/logger.js b/server/helpers/logger.js index 67f69a87512513b96be8b44d05f914fa737e53eb..1c15a0edd981a85fd1126ca2be5befd44f52029e 100644 --- a/server/helpers/logger.js +++ b/server/helpers/logger.js @@ -6,7 +6,7 @@ var path = require('path') var winston = require('winston') winston.emitErrs = true -var logDir = path.join(__dirname, '..', config.get('storage.logs')) +var logDir = path.join(__dirname, '..', '..', config.get('storage.logs')) var logger = new winston.Logger({ transports: [ new winston.transports.File({ diff --git a/server/helpers/peertubeCrypto.js b/server/helpers/peertubeCrypto.js index 29b9d79c993caa965210266119672723713953f0..7e65c43e12f7de8c080105ea43e0103ba9ae3b5d 100644 --- a/server/helpers/peertubeCrypto.js +++ b/server/helpers/peertubeCrypto.js @@ -9,7 +9,7 @@ var ursa = require('ursa') var logger = require('./logger') -var certDir = path.join(__dirname, '..', config.get('storage.certs')) +var certDir = path.join(__dirname, '..', '..', config.get('storage.certs')) var algorithm = 'aes-256-ctr' var peertubeCrypto = { diff --git a/server/initializers/checker.js b/server/initializers/checker.js index ec7bc0ad2ff14d32d1b55bab385f5a704b01436d..44d9ea85b8abac6dbc7fb9ceb4e15fa59108af5a 100644 --- a/server/initializers/checker.js +++ b/server/initializers/checker.js @@ -34,7 +34,7 @@ function createDirectoriesIfNotExist () { for (var key of Object.keys(storages)) { var dir = storages[key] try { - mkdirp.sync(path.join(__dirname, '..', dir)) + mkdirp.sync(path.join(__dirname, '..', '..', dir)) } catch (error) { throw new Error('Cannot create ' + path + ':' + error) } diff --git a/server/lib/videos.js b/server/lib/videos.js index 1e0a83b8b44150aed8437080f36fee28f58a0054..3c6ee4a650605a1b4ccad24bea60a191300fbb58 100644 --- a/server/lib/videos.js +++ b/server/lib/videos.js @@ -9,7 +9,7 @@ var webtorrent = require('../lib/webtorrent') var logger = require('../helpers/logger') var Videos = require('../models/videos') -var uploadDir = path.join(__dirname, '..', config.get('storage.uploads')) +var uploadDir = path.join(__dirname, '..', '..', config.get('storage.uploads')) var videos = { seed: seed, diff --git a/server/models/videos.js b/server/models/videos.js index 5e2eeae07bbfb7605e7fa692fe6f30e7abd540e9..436c08bfd473d62474c8fb2375cafd12980f11b4 100644 --- a/server/models/videos.js +++ b/server/models/videos.js @@ -12,7 +12,7 @@ var logger = require('../helpers/logger') var http = config.get('webserver.https') === true ? 'https' : 'http' var host = config.get('webserver.host') var port = config.get('webserver.port') -var uploadDir = path.join(__dirname, '..', config.get('storage.uploads')) +var uploadDir = path.join(__dirname, '..', '..', config.get('storage.uploads')) // --------------------------------------------------------------------------- diff --git a/server/tests/api/checkParams.js b/server/tests/api/checkParams.js index 1c1ec71b3cf336442dc642017508b98617c2da6b..01b620873398f66829e30f5206266d3cd61befa6 100644 --- a/server/tests/api/checkParams.js +++ b/server/tests/api/checkParams.js @@ -252,7 +252,7 @@ describe('Test parameters validator', function () { it('Should have 404 with nothing', function (done) { request(url) .delete(path) - .expect(404, done) + .expect(400, done) }) it('Should fail without a mongodb id', function (done) { diff --git a/server/tests/api/singlePod.js b/server/tests/api/singlePod.js index 3dd72c01b8c45692b4269ecff123b784d045cd90..d572aacf67d64ddcd83800a3420f7d9ba6f2d598 100644 --- a/server/tests/api/singlePod.js +++ b/server/tests/api/singlePod.js @@ -112,7 +112,7 @@ describe('Test a single pod', function () { utils.removeVideo(url, video_id, function (err) { if (err) throw err - fs.readdir(pathUtils.join(__dirname, '../../test1/uploads/'), function (err, files) { + fs.readdir(pathUtils.join(__dirname, '../../../test1/uploads/'), function (err, files) { if (err) throw err expect(files.length).to.equal(0) diff --git a/server/tests/api/utils.js b/server/tests/api/utils.js index 47b7062946256299a870522a02b4e6e4fa01495b..c706965714684fac06241d09d722dedd9228dcda 100644 --- a/server/tests/api/utils.js +++ b/server/tests/api/utils.js @@ -22,7 +22,7 @@ var testUtils = { // ---------------------- Export functions -------------------- function flushTests (callback) { - exec(pathUtils.join(__dirname, '../../scripts/clean_test.sh'), callback) + exec(pathUtils.join(__dirname, '../../../bin/clean_test.sh'), callback) } function getFriendsList (url, end) { @@ -139,7 +139,7 @@ function runServer (number, callback) { detached: true } - var app = fork(pathUtils.join(__dirname, '../../server.js'), [], options) + var app = fork(pathUtils.join(__dirname, '../../../server.js'), [], options) app.stdout.on('data', function onStdout (data) { var dont_continue = false // Check if all required sentences are here