Skip to content

Commit

Permalink
Merge pull request #60 from clue-labs/phar
Browse files Browse the repository at this point in the history
Update test suite & clue/phar-composer to avoid skipped tests on Windows
  • Loading branch information
SimonFrings authored Feb 14, 2022
2 parents a949565 + 0617bcb commit cfd0cd4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
11 changes: 4 additions & 7 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ on:

jobs:
PHPUnit:
name: PHPUnit (PHP ${{ matrix.php }} on ${{ matrix.os }})
runs-on: ${{ matrix.os }}
strategy:
matrix:
Expand All @@ -25,19 +26,15 @@ jobs:
- 5.4
steps:
- uses: actions/checkout@v2
- name: Setup PHP
uses: shivammathur/setup-php@v2
- uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php }}
extensions: sqlite3
coverage: xdebug
- run: composer install
- run: vendor/bin/phpunit --coverage-text
if: ${{ matrix.php >= 7.3 }}
- run: vendor/bin/phpunit --coverage-text -c phpunit.xml.legacy
if: ${{ matrix.php < 7.3 }}
- run: cd tests/install-as-dep && composer install && php query.php
- run: cd tests/install-as-dep && php -d phar.readonly=0 vendor/bin/phar-composer build . query.phar
continue-on-error: ${{ matrix.os == 'windows-2019' }}
id: phar
- run: php tests/install-as-dep/query.phar
if: ${{ steps.phar.outcome == 'success' }}
- run: cd tests/install-as-dep && php -d phar.readonly=0 vendor/bin/phar-composer build . query.phar && php query.phar
2 changes: 1 addition & 1 deletion tests/install-as-dep/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"clue/reactphp-sqlite": "*@dev"
},
"require-dev": {
"clue/phar-composer": "^1.0"
"clue/phar-composer": "^1.4"
},
"bin": [
"query.php"
Expand Down

0 comments on commit cfd0cd4

Please sign in to comment.