Skip to content

Commit

Permalink
Refactor client handling (#35)
Browse files Browse the repository at this point in the history
refactor: optimize app structure and increase test coverage
  • Loading branch information
gearsdigital authored Oct 26, 2021
1 parent 9849420 commit 9ce5787
Show file tree
Hide file tree
Showing 52 changed files with 1,675 additions and 2,918 deletions.
35 changes: 27 additions & 8 deletions .github/workflows/php.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,18 +3,37 @@ name: PHP Composer
on: [push]

jobs:
build:
test:

runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v2
with:
path: 'kirby-reporter'

- name: Validate composer.json and composer.lock
run: composer validate
- name: Install Starterkit
run: |
git clone https://github.com/getkirby/starterkit.git
mkdir starterkit/site/plugins
- name: Install dependencies
run: composer install --prefer-dist --no-progress --no-suggest
- name: Update Kirby
run: |
rm -rf starterkit/kirby
composer require getkirby/cms:3.6.0-rc.1@dev
mv kirby starterkit
cd starterkit/kirby
composer install
- name: Run test suite
run: composer run-script test
- name: Install
run: |
mv kirby-reporter starterkit/site/plugins
cd starterkit/site/plugins/kirby-reporter
composer install
- name: Tests
run: |
cd starterkit/site/plugins/kirby-reporter
export KIRBY_REPORTER_TEST=true
composer validate
composer test
7 changes: 3 additions & 4 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,16 +18,15 @@
"ext-json": "*",
"getkirby/composer-installer": "^1.2",
"ql/uri-template": "^1.1",
"guzzlehttp/guzzle": "~7.0"
"guzzlehttp/guzzle": "~7.4.0"
},
"config": {
"optimize-autoloader": true
},
"require-dev": {
"phpunit/phpunit": "^8",
"spatie/phpunit-watcher": "dev-master"
"phpunit/phpunit": "^9.5.10"
},
"scripts": {
"test": "phpunit --bootstrap vendor/autoload.php tests"
"test": "phpunit --bootstrap tests/bootstrap.php tests"
}
}
Loading

0 comments on commit 9ce5787

Please sign in to comment.