Browse Source

Updated warning for boundary.circle

pull/617/head
Alec Vulfson 8 years ago
parent
commit
254b7ff66b
  1. 4
      sanitiser/_geo_reverse.js
  2. 6
      test/unit/sanitiser/_geo_reverse.js

4
sanitiser/_geo_reverse.js

@ -15,8 +15,8 @@ module.exports = function sanitize( raw, clean ){
return raw.hasOwnProperty('boundary.circle.' + field);
};
if (['lat', 'lon', 'radius'].some(hasBoundaryCircleField)) {
messages.warnings.push('boundary.circle is currently unsupported');
if (['lat', 'lon'].some(hasBoundaryCircleField)) {
messages.warnings.push('boundary.circle.lat/boundary.circle.lon are currently unsupported');
}
try {

6
test/unit/sanitiser/_geo_reverse.js

@ -16,7 +16,7 @@ module.exports.tests.sanitize_boundary_country = function(test, common) {
t.equals(clean['boundary.circle.lat'], 12.121212, 'should be set to point.lat');
t.deepEquals(errorsAndWarnings, {
errors: [],
warnings: ['boundary.circle is currently unsupported']
warnings: ['boundary.circle.lat/boundary.circle.lon are currently unsupported']
}, 'no warnings/errors');
t.end();
});
@ -33,7 +33,7 @@ module.exports.tests.sanitize_boundary_country = function(test, common) {
t.equals(clean['boundary.circle.lon'], 21.212121, 'should be set to point.lon');
t.deepEquals(errorsAndWarnings, {
errors: [],
warnings: ['boundary.circle is currently unsupported']
warnings: ['boundary.circle.lat/boundary.circle.lon are currently unsupported']
}, 'no warnings/errors');
t.end();
});
@ -50,7 +50,7 @@ module.exports.tests.sanitize_boundary_country = function(test, common) {
// t.equals(clean['boundary.circle.radius'], 12.121212, 'should be set to point.lat')
t.deepEquals(errorsAndWarnings, {
errors: [],
warnings: ['boundary.circle is currently unsupported']
warnings: []
}, 'no warnings/errors');
t.end();
});

Loading…
Cancel
Save