Merge pull request #576 from terrestris/tests

fix: tests and introduce pipeline for v5-main branch

Author: ahennr

Diff for: .github/workflows/on-pull-request.yml

@@ -0,0 +1,38 @@
+name: Test successful build of mapfish print manager (v5-main)
+
+on: pull_request
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [18.x]
+
+    steps:
+      - name: Checkout sources
+        uses: actions/checkout@v3
+
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node-version }}
+
+      - name: Cache Node.js modules 💾
+        uses: actions/cache@v3
+        with:
+          path: ~/.npm
+          key: "${{ runner.OS }}-node-${{ hashFiles('**/package-lock.json') }}"
+          restore-keys: |
+            ${{ runner.OS }}-node-
+            ${{ runner.OS }}-
+
+      - name: Install dependencies ⏬
+        run: npm install
+
+      - name: Run tests 🩺
+        run: npm run test
+
+      - name: Build artifacts 🏗️
+        run: npm run build:dist

Diff for: spec/serializer/MapFishPrintV3GeoJsonSerializer.spec.js

@@ -79,6 +79,7 @@ describe('MapFishPrintV3GeoJsonSerializer', () => {
             coordinates: [0, 0]
           },
           properties: {
+            '_style': 0
           }
         }, {
           type: 'Feature',
@@ -87,6 +88,7 @@ describe('MapFishPrintV3GeoJsonSerializer', () => {
             coordinates: [[0, 0], [1, 1]]
           },
           properties: {
+            '_style': 1
           }
         }, {
           type: 'Feature',
@@ -95,12 +97,37 @@ describe('MapFishPrintV3GeoJsonSerializer', () => {
             coordinates: [[[0, 0], [1, 1], [0, 0]]]
           },
           properties: {
+            '_style': 2
           }
         }]
       },
       name: layer.get('name') || 'Vector Layer',
       opacity: layer.getOpacity(),
-      style: {},
+      style: {
+        '0': {
+          fillColor: '#ffffff',
+          fillOpacity: 0.4,
+          graphicName: 'circle',
+          pointRadius: 5,
+          rotation: 0,
+          strokeColor: '#3399cc',
+          strokeOpacity: 1,
+          strokeWidth: 1.25,
+          version: 2,
+        },
+        '1': {
+          strokeColor: '#3399cc',
+          strokeOpacity: 1,
+          strokeWidth: 1.25,
+        },
+        '2': {
+          fillColor: '#ffffff',
+          fillOpacity: 0.4,
+          strokeColor: '#3399cc',
+          strokeOpacity: 1,
+          strokeWidth: 1.25,
+        },
+      },
       type: 'geojson'
     });
   });