CalvinBackup/iD
1
0
Fork 0
mirror of https://github.com/FoggedLens/iD.git synced 2026-05-14 13:18:15 +02:00
Code Issues Packages Projects Releases Wiki Activity

Debounce should debounce all requests even cached, fix tests

Browse Source 
(see https://github.com/openstreetmap/iD/pull/3975#issue-222747308)
This commit is contained in:
Bryan Housel
2017-04-19 13:13:12 -04:00
parent 494dd979a9
commit 62d0bc0bf5
2 changed files with 235 additions and 177 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/services/taginfo.js
+99 -116
View File
@@ -1,6 +1,6 @@
import * as d3 from 'd3';
import _ from 'lodash';
import { utilQsString } from '../util/index';
import { utilQsString } from '../util';
var endpoint = 'https://taginfo.openstreetmap.org/api/4/',
@@ -34,31 +34,31 @@ var endpoint = 'https://taginfo.openstreetmap.org/api/4/',
};
function sets(parameters, n, o) {
if (parameters.geometry && o[parameters.geometry]) {
parameters[n] = o[parameters.geometry];
function sets(params, n, o) {
if (params.geometry && o[params.geometry]) {
params[n] = o[params.geometry];
}
return parameters;
return params;
}
function setFilter(parameters) {
return sets(parameters, 'filter', tag_filters);
function setFilter(params) {
return sets(params, 'filter', tag_filters);
}
function setSort(parameters) {
return sets(parameters, 'sortname', tag_sorts);
function setSort(params) {
return sets(params, 'sortname', tag_sorts);
}
function setSortMembers(parameters) {
return sets(parameters, 'sortname', tag_sort_members);
function setSortMembers(params) {
return sets(params, 'sortname', tag_sort_members);
}
function clean(parameters) {
return _.omit(parameters, 'geometry', 'debounce');
function clean(params) {
return _.omit(params, 'geometry', 'debounce');
}
@@ -130,23 +130,18 @@ function sortKeys(a, b) {
}
var debounced = _.debounce(d3.json, 750, true);
var debouncedRequest = _.debounce(request, 750, { leading: false });
function request(url, debounce, callback) {
function request(url, params, callback) {
if (taginfoCache[url]) {
callback(null, taginfoCache[url]);
} else if (debounce) {
debounced(url, done);
} else {
d3.json(url, done);
}
function done(err, data) {
if (!err) {
taginfoCache[url] = data;
}
callback(err, data);
d3.json(url, function (err, data) {
if (!err) {
taginfoCache[url] = data;
}
callback(err, data);
});
}
}
@@ -160,13 +155,8 @@ export default {
// Fetch popular keys. We'll exclude these from `values`
// lookups because they stress taginfo, and they aren't likely
// to yield meaningful autocomplete results.. see #3955
this.keys({
rp: 100,
sortname: 'values_all',
sortorder: 'desc',
page: 1,
debounce: false
}, function(err, data) {
var params = { rp: 100, sortname: 'values_all', sortorder: 'desc', page: 1, debounce: false };
this.keys(params, function(err, data) {
if (err) return;
data.forEach(function(d) {
if (d === 'opening_hours') return; // exception
@@ -179,114 +169,107 @@ export default {
reset: function() { },
keys: function(parameters, callback) {
var debounce = parameters.debounce;
parameters = clean(setSort(parameters));
request(endpoint + 'keys/all?' +
utilQsString(_.extend({
rp: 10,
sortname: 'count_all',
sortorder: 'desc',
page: 1
}, parameters)), debounce, function(err, d) {
if (err) {
callback(err);
} else {
var f = filterKeys(parameters.filter);
callback(null, d.data.filter(f).sort(sortKeys).map(valKey));
}
}
keys: function(params, callback) {
var req = params.debounce ? debouncedRequest : request;
params = clean(setSort(params));
var qs = utilQsString(
_.extend({ rp: 10, sortname: 'count_all', sortorder: 'desc', page: 1 }, params)
);
req(endpoint + 'keys/all?' + qs, params, function(err, d) {
if (err) {
callback(err);
} else {
var f = filterKeys(params.filter);
callback(null, d.data.filter(f).sort(sortKeys).map(valKey));
}
});
},
multikeys: function(parameters, callback) {
var debounce = parameters.debounce;
parameters = clean(setSort(parameters));
var prefix = parameters.query;
request(endpoint + 'keys/all?' +
utilQsString(_.extend({
rp: 25,
sortname: 'count_all',
sortorder: 'desc',
page: 1
}, parameters)), debounce, function(err, d) {
if (err) {
callback(err);
} else {
var f = filterMultikeys(prefix);
callback(null, d.data.filter(f).map(valKey));
}
}
multikeys: function(params, callback) {
var req = params.debounce ? debouncedRequest : request;
params = clean(setSort(params));
var prefix = params.query;
var qs = utilQsString(
_.extend({ rp: 25, sortname: 'count_all', sortorder: 'desc', page: 1 }, params)
);
req(endpoint + 'keys/all?' + qs, params, function(err, d) {
if (err) {
callback(err);
} else {
var f = filterMultikeys(prefix);
callback(null, d.data.filter(f).map(valKey));
}
});
},
values: function(parameters, callback) {
values: function(params, callback) {
// Exclude popular keys from values lookups.. see #3955
var key = parameters.key;
var key = params.key;
if (key && popularKeys[key]) {
callback(null, []);
return;
}
var debounce = parameters.debounce;
parameters = clean(setSort(setFilter(parameters)));
request(endpoint + 'key/values?' +
utilQsString(_.extend({
rp: 25,
sortname: 'count_all',
sortorder: 'desc',
page: 1
}, parameters)), debounce, function(err, d) {
if (err) {
callback(err);
} else {
// In most cases we prefer taginfo value results with lowercase letters.
// A few OSM keys expect values to contain uppercase values (see #3377).
// This is not an exhaustive list (e.g. `name` also has uppercase values)
// but these are the fields where taginfo value lookup is most useful.
var re = /network|taxon|genus|species|brand|grape_variety|_hours|_times/;
var allowUpperCase = (parameters.key.match(re) !== null);
var f = filterValues(allowUpperCase);
callback(null, d.data.filter(f).map(valKeyDescription));
}
}
var req = params.debounce ? debouncedRequest : request;
params = clean(setSort(setFilter(params)));
var qs = utilQsString(
_.extend({ rp: 25, sortname: 'count_all', sortorder: 'desc', page: 1 }, params)
);
req(endpoint + 'key/values?' + qs, params, function(err, d) {
if (err) {
callback(err);
} else {
// In most cases we prefer taginfo value results with lowercase letters.
// A few OSM keys expect values to contain uppercase values (see #3377).
// This is not an exhaustive list (e.g. `name` also has uppercase values)
// but these are the fields where taginfo value lookup is most useful.
var re = /network|taxon|genus|species|brand|grape_variety|_hours|_times/;
var allowUpperCase = (params.key.match(re) !== null);
var f = filterValues(allowUpperCase);
callback(null, d.data.filter(f).map(valKeyDescription));
}
});
},
roles: function(parameters, callback) {
var debounce = parameters.debounce;
var geometry = parameters.geometry;
parameters = clean(setSortMembers(parameters));
request(endpoint + 'relation/roles?' +
utilQsString(_.extend({
rp: 25,
sortname: 'count_all_members',
sortorder: 'desc',
page: 1
}, parameters)), debounce, function(err, d) {
if (err) {
callback(err);
} else {
var f = filterRoles(geometry);
callback(null, d.data.filter(f).map(roleKey));
}
}
roles: function(params, callback) {
var req = params.debounce ? debouncedRequest : request;
var geometry = params.geometry;
params = clean(setSortMembers(params));
var qs = utilQsString(
_.extend({ rp: 25, sortname: 'count_all_members', sortorder: 'desc', page: 1 }, params)
);
req(endpoint + 'relation/roles?' + qs, params, function(err, d) {
if (err) {
callback(err);
} else {
var f = filterRoles(geometry);
callback(null, d.data.filter(f).map(roleKey));
}
});
},
docs: function(parameters, callback) {
var debounce = parameters.debounce;
parameters = clean(setSort(parameters));
docs: function(params, callback) {
var req = params.debounce ? debouncedRequest : request;
params = clean(setSort(params));
var path = 'key/wiki_pages?';
if (parameters.value) path = 'tag/wiki_pages?';
else if (parameters.rtype) path = 'relation/wiki_pages?';
if (params.value) path = 'tag/wiki_pages?';
else if (params.rtype) path = 'relation/wiki_pages?';
request(endpoint + path + utilQsString(parameters), debounce, function(err, d) {
req(endpoint + path + utilQsString(params), params, function(err, d) {
if (err) {
callback(err);
} else {
test/spec/services/taginfo.js
+136 -61
View File
@@ -4,7 +4,16 @@ describe('iD.serviceTaginfo', function() {
beforeEach(function() {
server = sinon.fakeServer.create();
taginfo = iD.services.taginfo;
taginfo.reset();
// prepopulate popular keys list with "name"
taginfo.init();
server.respondWith('GET',
new RegExp('\/keys\/all.*sortname=values_all'),
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":56136034,"key":"name","count_all_fraction":0.0132}]}']
);
server.respond();
server = sinon.fakeServer.create();
});
afterEach(function() {
@@ -15,55 +24,67 @@ describe('iD.serviceTaginfo', function() {
return iD.utilStringQs(url.substring(url.indexOf('?') + 1));
}
describe('#keys', function() {
it('calls the given callback with the results of the keys query', function() {
var callback = sinon.spy();
taginfo.keys({query: 'amen'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":5190337,"key":"amenity","count_all_fraction":1.0}]}']);
'{"data":[{"count_all":5190337,"key":"amenity","count_all_fraction":1.0}]}']
);
server.respond();
expect(query(server.requests[0].url)).to.eql(
{query: 'amen', page: '1', rp: '10', sortname: 'count_all', sortorder: 'desc'});
expect(callback).to.have.been.calledWith(null, [{'title':'amenity', 'value':'amenity'}]);
{query: 'amen', page: '1', rp: '10', sortname: 'count_all', sortorder: 'desc'}
);
expect(callback).to.have.been.calledWith(
null, [{'title':'amenity', 'value':'amenity'}]
);
});
it('includes popular keys', function() {
var callback = sinon.spy();
taginfo.keys({query: 'amen'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":5190337,"key":"amenity","count_all_fraction":1.0, "count_nodes_fraction":1.0},'
+ '{"count_all":1,"key":"amenityother","count_all_fraction":0.0, "count_nodes_fraction":0.0}]}']);
'{"data":[{"count_all":5190337,"key":"amenity","count_all_fraction":1.0,"count_nodes_fraction":1.0},'
+ '{"count_all":1,"key":"amenityother","count_all_fraction":0.0,"count_nodes_fraction":0.0}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'title':'amenity', 'value':'amenity'}]);
expect(callback).to.have.been.calledWith(
null, [{'title':'amenity', 'value':'amenity'}]
);
});
it('includes popular keys with an entity type filter', function() {
var callback = sinon.spy();
taginfo.keys({query: 'amen', filter: 'nodes'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":5190337,"count_nodes":500000,"key":"amenity","count_all_fraction":1.0, "count_nodes_fraction":1.0},'
+ '{"count_all":1,"key":"amenityother","count_all_fraction":0.0, "count_nodes":100}]}']);
+ '{"count_all":1,"key":"amenityother","count_all_fraction":0.0, "count_nodes":100}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'title':'amenity', 'value':'amenity'}]);
expect(callback).to.have.been.calledWith(
null, [{'title':'amenity', 'value':'amenity'}]
);
});
it('includes unpopular keys with a wiki page', function() {
var callback = sinon.spy();
taginfo.keys({query: 'amen'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":5190337,"key":"amenity","count_all_fraction":1.0, "count_nodes_fraction":1.0},'
+ '{"count_all":1,"key":"amenityother","count_all_fraction":0.0, "count_nodes_fraction":0.0, "in_wiki": true}]}']);
+ '{"count_all":1,"key":"amenityother","count_all_fraction":0.0, "count_nodes_fraction":0.0, "in_wiki": true}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [
@@ -76,12 +97,16 @@ describe('iD.serviceTaginfo', function() {
var callback = sinon.spy();
taginfo.keys({query: 'ref'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"key":"ref:bag","count_all":9790586,"count_all_fraction":0.0028},{"key":"ref","count_all":7933528,"count_all_fraction":0.0023}]}']);
'{"data":[{"key":"ref:bag","count_all":9790586,"count_all_fraction":0.0028},' +
'{"key":"ref","count_all":7933528,"count_all_fraction":0.0023}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'title':'ref', 'value':'ref'},{'title':'ref:bag', 'value':'ref:bag'}]);
expect(callback).to.have.been.calledWith(
null, [{'title':'ref', 'value':'ref'},{'title':'ref:bag', 'value':'ref:bag'}]
);
});
});
@@ -90,40 +115,50 @@ describe('iD.serviceTaginfo', function() {
var callback = sinon.spy();
taginfo.multikeys({query: 'recycling:'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":69593,"key":"recycling:glass","count_all_fraction":0.0}]}']);
'{"data":[{"count_all":69593,"key":"recycling:glass","count_all_fraction":0.0}]}']
);
server.respond();
expect(query(server.requests[0].url)).to.eql(
{query: 'recycling:', page: '1', rp: '25', sortname: 'count_all', sortorder: 'desc'});
expect(callback).to.have.been.calledWith(null, [{'title':'recycling:glass', 'value':'recycling:glass'}]);
{query: 'recycling:', page: '1', rp: '25', sortname: 'count_all', sortorder: 'desc'}
);
expect(callback).to.have.been.calledWith(
null, [{'title':'recycling:glass', 'value':'recycling:glass'}]
);
});
it('excludes multikeys with extra colons', function() {
var callback = sinon.spy();
taginfo.multikeys({query: 'service:bicycle:'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":4426,"key":"service:bicycle:retail","count_all_fraction":0.0},'
+ '{"count_all":22,"key":"service:bicycle:retail:ebikes","count_all_fraction":0.0}]}']);
'{"data":[{"count_all":4426,"key":"service:bicycle:retail","count_all_fraction":0.0},' +
'{"count_all":22,"key":"service:bicycle:retail:ebikes","count_all_fraction":0.0}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'title':'service:bicycle:retail', 'value':'service:bicycle:retail'}]);
expect(callback).to.have.been.calledWith(
null, [{'title':'service:bicycle:retail', 'value':'service:bicycle:retail'}]
);
});
it('excludes multikeys with wrong prefix', function() {
var callback = sinon.spy();
taginfo.multikeys({query: 'service:bicycle:'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/keys/all'),
server.respondWith('GET', /\/keys\/all/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"count_all":4426,"key":"service:bicycle:retail","count_all_fraction":0.0},'
+ '{"count_all":22,"key":"disused:service:bicycle","count_all_fraction":0.0}]}']);
'{"data":[{"count_all":4426,"key":"service:bicycle:retail","count_all_fraction":0.0},' +
'{"count_all":22,"key":"disused:service:bicycle","count_all_fraction":0.0}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'title':'service:bicycle:retail', 'value':'service:bicycle:retail'}]);
expect(callback).to.have.been.calledWith(
null, [{'title':'service:bicycle:retail', 'value':'service:bicycle:retail'}]
);
});
});
@@ -132,37 +167,59 @@ describe('iD.serviceTaginfo', function() {
var callback = sinon.spy();
taginfo.values({key: 'amenity', query: 'par'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"parking","description":"A place for parking cars", "fraction":0.1}]}']);
'{"data":[{"value":"parking","description":"A place for parking cars", "fraction":0.1}]}']
);
server.respond();
expect(query(server.requests[0].url)).to.eql(
{key: 'amenity', query: 'par', page: '1', rp: '25', sortname: 'count_all', sortorder: 'desc'});
expect(callback).to.have.been.calledWith(null, [{'value':'parking','title':'A place for parking cars'}]);
{key: 'amenity', query: 'par', page: '1', rp: '25', sortname: 'count_all', sortorder: 'desc'}
);
expect(callback).to.have.been.calledWith(
null, [{'value':'parking','title':'A place for parking cars'}]
);
});
it('includes popular values', function() {
var callback = sinon.spy();
taginfo.values({key: 'amenity', query: 'par'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"parking","description":"A place for parking cars", "fraction":1.0},' +
'{"value":"party","description":"A place for partying", "fraction":0.0}]}']);
'{"value":"party","description":"A place for partying", "fraction":0.0}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'value':'parking','title':'A place for parking cars'}]);
expect(callback).to.have.been.calledWith(
null, [{'value':'parking','title':'A place for parking cars'}]
);
});
it('does not get values for extremely popular keys', function() {
var callback = sinon.spy();
taginfo.values({key: 'name', query: 'ste'}, callback);
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"Rue Pasteur","description":"", "fraction":0.0001},' +
'{"value":"Via Trieste","description":"", "fraction":0.0001}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, []);
});
it('includes unpopular values with a wiki page', function() {
var callback = sinon.spy();
taginfo.values({key: 'amenity', query: 'par'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"parking","description":"A place for parking cars", "fraction":1.0},'
+ '{"value":"party","description":"A place for partying", "fraction":0.0, "in_wiki": true}]}']);
'{"data":[{"value":"parking","description":"A place for parking cars", "fraction":1.0},' +
'{"value":"party","description":"A place for partying", "fraction":0.0, "in_wiki": true}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [
@@ -175,29 +232,33 @@ describe('iD.serviceTaginfo', function() {
var callback = sinon.spy();
taginfo.values({key: 'amenity', query: 'par'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"parking","description":"A place for parking cars", "fraction":0.2},'
+ '{"value":"PArking","description":"A common mispelling", "fraction":0.2},'
+ '{"value":"parking;partying","description":"A place for parking cars *and* partying", "fraction":0.2},'
+ '{"value":"parking, partying","description":"A place for parking cars *and* partying", "fraction":0.2},'
+ '{"value":"*","description":"", "fraction":0.2}]}']);
+ '{"value":"*","description":"", "fraction":0.2}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'value':'parking','title':'A place for parking cars'}]);
expect(callback).to.have.been.calledWith(
null, [{'value':'parking','title':'A place for parking cars'}]
);
});
it('includes network values with capital letters and some punctuation', function() {
var callback = sinon.spy();
taginfo.values({key: 'network', query: 'us'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"US:TX:FM","description":"Farm to Market Roads in the U.S. state of Texas.", "fraction":0.34},'
+ '{"value":"US:KY","description":"Primary and secondary state highways in the U.S. state of Kentucky.", "fraction":0.31},'
+ '{"value":"US:US","description":"U.S. routes in the United States.", "fraction":0.19},'
+ '{"value":"US:I","description":"Interstate highways in the United States.", "fraction":0.11},'
+ '{"value":"US:MD","description":"State highways in the U.S. state of Maryland.", "fraction":0.06}]}']);
+ '{"value":"US:MD","description":"State highways in the U.S. state of Maryland.", "fraction":0.06}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [
@@ -213,36 +274,45 @@ describe('iD.serviceTaginfo', function() {
var callback = sinon.spy();
taginfo.values({key: 'genus', query: 'qu'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"Quercus","description":"Oak", "fraction":0.5}]}']);
'{"data":[{"value":"Quercus","description":"Oak", "fraction":0.5}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'value':'Quercus','title':'Oak'}]);
expect(callback).to.have.been.calledWith(
null, [{'value':'Quercus','title':'Oak'}]
);
});
it('includes biological taxon values with capital letters', function() {
var callback = sinon.spy();
taginfo.values({key: 'taxon', query: 'qu'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"Quercus robur","description":"Oak", "fraction":0.5}]}']);
'{"data":[{"value":"Quercus robur","description":"Oak", "fraction":0.5}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'value':'Quercus robur','title':'Oak'}]);
expect(callback).to.have.been.calledWith(
null, [{'value':'Quercus robur','title':'Oak'}]
);
});
it('includes biological species values with capital letters', function() {
var callback = sinon.spy();
taginfo.values({key: 'species', query: 'qu'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/key/values'),
server.respondWith('GET', /\/key\/values/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"value":"Quercus robur","description":"Oak", "fraction":0.5}]}']);
'{"data":[{"value":"Quercus robur","description":"Oak", "fraction":0.5}]}']
);
server.respond();
expect(callback).to.have.been.calledWith(null, [{'value':'Quercus robur','title':'Oak'}]);
expect(callback).to.have.been.calledWith(
null, [{'value':'Quercus robur','title':'Oak'}]
);
});
});
@@ -251,14 +321,16 @@ describe('iD.serviceTaginfo', function() {
var callback = sinon.spy();
taginfo.roles({rtype: 'route', query: 's', geometry: 'relation'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/relation/roles'),
server.respondWith('GET', /\/relation\/roles/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"role":"stop","count_relation_members_fraction":0.1757},' +
'{"role":"south","count_relation_members_fraction":0.0035}]}']);
'{"role":"south","count_relation_members_fraction":0.0035}]}']
);
server.respond();
expect(query(server.requests[0].url)).to.eql(
{rtype: 'route', query: 's', page: '1', rp: '25', sortname: 'count_relation_members', sortorder: 'desc'});
{rtype: 'route', query: 's', page: '1', rp: '25', sortname: 'count_relation_members', sortorder: 'desc'}
);
expect(callback).to.have.been.calledWith(null, [
{'value': 'stop', 'title': 'stop'},
{'value': 'south', 'title': 'south'}
@@ -271,15 +343,18 @@ describe('iD.serviceTaginfo', function() {
var callback = sinon.spy();
taginfo.docs({key: 'amenity', value: 'parking'}, callback);
server.respondWith('GET', new RegExp('https://taginfo.openstreetmap.org/api/4/tag/wiki_page'),
server.respondWith('GET', /\/tag\/wiki_page/,
[200, { 'Content-Type': 'application/json' },
'{"data":[{"on_way":false,"lang":"en","on_area":true,"image":"File:Car park2.jpg"}]}']);
'{"data":[{"on_way":false,"lang":"en","on_area":true,"image":"File:Car park2.jpg"}]}']
);
server.respond();
expect(query(server.requests[0].url)).to.eql(
{key: 'amenity', value: 'parking'});
expect(callback).to.have.been.calledWith(null,
[{'on_way':false,'lang':'en','on_area':true,'image':'File:Car park2.jpg'}]);
{key: 'amenity', value: 'parking'}
);
expect(callback).to.have.been.calledWith(
null, [{'on_way':false,'lang':'en','on_area':true,'image':'File:Car park2.jpg'}]
);
});
});