Merge pull request #538 from stesie/feature/ci-test-on-alpine

add ci test on alpine
phpv8 · Sep 29, 2024 · 5234490 · 5234490
2 parents bf319f7 + eaf4b73
commit 5234490
Show file tree

Hide file tree

Showing 2 changed files with 49 additions and 1 deletion.
diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml
@@ -94,3 +94,45 @@ jobs:
                     ./configure --with-v8js=/opt/v8/self-built LDFLAGS="-lstdc++" CPPFLAGS="-DV8_COMPRESS_POINTERS -DV8_ENABLE_SANDBOX"
                     make
                     make test
+
+            -   name: Archive test results
+                if: failure()
+                uses: actions/upload-artifact@v4
+                with:
+                    name: phpt-test-results
+                    path: |
+                        php_test_results*.txt
+                        tests/*.out
+
+    alpine:
+        runs-on: ubuntu-latest
+
+        steps:
+            -   name: Checkout code
+                uses: actions/checkout@v2
+
+            -   name: Setup latest Alpine Linux
+                uses: jirutka/setup-alpine@v1
+
+            -   name: Install dependencies
+                run: |
+                    cat /etc/alpine-release
+                    apk add php83-dev nodejs-dev g++ make
+                shell: alpine.sh --root {0}
+
+            -   name: Build extension
+                run: |
+                    phpize
+                    ./configure
+                    make
+                    make test
+                shell: alpine.sh {0}
+
+            -   name: Archive test results
+                if: failure()
+                uses: actions/upload-artifact@v4
+                with:
+                    name: phpt-test-results
+                    path: |
+                        php_test_results*.txt
+                        tests/*.out
diff --git a/tests/array_access_002.phpt b/tests/array_access_002.phpt
@@ -1,7 +1,13 @@
 --TEST--
 Test V8::executeString() : Use ArrayAccess with JavaScript native push method
 --SKIPIF--
-<?php require_once(dirname(__FILE__) . '/skipif.inc'); ?>
+<?php require_once(dirname(__FILE__) . '/skipif.inc');
+
+if (str_starts_with(V8Js::V8_VERSION, '11.3.244.8')) {
+    die("skip V8 version known to call setter twice");
+}
+
+?>
 --INI--
 v8js.use_array_access = 1
 --FILE--