From ed9d10e96c8a2c6527bb7efabfa66bef25f8e829 Mon Sep 17 00:00:00 2001
From: Michael Krasnyk <michael.krasnyk@gmail.com>
Date: Thu, 22 Dec 2016 23:17:59 +0100
Subject: [PATCH] Tests for profile API version

---
 features/options/profiles/version0.feature | 106 +++++++++++++++++++++
 features/step_definitions/options.js       |   5 +-
 2 files changed, 109 insertions(+), 2 deletions(-)
 create mode 100644 features/options/profiles/version0.feature

diff --git a/features/options/profiles/version0.feature b/features/options/profiles/version0.feature
new file mode 100644
index 000000000..5e173a338
--- /dev/null
+++ b/features/options/profiles/version0.feature
@@ -0,0 +1,106 @@
+Feature: Profile API version 0
+
+    Background:
+        Given a grid size of 100 meters
+
+    Scenario: Not-defined API version
+        Given the profile file
+          """
+function way_function(way, result)
+  result.forward_mode = mode.driving
+  result.forward_speed = 1
+end
+          """
+        And the node map
+          """
+            ab
+          """
+        And the ways
+            | nodes  |
+            | ab     |
+        And the data has been saved to disk
+
+        When I try to run "osrm-extract --profile {profile_file} {osm_file}"
+        Then it should exit successfully
+        And stderr should not contain "Invalid profile API version"
+
+    Scenario: Out-bound API version
+        Given the profile file
+          """
+api_version = 1
+          """
+        And the node map
+          """
+            ab
+          """
+        And the ways
+            | nodes  |
+            | ab     |
+        And the data has been saved to disk
+
+        When I try to run "osrm-extract --profile {profile_file} {osm_file}"
+        Then it should exit with an error
+        And stderr should contain "Invalid profile API version"
+
+
+    Scenario: Basic profile function calls and property values
+        Given the profile file
+           """
+api_version = 0
+
+-- set profile properties
+properties.u_turn_penalty                  = 20
+properties.traffic_signal_penalty          = 2
+properties.max_speed_for_map_matching      = 180/3.6
+properties.use_turn_restrictions           = true
+properties.continue_straight_at_waypoint   = true
+properties.left_hand_driving               = false
+
+function node_function (node, result)
+  print ('node_function ' .. node:id())
+end
+
+function way_function(way, result)
+  result.name = way:get_value_by_key('name')
+  result.forward_mode = mode.driving
+  result.backward_mode = mode.driving
+  result.forward_speed = 36
+  result.backward_speed = 36
+  print ('way_function ' .. way:id() .. ' ' .. result.name)
+end
+
+function turn_function (angle)
+  print('turn_function ' .. angle)
+  return angle == 0 and 0 or 42
+end
+
+function segment_function (source, target, distance, weight)
+    print ('segment_function ' .. source.lon .. ' ' .. source.lat)
+end
+           """
+        And the node map
+           """
+               a
+              bcd
+               e
+           """
+        And the ways
+            | nodes  |
+            | ac     |
+            | cb     |
+            | cd     |
+            | ce     |
+        And the data has been saved to disk
+
+        When I run "osrm-extract --profile {profile_file} {osm_file}"
+        Then it should exit successfully
+        And stdout should contain "node_function"
+        And stdout should contain "way_function"
+        And stdout should contain "turn_function"
+        And stdout should contain "segment_function"
+
+        When I route I should get
+           | from | to | route    | time  |
+           | a    | b  | ac,cb,cb | 19.2s |
+           | a    | d  | ac,cd,cd | 19.2s |
+           | a    | e  | ac,ce,ce | 20s   |
diff --git a/features/step_definitions/options.js b/features/step_definitions/options.js
index a6942e9fb..47273aa16 100644
--- a/features/step_definitions/options.js
+++ b/features/step_definitions/options.js
@@ -70,8 +70,9 @@ module.exports = function () {
         assert.ok(this.stdout.indexOf(str) > -1);
     });
 
-    this.Then(/^stderr should contain "(.*?)"$/, (str) => {
-        assert.ok(this.stderr.indexOf(str) > -1);
+    this.Then(/^stderr should( not)? contain "(.*?)"$/, (not, str) => {
+        const contains = this.stderr.indexOf(str) > -1;
+        assert.ok(typeof not === 'undefined' ? contains : !contains);
     });
 
     this.Then(/^stdout should contain \/(.*)\/$/, (regexStr) => {