Open-Harbor/osrm-backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

Avoid doubled reprocessing for scenarios without caching

Browse Source
This commit is contained in:
Michael Krasnyk 2016-09-30 00:11:04 +02:00 committed by Patrick Niklaus
parent 9a85a50586
commit da15c014f5
3 changed files with 26 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
features/step_definitions/data.js
View File

@ -265,7 +265,7 @@ module.exports = function () {
});
this.Given(/^the data has been saved to disk$/, (callback) => {
this.reprocess(callback);
this.writeAndLinkOSM(callback);
});
this.Given(/^the data has been extracted$/, (callback) => {

24
features/step_definitions/options.js
View File

@ -1,5 +1,7 @@
var assert = require('assert');
var fs = require('fs');
'use strict';
const assert = require('assert');
const fs = require('fs');
module.exports = function () {
this.resetOptionsOutput = () => {
@ -24,11 +26,19 @@ module.exports = function () {
});
this.When(/^I run "osrm\-extract\s?(.*?)"$/, (options, callback) => {
this.runAndSafeOutput('osrm-extract', options, callback);
const stamp = this.processedCacheFile + '.extract';
this.runAndSafeOutput('osrm-extract', options, (err) => {
if (err) return callback(err);
fs.writeFile(stamp, 'ok', callback);
});
});
this.When(/^I run "osrm\-contract\s?(.*?)"$/, (options, callback) => {
this.runAndSafeOutput('osrm-contract', options, callback);
const stamp = this.processedCacheFile + '.contract';
this.runAndSafeOutput('osrm-contract', options, (err) => {
if (err) return callback(err);
fs.writeFile(stamp, 'ok', callback);
});
});
this.When(/^I try to run "osrm\-routed\s?(.*?)"$/, (options, callback) => {
@ -65,12 +75,12 @@ module.exports = function () {
});
this.Then(/^stdout should contain \/(.*)\/$/, (regexStr) => {
var re = new RegExp(regexStr);
const re = new RegExp(regexStr);
assert.ok(this.stdout.match(re));
});
this.Then(/^stderr should contain \/(.*)\/$/, (regexStr) => {
var re = new RegExp(regexStr);
const re = new RegExp(regexStr);
assert.ok(this.stdout.match(re));
});
@ -91,7 +101,7 @@ module.exports = function () {
});
this.Then(/^datasource names should contain "(.+)"$/, (expectedData) => {
var actualData = fs.readFileSync(this.processedCacheFile + '.datasource_names', {encoding:'UTF-8'}).trim().split('\n').join(',');
const actualData = fs.readFileSync(this.processedCacheFile + '.datasource_names', {encoding:'UTF-8'}).trim().split('\n').join(',');
assert.equal(actualData, expectedData);
});

11
features/support/data.js
View File

@ -204,18 +204,23 @@ module.exports = function () {
queue.awaitAll(callback);
};
this.reprocess = (callback) => {
this.writeAndLinkOSM = (callback) => {
let queue = d3.queue(1);
queue.defer(this.writeOSM.bind(this));
queue.defer(this.linkOSM.bind(this));
queue.awaitAll(callback);
};
this.reprocess = (callback) => {
let queue = d3.queue(1);
queue.defer(this.writeAndLinkOSM.bind(this));
queue.defer(this.extractAndContract.bind(this));
queue.awaitAll(callback);
};
this.reprocessAndLoadData = (callback) => {
let queue = d3.queue(1);
queue.defer(this.writeOSM.bind(this));
queue.defer(this.linkOSM.bind(this));
queue.defer(this.writeAndLinkOSM.bind(this));
queue.defer(this.extractAndContract.bind(this));
queue.defer(this.osrmLoader.load.bind(this.osrmLoader), this.processedCacheFile);
queue.awaitAll(callback);