Skip to content

Commit 5234490

Browse files
authored
Merge pull request #538 from stesie/feature/ci-test-on-alpine
add ci test on alpine
2 parents bf319f7 + eaf4b73 commit 5234490

File tree

2 files changed

+49
-1
lines changed

2 files changed

+49
-1
lines changed

.github/workflows/build-test.yml

+42
Original file line numberDiff line numberDiff line change
@@ -94,3 +94,45 @@ jobs:
9494
./configure --with-v8js=/opt/v8/self-built LDFLAGS="-lstdc++" CPPFLAGS="-DV8_COMPRESS_POINTERS -DV8_ENABLE_SANDBOX"
9595
make
9696
make test
97+
98+
- name: Archive test results
99+
if: failure()
100+
uses: actions/upload-artifact@v4
101+
with:
102+
name: phpt-test-results
103+
path: |
104+
php_test_results*.txt
105+
tests/*.out
106+
107+
alpine:
108+
runs-on: ubuntu-latest
109+
110+
steps:
111+
- name: Checkout code
112+
uses: actions/checkout@v2
113+
114+
- name: Setup latest Alpine Linux
115+
uses: jirutka/setup-alpine@v1
116+
117+
- name: Install dependencies
118+
run: |
119+
cat /etc/alpine-release
120+
apk add php83-dev nodejs-dev g++ make
121+
shell: alpine.sh --root {0}
122+
123+
- name: Build extension
124+
run: |
125+
phpize
126+
./configure
127+
make
128+
make test
129+
shell: alpine.sh {0}
130+
131+
- name: Archive test results
132+
if: failure()
133+
uses: actions/upload-artifact@v4
134+
with:
135+
name: phpt-test-results
136+
path: |
137+
php_test_results*.txt
138+
tests/*.out

tests/array_access_002.phpt

+7-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
11
--TEST--
22
Test V8::executeString() : Use ArrayAccess with JavaScript native push method
33
--SKIPIF--
4-
<?php require_once(dirname(__FILE__) . '/skipif.inc'); ?>
4+
<?php require_once(dirname(__FILE__) . '/skipif.inc');
5+
6+
if (str_starts_with(V8Js::V8_VERSION, '11.3.244.8')) {
7+
die("skip V8 version known to call setter twice");
8+
}
9+
10+
?>
511
--INI--
612
v8js.use_array_access = 1
713
--FILE--

0 commit comments

Comments
 (0)