Skip to content

Commit e2530cb

Browse files
committed
Single entrypoint for non-phpt tests
1 parent 66a45d5 commit e2530cb

File tree

11 files changed

+253
-63
lines changed

11 files changed

+253
-63
lines changed

.github/actions/apk/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ runs:
66
run: |
77
set -x
88
9+
OPCACHE_TLS_TESTS_DEPS="clang gcc binutils-gold lld"
10+
911
apk update -q
1012
apk add \
1113
util-linux \
@@ -53,4 +55,5 @@ runs:
5355
postgresql14-dev \
5456
tzdata \
5557
musl-locales \
56-
musl-locales-lang
58+
musl-locales-lang \
59+
$OPCACHE_TLS_TESTS_DEPS

.github/actions/apt-x32/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ runs:
66
run: |
77
set -x
88
9+
OPCACHE_TLS_TESTS_DEPS="gcc clang lld"
10+
911
export DEBIAN_FRONTEND=noninteractive
1012
dpkg --add-architecture i386
1113
apt-get update -y | true
@@ -45,4 +47,5 @@ runs:
4547
re2c \
4648
unzip \
4749
wget \
48-
zlib1g-dev:i386
50+
zlib1g-dev:i386 \
51+
$OPCACHE_TLS_TESTS_DEPS

.github/actions/apt-x64/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@ runs:
1010
run: |
1111
set -x
1212
13+
OPCACHE_TLS_TESTS_DEPS="gcc clang lld"
14+
1315
export DEBIAN_FRONTEND=noninteractive
1416
1517
# Install sudo in Docker for consistent actions
@@ -74,4 +76,5 @@ runs:
7476
libqdbm-dev \
7577
libjpeg-dev \
7678
libpng-dev \
77-
libfreetype6-dev
79+
libfreetype6-dev \
80+
$OPCACHE_TLS_TESTS_DEPS \
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
name: Extra tests
2+
inputs:
3+
debug:
4+
default: 'false'
5+
required: false
6+
os:
7+
default: ''
8+
required: false
9+
runs:
10+
using: composite
11+
steps:
12+
- shell: sh
13+
run: |
14+
TEST_PHP_OS=${{ inputs.os }} \
15+
TEST_PHP_DEBUG=${{ inputs.debug }} \
16+
sapi/cli/php run-extra-tests.php

.github/actions/freebsd/action.yml

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ runs:
1717
prepare: |
1818
cd $GITHUB_WORKSPACE
1919
20+
OPCACHE_TLS_TESTS_DEPS="gcc"
21+
2022
kldload accf_http
2123
pkg install -y \
2224
autoconf \
@@ -42,7 +44,7 @@ runs:
4244
libavif \
4345
`#sqlite3` \
4446
curl \
45-
gcc
47+
$OPCACHE_TLS_TESTS_DEPS
4648
4749
./buildconf -f
4850
CC=clang CXX=clang++ \
@@ -110,8 +112,3 @@ runs:
110112
--show-slow 1000 \
111113
--set-timeout 120 \
112114
-d zend_extension=opcache.so
113-
114-
export TLSC=$(pwd)/ext/opcache/jit/tls/zend_jit_tls_x86_64.c
115-
if sapi/cli/php -r 'exit(!PHP_ZTS);'; then
116-
./ext/opcache/jit/tls/testing/test.sh
117-
fi

.github/actions/test-tls-alpine/action.yml

Lines changed: 0 additions & 10 deletions
This file was deleted.

.github/actions/test-tls-macos/action.yml

Lines changed: 0 additions & 9 deletions
This file was deleted.

.github/actions/test-tls-x32/action.yml

Lines changed: 0 additions & 12 deletions
This file was deleted.

.github/actions/test-tls-x64/action.yml

Lines changed: 0 additions & 11 deletions
This file was deleted.

.github/workflows/nightly.yml

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,10 @@ jobs:
134134
--asan -x
135135
-d zend_extension=opcache.so
136136
-d opcache.enable_cli=1
137-
- name: Test TLS resolution
138-
uses: ./.github/actions/test-tls-alpine
137+
- name: Extra tests
138+
uses: ./.github/actions/extra-tests
139+
with:
140+
debug: true
139141
- name: Notify Slack
140142
if: failure()
141143
uses: ./.github/actions/notify-slack
@@ -268,9 +270,10 @@ jobs:
268270
${{ matrix.run_tests_parameters }}
269271
-d zend_extension=opcache.so
270272
-d opcache.enable_cli=1
271-
- name: Test TLS resolution
272-
if: matrix.debug && matrix.zts && !matrix.asan
273-
uses: ./.github/actions/test-tls-x64
273+
- name: Extra tests
274+
uses: ./.github/actions/extra-tests
275+
with:
276+
debug: ${{ matrix.debug }}
274277
- name: Verify generated files are up to date
275278
uses: ./.github/actions/verify-generated-files
276279
- name: Notify Slack
@@ -360,9 +363,10 @@ jobs:
360363
${{ matrix.run_tests_parameters }}
361364
-d zend_extension=opcache.so
362365
-d opcache.enable_cli=1
363-
- name: Test TLS resolution
364-
if: matrix.debug && matrix.zts
365-
uses: ./.github/actions/test-tls-x32
366+
- name: Extra tests
367+
uses: ./.github/actions/extra-tests
368+
with:
369+
debug: ${{ matrix.debug }}
366370
- name: Notify Slack
367371
if: failure()
368372
uses: ./.github/actions/notify-slack
@@ -422,10 +426,10 @@ jobs:
422426
runTestsParameters: >-
423427
-d zend_extension=opcache.so
424428
-d opcache.enable_cli=1
425-
- name: Test TLS resolution
426-
# JIT+ZTS not supported yet on ARM64
427-
if: matrix.os != '14' && matrix.debug && matrix.zts
428-
uses: ./.github/actions/test-tls-macos
429+
- name: Extra tests
430+
uses: ./.github/actions/extra-tests
431+
with:
432+
debug: ${{ matrix.debug }}
429433
- name: Verify generated files are up to date
430434
uses: ./.github/actions/verify-generated-files
431435
- name: Notify Slack

0 commit comments

Comments
 (0)