Merge branch 'private-param-235' of github.com:pelias/api into private-param-235

sanitiser/_details.js

@@ -1,9 +1,10 @@
 var isObject = require('is-object');
+var isTruthy = require('./_truthy');
 
 // validate inputs, convert types and apply defaults
 function sanitize( req, default_value ){
 
-  var clean = req.clean || {};
+  req.clean = req.clean || {};
   var params= req.query;
 
   if (default_value === undefined) {
@@ -17,25 +18,14 @@ function sanitize( req, default_value ){
     params = {};
   }
 
-  if (params.details !== undefined) {
+  if (params.details === undefined) {
-    clean.details = isTruthy(params.details);
+    req.clean.details = default_value;
   } else {
-    clean.details = default_value;
+    req.clean.details = isTruthy(params.details);
   }
 
-  req.clean = clean;
-  
   return {'error':false};
 
 }
 
-function isTruthy(val) {
-  if (typeof val === 'string') {
-    return ['true', '1', 'yes', 'y'].indexOf(val) !== -1;
-  }
-
-  return val === 1 || val === true;
-}
-
-// export function
 module.exports = sanitize;

sanitiser/_private.js

@@ -0,0 +1,31 @@
+var isObject = require('is-object');
+var isTruthy = require('./_truthy');
+
+// validate inputs, convert types and apply defaults
+function sanitize( req, default_value ){
+
+  req.clean = req.clean || {};
+  var params= req.query;
+
+  if (default_value === undefined) {
+    default_value = true;
+  }
+
+  default_value = !!default_value;
+
+  // ensure the input params are a valid object
+  if( !isObject( params ) ){
+    params = {};
+  }
+
+  if (params.private === undefined) {
+    req.clean.private = default_value;
+  } else {
+    req.clean.private = isTruthy(params.private);
+  }
+
+  return {'error':false};
+
+}
+
+module.exports = sanitize;

sanitiser/_truthy.js

@@ -0,0 +1,9 @@
+function isTruthy(val) {
+  if (typeof val === 'string') {
+    return ['true', '1', 'yes', 'y'].indexOf(val) !== -1;
+  }
+
+  return val === 1 || val === true;
+}
+
+module.exports = isTruthy;

sanitiser/search.js

@@ -6,6 +6,7 @@ var _sanitize = require('../sanitiser/_sanitize'),
       layers: require('../sanitiser/_layers'),
       source: require('../sanitiser/_source'),
       details: require('../sanitiser/_details'),
+      private: require('../sanitiser/_private'),
       latlonzoom: require('../sanitiser/_geo_search')
     };
 

test/unit/run.js

@@ -8,7 +8,10 @@ var tests = [
   require('./controller/search'),
   require('./service/mget'),
   require('./service/search'),
+  require('./sanitiser/_details'),
+  require('./sanitiser/_private'),
   require('./sanitiser/_source'),
+  require('./sanitiser/_truthy'),
   require('./sanitiser/search'),
   require('./sanitiser/reverse'),
   require('./sanitiser/place'),

test/unit/sanitiser/_details.js

@@ -0,0 +1,53 @@
+var sanitize = require('../../../sanitiser/_details');
+
+module.exports.tests = {};
+
+module.exports.tests.sanitize_details = function(test, common) {
+  var invalid_values = [null, -1, 123, NaN, 'abc'];
+  invalid_values.forEach(function(detailsValue) {
+    test('invalid details param ' + detailsValue, function(t) {
+      var req = {query: { details: detailsValue }};
+      sanitize(req);
+      t.equal(req.clean.details, false, 'default details set (to false)');
+      t.end();
+    });
+  });
+
+  var valid_values = ['true', true, 1, '1', 'yes', 'y'];
+  valid_values.forEach(function(detailsValue) {
+    test('valid details param ' + detailsValue, function(t) {
+      var req = {query: { details: detailsValue }};
+      sanitize(req);
+      t.equal(req.clean.details, true, 'details set to true');
+      t.end();
+    });
+  });
+
+  var valid_false_values = ['false', false, 0, '0', 'no', 'n'];
+  valid_false_values.forEach(function(detailsValue) {
+    test('test setting false explicitly ' + detailsValue, function(t) {
+      var req = {query: { details: detailsValue }};
+      sanitize(req);
+      t.equal(req.clean.details, false, 'details set to false');
+      t.end();
+    });
+  });
+
+  test('test default behavior', function(t) {
+    var req = {query: {}};
+    sanitize(req);
+    t.equal(req.clean.details, true, 'details set to true');
+    t.end();
+  });
+};
+
+module.exports.all = function (tape, common) {
+
+  function test(name, testFunction) {
+    return tape('SANTIZE _details ' + name, testFunction);
+  }
+
+  for( var testCase in module.exports.tests ){
+    module.exports.tests[testCase](test, common);
+  }
+};

test/unit/sanitiser/_private.js

@@ -0,0 +1,53 @@
+var sanitize = require('../../../sanitiser/_private');
+
+module.exports.tests = {};
+
+module.exports.tests.sanitize_private = function(test, common) {
+  var invalid_values = [null, -1, 123, NaN, 'abc'];
+  invalid_values.forEach(function(privateValue) {
+    test('invalid private param ' + privateValue, function(t) {
+      var req = {query: { private: privateValue }};
+      sanitize(req);
+      t.equal(req.clean.private, false, 'default private set (to false)');
+      t.end();
+    });
+  });
+
+  var valid_values = ['true', true, 1, '1', 'yes', 'y'];
+  valid_values.forEach(function(privateValue) {
+    test('valid private param ' + privateValue, function(t) {
+      var req = {query: { private: privateValue }};
+      sanitize(req);
+      t.equal(req.clean.private, true, 'private set to true');
+      t.end();
+    });
+  });
+
+  var valid_false_values = ['false', false, 0, '0', 'no', 'n'];
+  valid_false_values.forEach(function(privateValue) {
+    test('test setting false explicitly ' + privateValue, function(t) {
+      var req = {query: { private: privateValue }};
+      sanitize(req);
+      t.equal(req.clean.private, false, 'private set to false');
+      t.end();
+    });
+  });
+
+  test('test default behavior', function(t) {
+    var req = {query: {}};
+    sanitize(req);
+    t.equal(req.clean.private, true, 'private set to true');
+    t.end();
+  });
+};
+
+module.exports.all = function (tape, common) {
+
+  function test(name, testFunction) {
+    return tape('SANTIZE _private ' + name, testFunction);
+  }
+
+  for( var testCase in module.exports.tests ){
+    module.exports.tests[testCase](test, common);
+  }
+};

test/unit/sanitiser/_truthy.js

@@ -0,0 +1,31 @@
+var isTruthy = require('../../../sanitiser/_truthy');
+
+module.exports.tests = {};
+
+module.exports.tests.sanitize_truthy = function(test, common) {
+  var valid_values = ['true', true, 1, '1', 'yes', 'y'];
+  valid_values.forEach(function(value) {
+    test('truthy value ' + value, function(t) {
+      t.equal(isTruthy(value), true, 'returns true');
+      t.end();
+    });
+  });
+
+  var valid_false_values = ['false', false, 0, '0', 'no', 'n', null, -1, 123, NaN, 'abc'];
+  valid_false_values.forEach(function(value) {
+    test('falsey value ' + value, function(t) {
+      t.equal(isTruthy(value), false, 'returns false');
+      t.end();
+    });
+  });
+};
+
+module.exports.all = function (tape, common) {
+  function test(name, testFunction) {
+    return tape('SANTIZE _truthy ' + name, testFunction);
+  }
+
+  for( var testCase in module.exports.tests ){
+    module.exports.tests[testCase](test, common);
+  }
+};

test/unit/sanitiser/search.js

@@ -86,6 +86,33 @@ module.exports.tests.sanitize_text_with_delim = function(test, common) {
   });
 };
 
+module.exports.tests.sanitize_private_no_value = function(test, common) {
+  test('default private should be set to true', function(t) {
+    sanitize({ text: 'test' }, function( err, clean ){
+      t.equal(clean.private, true, 'private set to true');
+    });
+    t.end();
+  });
+};
+
+module.exports.tests.sanitize_private_explicit_true_value = function(test, common) {
+  test('explicit private should be set to true', function(t) {
+    sanitize({ text: 'test', private: true }, function( err, clean ){
+      t.equal(clean.private, true, 'private set to true');
+    });
+    t.end();
+  });
+};
+
+module.exports.tests.sanitize_private_explicit_false_value = function(test, common) {
+  test('explicit private should be set to false', function(t) {
+    sanitize({ text: 'test', private: false }, function( err, clean ){
+      t.equal(clean.private, false, 'private set to false');
+    });
+    t.end();
+  });
+};
+
 module.exports.tests.sanitize_lat = function(test, common) {
   var lats = {
     invalid: [],
@@ -234,45 +261,6 @@ module.exports.tests.sanitize_size = function(test, common) {
   });
 };
 
-module.exports.tests.sanitize_details = function(test, common) {
-  var invalid_values = [null, -1, 123, NaN, 'abc'];
-  invalid_values.forEach(function(details) {
-    test('invalid details param ' + details, function(t) {
-      sanitize({ text: 'test', lat: 0, lon: 0, details: details }, function( err, clean ){
-        t.equal(clean.details, false, 'default details set (to false)');
-        t.end();
-      });
-    });
-  });
-
-  var valid_values = ['true', true, 1, '1', 'yes', 'y'];
-  valid_values.forEach(function(details) {
-    test('valid details param ' + details, function(t) {
-      sanitize({ text: 'test', details: details }, function( err, clean ){
-        t.equal(clean.details, true, 'details set to true');
-        t.end();
-      });
-    });
-  });
-
-  var valid_false_values = ['false', false, 0, '0', 'no', 'n'];
-  valid_false_values.forEach(function(details) {
-    test('test setting false explicitly ' + details, function(t) {
-      sanitize({ text: 'test', details: details }, function( err, clean ){
-        t.equal(clean.details, false, 'details set to false');
-        t.end();
-      });
-    });
-  });
-
-  test('test default behavior', function(t) {
-    sanitize({ text: 'test' }, function( err, clean ){
-      t.equal(clean.details, true, 'details set to true');
-      t.end();
-    });
-  });
-};
-
 module.exports.tests.sanitize_layers = function(test, common) {
   test('unspecified', function(t) {
     sanitize({ layers: undefined, text: 'test' }, function( err, clean ){