Skip to content

Commit 6de4c1e

Browse files
added temppath (#791)
* added temppath * fix: python version * fix: mapfile --------- Co-authored-by: Stefan van Beusekom <stefan.van.beusekom@amsterdam.nl>
1 parent 3221f95 commit 6de4c1e

4 files changed

Lines changed: 682 additions & 51 deletions

File tree

.github/workflows/test.yml

Lines changed: 50 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -13,61 +13,61 @@ jobs:
1313
check_mapfiles:
1414
runs-on: ubuntu-latest
1515
steps:
16-
- uses: actions/checkout@v3
17-
- name: Set up Python
18-
uses: actions/setup-python@v4
19-
with:
20-
python-version: 3.11
21-
cache: "pip"
22-
# Some mapfiles are generated by scripts, but also committed to the repo.
23-
# Check for mismatches between committed files and script output.
24-
- name: Regenerate mapfiles and check for equality
25-
run: |
26-
for script in gen_*.py; do
27-
mapfile=$(echo $script | sed 's/^gen_//; s/\.py$//').map
28-
if [ ! -f "$mapfile" ]; then
29-
mapfile="private/$mapfile"
16+
- uses: actions/checkout@v3
17+
- name: Set up Python
18+
uses: actions/setup-python@v4
19+
with:
20+
python-version: 3.13
21+
cache: "pip"
22+
# Some mapfiles are generated by scripts, but also committed to the repo.
23+
# Check for mismatches between committed files and script output.
24+
- name: Regenerate mapfiles and check for equality
25+
run: |
26+
for script in gen_*.py; do
27+
mapfile=$(echo $script | sed 's/^gen_//; s/\.py$//').map
3028
if [ ! -f "$mapfile" ]; then
31-
echo "Result of ${script} not committed"
32-
exit 1
29+
mapfile="private/$mapfile"
30+
if [ ! -f "$mapfile" ]; then
31+
echo "Result of ${script} not committed"
32+
exit 1
33+
fi
3334
fi
34-
fi
35-
python3 "$script" | cmp - "$mapfile"
36-
done
37-
# The index.json file is not committed, but generated by the Docker
38-
# entrypoint. It should at least be valid JSON.
39-
- name: Check index.json file
40-
run: |
41-
# Create some bogus connection files, or including them will fail:
42-
mkdir -p connections
43-
for c in bag basiskaart dataselectie dataservices panorama; do
44-
echo CONNECTIONTYPE postgis > connection/$c.inc
45-
done
46-
python3 tools/make_indexjson.py *.map | jq . > /dev/null
47-
- name: Authorization checks
48-
run: |
49-
python3 -m pip install -r tools/requirements.txt
50-
# adresseerbare_objecten and bgt_inrichtingselementen can't be parsed
51-
# due to limitations in mappyfile.
52-
# The other excluded maps should be fixed. They have workarounds in
53-
# place to prevent access to restricted fields.
54-
python3 tools/auth_check.py -e \
55-
adresseerbare_objecten bgt_inrichtingselementen \
56-
bomen brk2 ondergrond ziektenplagenexotengroen \
57-
meldingen_huidige_jaar meldingen_vorige_jaar meldingen_indicatoren \
58-
monumenten hr handelsregister hr_kvk eigendommen_v2 erfpacht_v2
35+
python3 "$script" | cmp - "$mapfile"
36+
done
37+
# The index.json file is not committed, but generated by the Docker
38+
# entrypoint. It should at least be valid JSON.
39+
- name: Check index.json file
40+
run: |
41+
# Create some bogus connection files, or including them will fail:
42+
mkdir -p connections
43+
for c in bag basiskaart dataselectie dataservices panorama; do
44+
echo CONNECTIONTYPE postgis > connection/$c.inc
45+
done
46+
python3 tools/make_indexjson.py *.map | jq . > /dev/null
47+
- name: Authorization checks
48+
run: |
49+
python3 -m pip install -r tools/requirements.txt
50+
# adresseerbare_objecten and bgt_inrichtingselementen can't be parsed
51+
# due to limitations in mappyfile.
52+
# The other excluded maps should be fixed. They have workarounds in
53+
# place to prevent access to restricted fields.
54+
python3 tools/auth_check.py -e \
55+
adresseerbare_objecten bgt_inrichtingselementen \
56+
bomen brk2 ondergrond ziektenplagenexotengroen \
57+
meldingen_huidige_jaar meldingen_vorige_jaar meldingen_indicatoren \
58+
monumenten hr handelsregister hr_kvk eigendommen_v2 erfpacht_v2
5959
6060
# Regression test. Asserts that we don't suffer from the MapServer 7.x
6161
# bounding box bug https://github.com/MapServer/MapServer/issues/6478.
6262
test_bbox:
6363
runs-on: ubuntu-latest
6464
steps:
65-
- uses: actions/checkout@v3
66-
- name: Run MapServer
67-
run: docker compose up -d public &&
68-
sleep 3
69-
- name: Projection test
70-
run: |
71-
curl 'http://localhost:8383/maps/projectiontest?REQUEST=GetFeature&SERVICE=wfs&OUTPUTFORMAT=application/json;%20subtype=geojson;%20charset=utf-8&Typename=feature&version=1.1.0&srsname=urn:ogc:def:crs:EPSG::4326&bbox=4.89337404,52.36483743,4.89351768,52.36489491';
72-
curl 'http://localhost:8383/maps/projectiontest?REQUEST=GetFeature&SERVICE=wfs&OUTPUTFORMAT=application/json;%20subtype=geojson;%20charset=utf-8&Typename=feature&version=1.1.0&srsname=urn:ogc:def:crs:EPSG::4326&bbox=4.89337404,52.36483743,4.89351768,52.36489491' |
73-
jq '.features | if length > 0 then . else halt_error(1) end'
65+
- uses: actions/checkout@v3
66+
- name: Run MapServer
67+
run: docker compose up -d public &&
68+
sleep 3
69+
- name: Projection test
70+
run: |
71+
curl 'http://localhost:8383/maps/projectiontest?REQUEST=GetFeature&SERVICE=wfs&OUTPUTFORMAT=application/json;%20subtype=geojson;%20charset=utf-8&Typename=feature&version=1.1.0&srsname=urn:ogc:def:crs:EPSG::4326&bbox=4.89337404,52.36483743,4.89351768,52.36489491';
72+
curl 'http://localhost:8383/maps/projectiontest?REQUEST=GetFeature&SERVICE=wfs&OUTPUTFORMAT=application/json;%20subtype=geojson;%20charset=utf-8&Typename=feature&version=1.1.0&srsname=urn:ogc:def:crs:EPSG::4326&bbox=4.89337404,52.36483743,4.89351768,52.36489491' |
73+
jq '.features | if length > 0 then . else halt_error(1) end'

compose.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ services:
2929

3030
# Bare bones 'webserver' to behave as public loadbalancer
3131
publicdummy:
32-
image: python:3.11-slim-bullseye
32+
image: python:3.13-slim-bullseye
3333
ports:
3434
- 8000:8000
3535
command: python3 -m http.server 8000

0 commit comments

Comments
 (0)