Skip to content

Commit 66f9d71

Browse files
cebecebe
authored
Merge pull request #146 from simPod/fix-test
Rework CI
2 parents 3a028ef + a008654 commit 66f9d71

File tree

6 files changed

+89
-72
lines changed

6 files changed

+89
-72
lines changed

‎.github/workflows/code-style.yml‎

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
name: Code Style
2+
3+
on:
4+
push:
5+
branches: [ master ]
6+
pull_request:
7+
branches: [ master ]
8+
9+
jobs:
10+
code-style:
11+
runs-on: ubuntu-latest
12+
steps:
13+
- uses: actions/checkout@v2
14+
15+
- name: Install PHP
16+
uses: shivammathur/setup-php@v2
17+
with:
18+
php-version: 7.1
19+
20+
- name: Check code style
21+
run: make check-style

‎.github/workflows/php.yml‎

Lines changed: 33 additions & 66 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: PHP Composer
1+
name: Tests
22

33
on:
44
push:
@@ -7,35 +7,40 @@ on:
77
branches: [ master ]
88

99
jobs:
10-
build:
11-
10+
phpunit:
11+
name: Tests
1212
strategy:
1313
fail-fast: false
1414
matrix:
15-
# os: [ubuntu-latest, macos-latest, windows-latest]
1615
os: [ubuntu-latest]
17-
php: ['7.1', '7.2', '7.3', '7.4', '8.0']
18-
# max 4.4.16, see https://github.com/symfony/symfony/issues/39521
19-
# max 5.1.8, see https://github.com/symfony/symfony/issues/39521
20-
yaml: ['5.2.9', '5.1.11', '4.4.24', '^3.4']
16+
php:
17+
- "7.1"
18+
- "7.2"
19+
- "7.3"
20+
- "7.4"
21+
- "8.0"
22+
dependencies:
23+
- "lowest"
24+
- "highest"
25+
symfony-yaml: ['^3.4', '^4', '^5']
26+
include:
27+
- os: "windows-latest"
28+
php: "8.0"
29+
dependencies: "highest"
30+
symfony-yaml: '5.4.2'
31+
- os: "macos-latest"
32+
php: "8.0"
33+
dependencies: "highest"
34+
symfony-yaml: '^5'
2135
exclude:
22-
# Symfony YAML does not run on PHP 7.1
36+
# symfony/yaml v5 does not run on PHP 7.1
2337
- php: '7.1'
24-
yaml: '5.1.11'
25-
- php: '7.1'
26-
yaml: '5.2.9'
27-
include:
28-
- php: '7.4'
29-
os: windows-latest
30-
yaml: '5.2.9'
31-
- php: '7.4'
32-
os: macos-latest
33-
yaml: '5.2.9'
34-
38+
symfony-yaml: '^5'
39+
# symfony/yaml v3.4 is not compatible with PHP 8.0 but has no upper-bound, so it installs on it
40+
- php: '8.0'
41+
symfony-yaml: '^3.4'
3542

3643
runs-on: ${{ matrix.os }}
37-
env:
38-
YAML: ${{ matrix.yaml }}
3944

4045
steps:
4146
- uses: actions/checkout@v2
@@ -44,60 +49,22 @@ jobs:
4449
uses: shivammathur/setup-php@v2
4550
with:
4651
php-version: ${{ matrix.php }}
47-
ini-values: date.timezone='UTC'
4852
coverage: pcov
4953
tools: composer:v2
5054

51-
- name: Determine composer cache directory (Linux/MacOS)
52-
if: matrix.os != 'windows-latest'
53-
run: echo "COMPOSER_CACHE_DIR=$(composer config cache-dir)" >> $GITHUB_ENV
54-
55-
- name: Determine composer cache directory (Windows)
56-
if: matrix.os == 'windows-latest'
57-
run: echo "COMPOSER_CACHE_DIR=~\AppData\Local\Composer" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
55+
- name: Require specific symfony/yaml version
56+
run: "composer require symfony/yaml:'${{ matrix.symfony-yaml }}' --prefer-dist --no-interaction --ansi --no-install"
5857

59-
- name: Cache dependencies installed with composer
60-
uses: actions/cache@v2
58+
- name: "Install dependencies with Composer"
59+
uses: "ramsey/composer-install@v2"
6160
with:
62-
path: ${{ env.COMPOSER_CACHE_DIR }}
63-
key: php${{ matrix.php }}-os${{ matrix.os }}-yaml${{ matrix.yaml }}-composer-${{ hashFiles('**/composer.json') }}
64-
65-
66-
- name: Validate composer.json and composer.lock
67-
run: composer validate --ansi
68-
69-
- name: Install dependencies (Linux/MacOS)
70-
if: matrix.os != 'windows-latest'
71-
run: |
72-
make install
73-
composer require symfony/yaml:"${YAML}" --prefer-dist --no-interaction --ansi
74-
75-
- name: Install dependencies (Windows)
76-
if: matrix.os == 'windows-latest'
77-
run: |
78-
composer install --prefer-dist --no-interaction --no-progress --ansi
79-
composer require symfony/yaml:5.1.8 --prefer-dist --no-interaction --ansi
61+
dependency-versions: "${{ matrix.dependencies }}"
8062

8163
- name: Validate test data
82-
if: matrix.os == 'ubuntu-latest'
8364
run: make lint
8465

85-
- name: PHP Stan analysis
86-
if: matrix.os == 'ubuntu-latest'
87-
run: make stan
88-
89-
- name: PHPUnit tests (Linux/MacOS)
90-
if: matrix.os != 'windows-latest'
66+
- name: PHPUnit tests
9167
run: make test
9268

93-
- name: PHPUnit tests (Windows)
94-
if: matrix.os == 'windows-latest'
95-
run: vendor/bin/phpunit --colors=always
96-
97-
- name: Check code style
98-
if: matrix.os == 'ubuntu-latest'
99-
run: make check-style
100-
10169
- name: Code coverage
102-
if: matrix.os == 'ubuntu-latest'
10370
run: make coverage

‎.github/workflows/phpstan.yml‎

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
name: PHPStan
2+
3+
on:
4+
push:
5+
branches: [ master ]
6+
pull_request:
7+
branches: [ master ]
8+
9+
jobs:
10+
phpstan:
11+
runs-on: ubuntu-latest
12+
steps:
13+
- uses: actions/checkout@v2
14+
15+
- name: Install PHP
16+
uses: shivammathur/setup-php@v2
17+
with:
18+
php-version: "7.1"
19+
tools: composer:v2
20+
21+
- name: "Install dependencies with Composer"
22+
uses: "ramsey/composer-install@v2"
23+
24+
- name: PHPStan analysis
25+
run: make stan

‎Makefile‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,12 @@ test:
2525
php $(PHPARGS) $(XPHPARGS) bin/php-openapi validate tests/spec/data/recursion.json
2626
php $(PHPARGS) $(XPHPARGS) bin/php-openapi validate tests/spec/data/recursion2.yaml
2727

28-
lint:
28+
lint: install
2929
php $(PHPARGS) $(XPHPARGS) bin/php-openapi validate tests/spec/data/reference/playlist.json
3030
php $(PHPARGS) $(XPHPARGS) bin/php-openapi validate tests/spec/data/recursion.json
3131
php $(PHPARGS) $(XPHPARGS) bin/php-openapi validate tests/spec/data/recursion2.yaml
32-
node_modules/.bin/speccy lint tests/spec/data/reference/playlist.json
33-
node_modules/.bin/speccy lint tests/spec/data/recursion.json
32+
yarn run speccy lint tests/spec/data/reference/playlist.json
33+
yarn run speccy lint tests/spec/data/recursion.json
3434

3535
stan:
3636
php $(PHPARGS) vendor/bin/phpstan analyse -l 5 src

‎composer.json‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,19 +20,21 @@
2020
"require": {
2121
"php": ">=7.1.0",
2222
"ext-json": "*",
23-
"symfony/yaml": "^3.4 | ^4.0 | ^5.0",
24-
"justinrainbow/json-schema": "^5.0"
23+
"symfony/yaml": "^3.4 || ^4 || ^5",
24+
"justinrainbow/json-schema": "^5.2"
2525
},
2626
"require-dev": {
2727
"cebe/indent": "*",
2828
"phpunit/phpunit": "^6.5 || ^7.5 || ^8.5 || ^9.4",
29-
3029
"oai/openapi-specification": "3.0.3",
3130
"mermade/openapi3-examples": "1.0.0",
3231
"apis-guru/openapi-directory": "1.0.0",
3332
"nexmo/api-specification": "1.0.0",
3433
"phpstan/phpstan": "^0.12.0"
3534
},
35+
"conflict": {
36+
"symfony/yaml": "3.4.0 - 3.4.4 || 4.0.0 - 4.4.17 || 5.0.0 - 5.1.9 || 5.2.0"
37+
},
3638
"autoload": {
3739
"psr-4": {
3840
"cebe\\openapi\\": "src/"

‎tests/bootstrap.php‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,5 @@
99
}
1010

1111
require __DIR__ . '/../vendor/autoload.php';
12+
13+
date_default_timezone_set('UTC');

0 commit comments

Comments
 (0)