pelias
/
api
mirror of https://github.com/pelias/api.git
1
0
Fork 0
Code Issues Releases Activity
Browse Source

change param:input to param:text

pull/229/head
Peter Johnson 9 years ago
parent
05bdf784e0
commit
22322eaa32
8 changed files with 107 additions and 107 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Unified View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 6
      helper/query_parser.js
  2. 44
      query/search.js
  3. 6
      query/sort.js
  4. 16
      sanitiser/_text.js
  5. 2
      sanitiser/search.js
  6. 20
      test/unit/query/search.js
  7. 8
      test/unit/sanitiser/_text.js
  8. 112
      test/unit/sanitiser/search.js

6
helper/query_parser.js
Unescape View File

@ -62,13 +62,13 @@ module.exports.get_parsed_address = function get_parsed_address(query) {
'admin_parts' 'admin_parts'
]; ];
var parsed_input = {}; var parsed_text = {};
address_parts.forEach(function(part){ address_parts.forEach(function(part){
if (parsedAddress[part]) { if (parsedAddress[part]) {
parsed_input[part] = parsedAddress[part]; parsed_text[part] = parsedAddress[part];
} }
}); });
return parsed_input; return parsed_text;
}; };

44
query/search.js
Unescape View File

@ -42,7 +42,7 @@ function generateQuery( clean ){
var vs = new peliasQuery.Vars( peliasQuery.defaults ); var vs = new peliasQuery.Vars( peliasQuery.defaults );
// set input text // set input text
vs.var( 'input:name', clean.input ); vs.var( 'input:name', clean.text );
// set size // set size
if( clean.size ){ if( clean.size ){
@ -68,17 +68,17 @@ function generateQuery( clean ){
} }
// address parsing // address parsing
if( clean.parsed_input ){ if( clean.parsed_text ){
// is it a street address? // is it a street address?
var isStreetAddress = clean.parsed_input.hasOwnProperty('number') && clean.parsed_input.hasOwnProperty('street'); var isStreetAddress = clean.parsed_text.hasOwnProperty('number') && clean.parsed_text.hasOwnProperty('street');
if( isStreetAddress ){ if( isStreetAddress ){
vs.var( 'input:name', clean.parsed_input.number + ' ' + clean.parsed_input.street ); vs.var( 'input:name', clean.parsed_text.number + ' ' + clean.parsed_text.street );
} }
// I don't understand this // I don't understand this
else if( clean.parsed_input.admin_parts ) { else if( clean.parsed_text.admin_parts ) {
vs.var( 'input:name', clean.parsed_input.name ); vs.var( 'input:name', clean.parsed_text.name );
} }
// or this.. // or this..
@ -91,35 +91,35 @@ function generateQuery( clean ){
// ==== add parsed matches [address components] ==== // ==== add parsed matches [address components] ====
// house number // house number
if( clean.parsed_input.hasOwnProperty('number') ){ if( clean.parsed_text.hasOwnProperty('number') ){
vs.var( 'input:housenumber', clean.parsed_input.number ); vs.var( 'input:housenumber', clean.parsed_text.number );
} }
// street name // street name
if( clean.parsed_input.hasOwnProperty('street') ){ if( clean.parsed_text.hasOwnProperty('street') ){
vs.var( 'input:street', clean.parsed_input.street ); vs.var( 'input:street', clean.parsed_text.street );
} }
// postal code // postal code
if( clean.parsed_input.hasOwnProperty('postalcode') ){ if( clean.parsed_text.hasOwnProperty('postalcode') ){
vs.var( 'input:postcode', clean.parsed_input.postalcode ); vs.var( 'input:postcode', clean.parsed_text.postalcode );
} }
// ==== add parsed matches [admin components] ==== // ==== add parsed matches [admin components] ====
// city // city
if( clean.parsed_input.hasOwnProperty('city') ){ if( clean.parsed_text.hasOwnProperty('city') ){
vs.var( 'input:admin2', clean.parsed_input.city ); vs.var( 'input:admin2', clean.parsed_text.city );
} }
// state // state
if( clean.parsed_input.hasOwnProperty('state') ){ if( clean.parsed_text.hasOwnProperty('state') ){
vs.var( 'input:admin1_abbr', clean.parsed_input.state ); vs.var( 'input:admin1_abbr', clean.parsed_text.state );
} }
// country // country
if( clean.parsed_input.hasOwnProperty('country') ){ if( clean.parsed_text.hasOwnProperty('country') ){
vs.var( 'input:alpha3', clean.parsed_input.country ); vs.var( 'input:alpha3', clean.parsed_text.country );
} }
// ==== deal with the 'leftover' components ==== // ==== deal with the 'leftover' components ====
@ -127,11 +127,11 @@ function generateQuery( clean ){
// a concept called 'leftovers' which is just 'admin_parts' /or 'regions'. // a concept called 'leftovers' which is just 'admin_parts' /or 'regions'.
var leftoversString = ''; var leftoversString = '';
if( clean.parsed_input.hasOwnProperty('admin_parts') ){ if( clean.parsed_text.hasOwnProperty('admin_parts') ){
leftoversString = clean.parsed_input.admin_parts; leftoversString = clean.parsed_text.admin_parts;
} }
else if( clean.parsed_input.hasOwnProperty('regions') ){ else if( clean.parsed_text.hasOwnProperty('regions') ){
leftoversString = clean.parsed_input.regions.join(' '); leftoversString = clean.parsed_text.regions.join(' ');
} }
// if we have 'leftovers' then assign them to any fields which // if we have 'leftovers' then assign them to any fields which

6
query/sort.js
Unescape View File

@ -66,9 +66,9 @@ module.exports = function( params ){
}; };
function getCategoryWeights(params) { function getCategoryWeights(params) {
if (params && params.hasOwnProperty('parsed_input') && if (params && params.hasOwnProperty('parsed_text') &&
(params.parsed_input.hasOwnProperty('number') || (params.parsed_text.hasOwnProperty('number') ||
params.parsed_input.hasOwnProperty('street'))) { params.parsed_text.hasOwnProperty('street'))) {
return category_weights.address; return category_weights.address;
} }
return category_weights.default; return category_weights.default;

16
sanitiser/_input.js → sanitiser/_text.js
Unescape View File

@ -1,30 +1,30 @@
var isObject = require('is-object'); var isObject = require('is-object');
var query_parser = require('../helper/query_parser'); var query_parser = require('../helper/query_parser');
// validate inputs, convert types and apply defaults // validate texts, convert types and apply defaults
function sanitize( req ){ function sanitize( req ){
req.clean = req.clean || {}; req.clean = req.clean || {};
var params= req.query; var params= req.query;
// ensure the input params are a valid object // ensure the text params are a valid object
if( !isObject( params ) ){ if( !isObject( params ) ){
params = {}; params = {};
} }
// input text // text text
if('string' !== typeof params.input || !params.input.length){ if('string' !== typeof params.text || !params.text.length){
return { return {
'error': true, 'error': true,
'message': 'invalid param \'input\': text length, must be >0' 'message': 'invalid param \'text\': text length, must be >0'
}; };
} }
req.clean.input = params.input; req.clean.text = params.text;
req.clean.parsed_input = query_parser.get_parsed_address(params.input); req.clean.parsed_text = query_parser.get_parsed_address(params.text);
req.clean.types = req.clean.layers || {}; req.clean.types = req.clean.layers || {};
req.clean.types.from_address_parsing = query_parser.get_layers(params.input); req.clean.types.from_address_parsing = query_parser.get_layers(params.text);
return { 'error': false }; return { 'error': false };
} }

2
sanitiser/search.js
Unescape View File

@ -1,7 +1,7 @@
var _sanitize = require('../sanitiser/_sanitize'), var _sanitize = require('../sanitiser/_sanitize'),
sanitizers = { sanitizers = {
input: require('../sanitiser/_input'), text: require('../sanitiser/_text'),
size: require('../sanitiser/_size'), size: require('../sanitiser/_size'),
layers: require('../sanitiser/_layers'), layers: require('../sanitiser/_layers'),
source: require('../sanitiser/_source'), source: require('../sanitiser/_source'),

20
test/unit/query/search.js
Unescape View File

@ -25,7 +25,7 @@ var sort = require('../fixture/sort_default');
module.exports.tests.query = function(test, common) { module.exports.tests.query = function(test, common) {
test('valid search + focus + bbox', function(t) { test('valid search + focus + bbox', function(t) {
var query = generate({ var query = generate({
input: 'test', size: 10, text: 'test', size: 10,
lat: 29.49136, lon: -82.50622, lat: 29.49136, lon: -82.50622,
bbox: { bbox: {
top: 47.47, top: 47.47,
@ -45,7 +45,7 @@ module.exports.tests.query = function(test, common) {
test('valid search + bbox', function(t) { test('valid search + bbox', function(t) {
var query = generate({ var query = generate({
input: 'test', size: 10, text: 'test', size: 10,
bbox: { bbox: {
top: 47.47, top: 47.47,
right: -61.84, right: -61.84,
@ -64,7 +64,7 @@ module.exports.tests.query = function(test, common) {
test('valid lingustic-only search', function(t) { test('valid lingustic-only search', function(t) {
var query = generate({ var query = generate({
input: 'test', size: 10, text: 'test', size: 10,
layers: ['test'] layers: ['test']
}); });
@ -77,7 +77,7 @@ module.exports.tests.query = function(test, common) {
test('search search + focus', function(t) { test('search search + focus', function(t) {
var query = generate({ var query = generate({
input: 'test', size: 10, text: 'test', size: 10,
lat: 29.49136, lon: -82.50622, lat: 29.49136, lon: -82.50622,
layers: ['test'] layers: ['test']
}); });
@ -91,12 +91,12 @@ module.exports.tests.query = function(test, common) {
test('valid query with a full valid address', function(t) { test('valid query with a full valid address', function(t) {
var address = '123 main st new york ny 10010 US'; var address = '123 main st new york ny 10010 US';
var query = generate({ input: address, var query = generate({ text: address,
layers: [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood', layers: [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood',
'locality', 'local_admin', 'osmaddress', 'openaddresses' ], 'locality', 'local_admin', 'osmaddress', 'openaddresses' ],
size: 10, size: 10,
details: true, details: true,
parsed_input: parser.get_parsed_address(address), parsed_text: parser.get_parsed_address(address),
}); });
var expected = require('../fixture/search_full_address'); var expected = require('../fixture/search_full_address');
@ -107,12 +107,12 @@ module.exports.tests.query = function(test, common) {
test('valid query with partial address', function(t) { test('valid query with partial address', function(t) {
var partial_address = 'soho grand, new york'; var partial_address = 'soho grand, new york';
var query = generate({ input: partial_address, var query = generate({ text: partial_address,
layers: [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood', layers: [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood',
'locality', 'local_admin', 'osmaddress', 'openaddresses' ], 'locality', 'local_admin', 'osmaddress', 'openaddresses' ],
size: 10, size: 10,
details: true, details: true,
parsed_input: parser.get_parsed_address(partial_address), parsed_text: parser.get_parsed_address(partial_address),
}); });
var expected = require('../fixture/search_partial_address'); var expected = require('../fixture/search_partial_address');
@ -124,12 +124,12 @@ module.exports.tests.query = function(test, common) {
test('valid query with regions in address', function(t) { test('valid query with regions in address', function(t) {
var partial_address = '1 water st manhattan ny'; var partial_address = '1 water st manhattan ny';
var query = generate({ input: partial_address, var query = generate({ text: partial_address,
layers: [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood', layers: [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood',
'locality', 'local_admin', 'osmaddress', 'openaddresses' ], 'locality', 'local_admin', 'osmaddress', 'openaddresses' ],
size: 10, size: 10,
details: true, details: true,
parsed_input: parser.get_parsed_address(partial_address), parsed_text: parser.get_parsed_address(partial_address),
}); });
var expected = require('../fixture/search_regions_address'); var expected = require('../fixture/search_regions_address');

8
test/unit/sanitiser/_input.js → test/unit/sanitiser/_text.js
Unescape View File

@ -1,18 +1,18 @@
var input = require('../../../sanitiser/_input'), var text = require('../../../sanitiser/_text'),
parser = require('../../../helper/query_parser'), parser = require('../../../helper/query_parser'),
delim = ',', delim = ',',
defaultError = 'invalid param \'input\': text length, must be >0', defaultError = 'invalid param \'text\': text length, must be >0',
allLayers = [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood', allLayers = [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood',
'locality', 'local_admin', 'osmaddress', 'openaddresses' ], 'locality', 'local_admin', 'osmaddress', 'openaddresses' ],
nonAddressLayers = [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood', nonAddressLayers = [ 'geoname', 'osmnode', 'osmway', 'admin0', 'admin1', 'admin2', 'neighborhood',
'locality', 'local_admin' ], 'locality', 'local_admin' ],
defaultParsed= { }, defaultParsed= { },
defaultClean = { input: 'test', defaultClean = { text: 'test',
layers: allLayers, layers: allLayers,
size: 10, size: 10,
details: true, details: true,
parsed_input: defaultParsed, parsed_text: defaultParsed,
lat:0, lat:0,
lon:0 lon:0
}, },

112
test/unit/sanitiser/search.js
Unescape View File

@ -1,18 +1,18 @@
var search = require('../../../sanitiser/search'), var search = require('../../../sanitiser/search'),
_input = require('../sanitiser/_input'), _text = require('../sanitiser/_text'),
parser = require('../../../helper/query_parser'), parser = require('../../../helper/query_parser'),
defaultParsed = _input.defaultParsed, defaultParsed = _text.defaultParsed,
_sanitize = search.sanitize, _sanitize = search.sanitize,
middleware = search.middleware, middleware = search.middleware,
delim = ',', delim = ',',
defaultError = 'invalid param \'input\': text length, must be >0', defaultError = 'invalid param \'text\': text length, must be >0',
defaultClean = { input: 'test', defaultClean = { text: 'test',
types: { types: {
}, },
size: 10, size: 10,
details: true, details: true,
parsed_input: defaultParsed, parsed_text: defaultParsed,
}, },
sanitize = function(query, cb) { _sanitize({'query':query}, cb); }; sanitize = function(query, cb) { _sanitize({'query':query}, cb); };
@ -31,12 +31,12 @@ module.exports.tests.interface = function(test, common) {
}); });
}; };
module.exports.tests.sanitize_invalid_input = function(test, common) { module.exports.tests.sanitize_invalid_text = function(test, common) {
test('invalid input', function(t) { test('invalid text', function(t) {
var invalid = [ '', 100, null, undefined, new Date() ]; var invalid = [ '', 100, null, undefined, new Date() ];
invalid.forEach( function( input ){ invalid.forEach( function( text ){
sanitize({ input: input }, function( err, clean ){ sanitize({ text: text }, function( err, clean ){
t.equal(err, 'invalid param \'input\': text length, must be >0', input + ' is an invalid input'); t.equal(err, 'invalid param \'text\': text length, must be >0', text + ' is an invalid text');
t.equal(clean, undefined, 'clean not set'); t.equal(clean, undefined, 'clean not set');
}); });
}); });
@ -44,41 +44,41 @@ module.exports.tests.sanitize_invalid_input = function(test, common) {
}); });
}; };
module.exports.tests.sanitise_valid_input = function(test, common) { module.exports.tests.sanitise_valid_text = function(test, common) {
test('valid short input', function(t) { test('valid short text', function(t) {
sanitize({ input: 'a' }, function( err, clean ){ sanitize({ text: 'a' }, function( err, clean ){
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
}); });
t.end(); t.end();
}); });
test('valid not-quite-as-short input', function(t) { test('valid not-quite-as-short text', function(t) {
sanitize({ input: 'aa' }, function( err, clean ){ sanitize({ text: 'aa' }, function( err, clean ){
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
}); });
t.end(); t.end();
}); });
test('valid longer input', function(t) { test('valid longer text', function(t) {
sanitize({ input: 'aaaaaaaa' }, function( err, clean ){ sanitize({ text: 'aaaaaaaa' }, function( err, clean ){
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
}); });
t.end(); t.end();
}); });
}; };
module.exports.tests.sanitize_input_with_delim = function(test, common) { module.exports.tests.sanitize_text_with_delim = function(test, common) {
var inputs = [ 'a,bcd', '123 main st, admin1', ',,,', ' ' ]; var texts = [ 'a,bcd', '123 main st, admin1', ',,,', ' ' ];
test('valid inputs with a comma', function(t) { test('valid texts with a comma', function(t) {
inputs.forEach( function( input ){ texts.forEach( function( text ){
sanitize({ input: input }, function( err, clean ){ sanitize({ text: text }, function( err, clean ){
var expected = JSON.parse(JSON.stringify( defaultClean )); var expected = JSON.parse(JSON.stringify( defaultClean ));
expected.input = input; expected.text = text;
expected.parsed_input = parser.get_parsed_address(input); expected.parsed_text = parser.get_parsed_address(text);
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
t.equal(clean.parsed_input.name, expected.parsed_input.name, 'clean name set correctly'); t.equal(clean.parsed_text.name, expected.parsed_text.name, 'clean name set correctly');
}); });
}); });
@ -93,7 +93,7 @@ module.exports.tests.sanitize_lat = function(test, common) {
}; };
test('invalid lat', function(t) { test('invalid lat', function(t) {
lats.invalid.forEach( function( lat ){ lats.invalid.forEach( function( lat ){
sanitize({ input: 'test', lat: lat, lon: 0 }, function( err, clean ){ sanitize({ text: 'test', lat: lat, lon: 0 }, function( err, clean ){
t.equal(err, 'invalid param \'lat\': must be >-90 and <90', lat + ' is an invalid latitude'); t.equal(err, 'invalid param \'lat\': must be >-90 and <90', lat + ' is an invalid latitude');
t.equal(clean, undefined, 'clean not set'); t.equal(clean, undefined, 'clean not set');
}); });
@ -102,7 +102,7 @@ module.exports.tests.sanitize_lat = function(test, common) {
}); });
test('valid lat', function(t) { test('valid lat', function(t) {
lats.valid.forEach( function( lat ){ lats.valid.forEach( function( lat ){
sanitize({ input: 'test', lat: lat, lon: 0 }, function( err, clean ){ sanitize({ text: 'test', lat: lat, lon: 0 }, function( err, clean ){
var expected = JSON.parse(JSON.stringify( defaultClean )); var expected = JSON.parse(JSON.stringify( defaultClean ));
expected.lat = parseFloat( lat ); expected.lat = parseFloat( lat );
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
@ -119,7 +119,7 @@ module.exports.tests.sanitize_lon = function(test, common) {
}; };
test('valid lon', function(t) { test('valid lon', function(t) {
lons.valid.forEach( function( lon ){ lons.valid.forEach( function( lon ){
sanitize({ input: 'test', lat: 0, lon: lon }, function( err, clean ){ sanitize({ text: 'test', lat: 0, lon: lon }, function( err, clean ){
var expected = JSON.parse(JSON.stringify( defaultClean )); var expected = JSON.parse(JSON.stringify( defaultClean ));
expected.lon = parseFloat( lon ); expected.lon = parseFloat( lon );
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
@ -132,7 +132,7 @@ module.exports.tests.sanitize_lon = function(test, common) {
module.exports.tests.sanitize_optional_geo = function(test, common) { module.exports.tests.sanitize_optional_geo = function(test, common) {
test('no lat/lon', function(t) { test('no lat/lon', function(t) {
sanitize({ input: 'test' }, function( err, clean ){ sanitize({ text: 'test' }, function( err, clean ){
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
t.equal(clean.lat, undefined, 'clean set without lat'); t.equal(clean.lat, undefined, 'clean set without lat');
t.equal(clean.lon, undefined, 'clean set without lon'); t.equal(clean.lon, undefined, 'clean set without lon');
@ -140,7 +140,7 @@ module.exports.tests.sanitize_optional_geo = function(test, common) {
t.end(); t.end();
}); });
test('no lat', function(t) { test('no lat', function(t) {
sanitize({ input: 'test', lon: 0 }, function( err, clean ){ sanitize({ text: 'test', lon: 0 }, function( err, clean ){
var expected_lon = 0; var expected_lon = 0;
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
t.deepEqual(clean.lon, expected_lon, 'clean set correctly (without any lat)'); t.deepEqual(clean.lon, expected_lon, 'clean set correctly (without any lat)');
@ -148,7 +148,7 @@ module.exports.tests.sanitize_optional_geo = function(test, common) {
t.end(); t.end();
}); });
test('no lon', function(t) { test('no lon', function(t) {
sanitize({ input: 'test', lat: 0 }, function( err, clean ){ sanitize({ text: 'test', lat: 0 }, function( err, clean ){
var expected_lat = 0; var expected_lat = 0;
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
t.deepEqual(clean.lat, expected_lat, 'clean set correctly (without any lon)'); t.deepEqual(clean.lat, expected_lat, 'clean set correctly (without any lon)');
@ -187,7 +187,7 @@ module.exports.tests.sanitize_bbox = function(test, common) {
}; };
test('invalid bbox', function(t) { test('invalid bbox', function(t) {
bboxes.invalid.forEach( function( bbox ){ bboxes.invalid.forEach( function( bbox ){
sanitize({ input: 'test', bbox: bbox }, function( err, clean ){ sanitize({ text: 'test', bbox: bbox }, function( err, clean ){
t.equal(err, undefined, 'no error'); t.equal(err, undefined, 'no error');
t.equal(clean.bbox, undefined, 'falling back on 50km distance from centroid'); t.equal(clean.bbox, undefined, 'falling back on 50km distance from centroid');
}); });
@ -196,7 +196,7 @@ module.exports.tests.sanitize_bbox = function(test, common) {
}); });
test('valid bbox', function(t) { test('valid bbox', function(t) {
bboxes.valid.forEach( function( bbox ){ bboxes.valid.forEach( function( bbox ){
sanitize({ input: 'test', bbox: bbox }, function( err, clean ){ sanitize({ text: 'test', bbox: bbox }, function( err, clean ){
var bboxArray = bbox.split(',').map(function(i) { var bboxArray = bbox.split(',').map(function(i) {
return parseInt(i); return parseInt(i);
}); });
@ -216,19 +216,19 @@ module.exports.tests.sanitize_bbox = function(test, common) {
module.exports.tests.sanitize_zoom = function(test, common) { module.exports.tests.sanitize_zoom = function(test, common) {
test('invalid zoom value', function(t) { test('invalid zoom value', function(t) {
sanitize({ zoom: 'a', input: 'test', lat: 0, lon: 0 }, function( err, clean ){ sanitize({ zoom: 'a', text: 'test', lat: 0, lon: 0 }, function( err, clean ){
t.equal(clean.zoom, undefined, 'zoom not set'); t.equal(clean.zoom, undefined, 'zoom not set');
t.end(); t.end();
}); });
}); });
test('below min zoom value', function(t) { test('below min zoom value', function(t) {
sanitize({ zoom: -100, input: 'test', lat: 0, lon: 0 }, function( err, clean ){ sanitize({ zoom: -100, text: 'test', lat: 0, lon: 0 }, function( err, clean ){
t.equal(clean.zoom, 1, 'min zoom set'); t.equal(clean.zoom, 1, 'min zoom set');
t.end(); t.end();
}); });
}); });
test('above max zoom value', function(t) { test('above max zoom value', function(t) {
sanitize({ zoom: 9999, input: 'test', lat: 0, lon: 0 }, function( err, clean ){ sanitize({ zoom: 9999, text: 'test', lat: 0, lon: 0 }, function( err, clean ){
t.equal(clean.zoom, 18, 'max zoom set'); t.equal(clean.zoom, 18, 'max zoom set');
t.end(); t.end();
}); });
@ -237,19 +237,19 @@ module.exports.tests.sanitize_zoom = function(test, common) {
module.exports.tests.sanitize_size = function(test, common) { module.exports.tests.sanitize_size = function(test, common) {
test('invalid size value', function(t) { test('invalid size value', function(t) {
sanitize({ size: 'a', input: 'test', lat: 0, lon: 0 }, function( err, clean ){ sanitize({ size: 'a', text: 'test', lat: 0, lon: 0 }, function( err, clean ){
t.equal(clean.size, 10, 'default size set'); t.equal(clean.size, 10, 'default size set');
t.end(); t.end();
}); });
}); });
test('below min size value', function(t) { test('below min size value', function(t) {
sanitize({ size: -100, input: 'test', lat: 0, lon: 0 }, function( err, clean ){ sanitize({ size: -100, text: 'test', lat: 0, lon: 0 }, function( err, clean ){
t.equal(clean.size, 1, 'min size set'); t.equal(clean.size, 1, 'min size set');
t.end(); t.end();
}); });
}); });
test('above max size value', function(t) { test('above max size value', function(t) {
sanitize({ size: 9999, input: 'test', lat: 0, lon: 0 }, function( err, clean ){ sanitize({ size: 9999, text: 'test', lat: 0, lon: 0 }, function( err, clean ){
t.equal(clean.size, 40, 'max size set'); t.equal(clean.size, 40, 'max size set');
t.end(); t.end();
}); });
@ -260,7 +260,7 @@ module.exports.tests.sanitize_details = function(test, common) {
var invalid_values = [null, -1, 123, NaN, 'abc']; var invalid_values = [null, -1, 123, NaN, 'abc'];
invalid_values.forEach(function(details) { invalid_values.forEach(function(details) {
test('invalid details param ' + details, function(t) { test('invalid details param ' + details, function(t) {
sanitize({ input: 'test', lat: 0, lon: 0, details: details }, function( err, clean ){ sanitize({ text: 'test', lat: 0, lon: 0, details: details }, function( err, clean ){
t.equal(clean.details, false, 'default details set (to false)'); t.equal(clean.details, false, 'default details set (to false)');
t.end(); t.end();
}); });
@ -270,7 +270,7 @@ module.exports.tests.sanitize_details = function(test, common) {
var valid_values = ['true', true, 1, '1', 'yes', 'y']; var valid_values = ['true', true, 1, '1', 'yes', 'y'];
valid_values.forEach(function(details) { valid_values.forEach(function(details) {
test('valid details param ' + details, function(t) { test('valid details param ' + details, function(t) {
sanitize({ input: 'test', lat: 0, lon: 0, details: details }, function( err, clean ){ sanitize({ text: 'test', lat: 0, lon: 0, details: details }, function( err, clean ){
t.equal(clean.details, true, 'details set to true'); t.equal(clean.details, true, 'details set to true');
t.end(); t.end();
}); });
@ -280,7 +280,7 @@ module.exports.tests.sanitize_details = function(test, common) {
var valid_false_values = ['false', false, 0, '0', 'no', 'n']; var valid_false_values = ['false', false, 0, '0', 'no', 'n'];
valid_false_values.forEach(function(details) { valid_false_values.forEach(function(details) {
test('test setting false explicitly ' + details, function(t) { test('test setting false explicitly ' + details, function(t) {
sanitize({ input: 'test', lat: 0, lon: 0, details: details }, function( err, clean ){ sanitize({ text: 'test', lat: 0, lon: 0, details: details }, function( err, clean ){
t.equal(clean.details, false, 'details set to false'); t.equal(clean.details, false, 'details set to false');
t.end(); t.end();
}); });
@ -288,7 +288,7 @@ module.exports.tests.sanitize_details = function(test, common) {
}); });
test('test default behavior', function(t) { test('test default behavior', function(t) {
sanitize({ input: 'test', lat: 0, lon: 0 }, function( err, clean ){ sanitize({ text: 'test', lat: 0, lon: 0 }, function( err, clean ){
t.equal(clean.details, true, 'details set to true'); t.equal(clean.details, true, 'details set to true');
t.end(); t.end();
}); });
@ -297,13 +297,13 @@ module.exports.tests.sanitize_details = function(test, common) {
module.exports.tests.sanitize_layers = function(test, common) { module.exports.tests.sanitize_layers = function(test, common) {
test('unspecified', function(t) { test('unspecified', function(t) {
sanitize({ layers: undefined, input: 'test' }, function( err, clean ){ sanitize({ layers: undefined, text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, defaultClean.types.from_layers, 'default layers set'); t.deepEqual(clean.types.from_layers, defaultClean.types.from_layers, 'default layers set');
t.end(); t.end();
}); });
}); });
test('invalid layer', function(t) { test('invalid layer', function(t) {
sanitize({ layers: 'test_layer', input: 'test' }, function( err, clean ){ sanitize({ layers: 'test_layer', text: 'test' }, function( err, clean ){
var msg = 'invalid param \'layers\': must be one or more of '; var msg = 'invalid param \'layers\': must be one or more of ';
t.true(err.match(msg), 'invalid layer requested'); t.true(err.match(msg), 'invalid layer requested');
t.true(err.length > msg.length, 'invalid error message'); t.true(err.length > msg.length, 'invalid error message');
@ -312,30 +312,30 @@ module.exports.tests.sanitize_layers = function(test, common) {
}); });
test('poi (alias) layer', function(t) { test('poi (alias) layer', function(t) {
var poi_layers = ['geoname','osmnode','osmway']; var poi_layers = ['geoname','osmnode','osmway'];
sanitize({ layers: 'poi', input: 'test' }, function( err, clean ){ sanitize({ layers: 'poi', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, poi_layers, 'poi layers set'); t.deepEqual(clean.types.from_layers, poi_layers, 'poi layers set');
t.end(); t.end();
}); });
}); });
test('admin (alias) layer', function(t) { test('admin (alias) layer', function(t) {
var admin_layers = ['admin0','admin1','admin2','neighborhood','locality','local_admin']; var admin_layers = ['admin0','admin1','admin2','neighborhood','locality','local_admin'];
sanitize({ layers: 'admin', input: 'test' }, function( err, clean ){ sanitize({ layers: 'admin', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, admin_layers, 'admin layers set'); t.deepEqual(clean.types.from_layers, admin_layers, 'admin layers set');
t.end(); t.end();
}); });
}); });
test('address (alias) layer', function(t) { test('address (alias) layer', function(t) {
var address_layers = ['osmaddress','openaddresses']; var address_layers = ['osmaddress','openaddresses'];
sanitize({ layers: 'address', input: 'test' }, function( err, clean ){ sanitize({ layers: 'address', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, address_layers, 'types from layers set'); t.deepEqual(clean.types.from_layers, address_layers, 'types from layers set');
t.deepEqual(clean.types.from_address_parser, _input.allLayers, 'address parser uses default layers'); t.deepEqual(clean.types.from_address_parser, _text.allLayers, 'address parser uses default layers');
t.end(); t.end();
}); });
}); });
test('poi alias layer plus regular layers', function(t) { test('poi alias layer plus regular layers', function(t) {
var poi_layers = ['geoname','osmnode','osmway']; var poi_layers = ['geoname','osmnode','osmway'];
var reg_layers = ['admin0', 'admin1']; var reg_layers = ['admin0', 'admin1'];
sanitize({ layers: 'poi,admin0,admin1', input: 'test' }, function( err, clean ){ sanitize({ layers: 'poi,admin0,admin1', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, reg_layers.concat(poi_layers), 'poi + regular layers'); t.deepEqual(clean.types.from_layers, reg_layers.concat(poi_layers), 'poi + regular layers');
t.end(); t.end();
}); });
@ -343,7 +343,7 @@ module.exports.tests.sanitize_layers = function(test, common) {
test('admin alias layer plus regular layers', function(t) { test('admin alias layer plus regular layers', function(t) {
var admin_layers = ['admin0','admin1','admin2','neighborhood','locality','local_admin']; var admin_layers = ['admin0','admin1','admin2','neighborhood','locality','local_admin'];
var reg_layers = ['geoname', 'osmway']; var reg_layers = ['geoname', 'osmway'];
sanitize({ layers: 'admin,geoname,osmway', input: 'test' }, function( err, clean ){ sanitize({ layers: 'admin,geoname,osmway', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, reg_layers.concat(admin_layers), 'admin + regular layers set'); t.deepEqual(clean.types.from_layers, reg_layers.concat(admin_layers), 'admin + regular layers set');
t.end(); t.end();
}); });
@ -351,21 +351,21 @@ module.exports.tests.sanitize_layers = function(test, common) {
test('address alias layer plus regular layers', function(t) { test('address alias layer plus regular layers', function(t) {
var address_layers = ['osmaddress','openaddresses']; var address_layers = ['osmaddress','openaddresses'];
var reg_layers = ['geoname', 'osmway']; var reg_layers = ['geoname', 'osmway'];
sanitize({ layers: 'address,geoname,osmway', input: 'test' }, function( err, clean ){ sanitize({ layers: 'address,geoname,osmway', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, reg_layers.concat(address_layers), 'address + regular layers set'); t.deepEqual(clean.types.from_layers, reg_layers.concat(address_layers), 'address + regular layers set');
t.end(); t.end();
}); });
}); });
test('alias layer plus regular layers (no duplicates)', function(t) { test('alias layer plus regular layers (no duplicates)', function(t) {
var poi_layers = ['geoname','osmnode','osmway']; var poi_layers = ['geoname','osmnode','osmway'];
sanitize({ layers: 'poi,geoname,osmnode', input: 'test' }, function( err, clean ){ sanitize({ layers: 'poi,geoname,osmnode', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, poi_layers, 'poi layers found (no duplicates)'); t.deepEqual(clean.types.from_layers, poi_layers, 'poi layers found (no duplicates)');
t.end(); t.end();
}); });
}); });
test('multiple alias layers (no duplicates)', function(t) { test('multiple alias layers (no duplicates)', function(t) {
var alias_layers = ['geoname','osmnode','osmway','admin0','admin1','admin2','neighborhood','locality','local_admin']; var alias_layers = ['geoname','osmnode','osmway','admin0','admin1','admin2','neighborhood','locality','local_admin'];
sanitize({ layers: 'poi,admin', input: 'test' }, function( err, clean ){ sanitize({ layers: 'poi,admin', text: 'test' }, function( err, clean ){
t.deepEqual(clean.types.from_layers, alias_layers, 'all layers found (no duplicates)'); t.deepEqual(clean.types.from_layers, alias_layers, 'all layers found (no duplicates)');
t.end(); t.end();
}); });
@ -373,7 +373,7 @@ module.exports.tests.sanitize_layers = function(test, common) {
}; };
module.exports.tests.invalid_params = function(test, common) { module.exports.tests.invalid_params = function(test, common) {
test('invalid input params', function(t) { test('invalid text params', function(t) {
sanitize( undefined, function( err, clean ){ sanitize( undefined, function( err, clean ){
t.equal(err, defaultError, 'handle invalid params gracefully'); t.equal(err, defaultError, 'handle invalid params gracefully');
t.end(); t.end();
@ -396,7 +396,7 @@ module.exports.tests.middleware_failure = function(test, common) {
module.exports.tests.middleware_success = function(test, common) { module.exports.tests.middleware_success = function(test, common) {
test('middleware success', function(t) { test('middleware success', function(t) {
var req = { query: { input: 'test' }}; var req = { query: { text: 'test' }};
var next = function( message ){ var next = function( message ){
t.equal(message, undefined, 'no error message set'); t.equal(message, undefined, 'no error message set');
t.end(); t.end();

Write Preview
Loading…
Cancel
Save