[Tile-serving] [openstreetmap/osm2pgsql] Convert next batch of tests to BDD tests (PR #1813)

Jochen Topf notifications at github.com
Fri Nov 4 20:04:11 UTC 2022


@joto requested changes on this pull request.

I believe the checks from test-output-flex-way-relation-del.cpp are missing.

> +            | -a     |
+            | --slim |
+
+        Then table osm2pgsql_test_t1 contains exactly
+            | way_id |
+            | 11     |
+        Then table osm2pgsql_test_t2 contains exactly
+            | way_id |
+            | 12     |
+        Then table osm2pgsql_test_tboth contains exactly
+            | way_id |
+            | 13     |
+            | 14     |
+
+
+    Scenario: add to t2

should be `add to t1`

> +            | 10     |
+            | 11     |
+        Then table osm2pgsql_test_t2 contains exactly
+            | way_id |
+            | 12     |
+        Then table osm2pgsql_test_tboth contains exactly
+            | way_id |
+            | 13     |
+            | 14     |
+
+        Given the OSM data
+            """
+            <input>
+            """
+        When running osm2pgsql flex with parameters
+            | --slim | -a |

Consistently align command line options vertically or horizontally?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/1813#pullrequestreview-1168928818
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/osm2pgsql/pull/1813/review/1168928818 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20221104/813418b6/attachment.htm>


More information about the Tile-serving mailing list