Browse Source

Merge pull request #949 from pelias/remove-is-service-enabled-predicate

removed unused is_service_enabled predicate
pull/951/head
Stephen K Hess 7 years ago committed by GitHub
parent
commit
078912ea92
  1. 5
      controller/predicates/is_service_enabled.js
  2. 42
      test/unit/controller/predicates/is_service_enabled.js
  3. 1
      test/unit/run.js

5
controller/predicates/is_service_enabled.js

@ -1,5 +0,0 @@
module.exports = (uri) => {
return (request, response) => {
return uri !== undefined;
};
};

42
test/unit/controller/predicates/is_service_enabled.js

@ -1,42 +0,0 @@
'use strict';
const _ = require('lodash');
const is_service_enabled = require('../../../../controller/predicates/is_service_enabled');
module.exports.tests = {};
module.exports.tests.interface = (test, common) => {
test('valid interface', (t) => {
t.equal(typeof is_service_enabled, 'function', 'is_service_enabled is a function');
t.equal(typeof is_service_enabled(), 'function', 'is_service_enabled() is a function');
t.end();
});
};
module.exports.tests.true_conditions = (test, common) => {
test('string uri should return true', (t) => {
t.ok(is_service_enabled('pip uri')());
t.end();
});
};
module.exports.tests.false_conditions = (test, common) => {
test('undefined uri should return false', (t) => {
t.notOk(is_service_enabled()());
t.end();
});
};
module.exports.all = (tape, common) => {
function test(name, testFunction) {
return tape(`GET /is_service_enabled ${name}`, testFunction);
}
for( const testCase in module.exports.tests ){
module.exports.tests[testCase](test, common);
}
};

1
test/unit/run.js

@ -29,7 +29,6 @@ var tests = [
require('./controller/predicates/is_coarse_reverse'),
require('./controller/predicates/is_only_non_admin_layers'),
require('./controller/predicates/is_request_sources_only_whosonfirst'),
require('./controller/predicates/is_service_enabled'),
require('./helper/debug'),
require('./helper/diffPlaces'),
require('./helper/geojsonify'),

Loading…
Cancel
Save