This adds new syntax options for expressing a geometry in the BDD tests:

```
[ <geometry>; ... ]
```

will mean that you expect a MULTI* geometry. The geometries need to be of the 
same type, obviously.

```
{ <geometry>; ... }
```

means to expect a GEOMETRYCOLLECTION. The inner geometries can be arbitrary 
simple types. No multi geometries, no collections.

This makes the tests involving multi geometries and collections considerably 
simpler. It incidentally also works around a [bug with ST_GeometryN in PostGIS 
3.6.0](https://trac.osgeo.org/postgis/ticket/5987).

NB: also changes the CI to using pbf for tests as xml files are not offered 
anymore by Geofabrik.
You can view, comment on, or merge this pull request online at:

  https://github.com/osm2pgsql-dev/osm2pgsql/pull/2417

-- Commit Summary --

  * allow BDD tests involving multigeometries
  * use PBF for testing

-- File Changes --

    M .github/workflows/ci.yml (7)
    M tests/bdd/flex/geometry-collection.feature (14)
    M tests/bdd/flex/geometry-linestring.feature (6)
    M tests/bdd/flex/geometry-multilinestring.feature (16)
    M tests/bdd/flex/geometry-multipoint.feature (10)
    M tests/bdd/steps/steps_db.py (93)

-- Patch Links --

https://github.com/osm2pgsql-dev/osm2pgsql/pull/2417.patch
https://github.com/osm2pgsql-dev/osm2pgsql/pull/2417.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/osm2pgsql-dev/osm2pgsql/pull/2417
You are receiving this because you are subscribed to this thread.

Message ID: <osm2pgsql-dev/osm2pgsql/pull/[email protected]>
_______________________________________________
Tile-serving mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/tile-serving

Reply via email to