Skip to content

Improve type-security with phpstan and set up phpunit #1004

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
alsciende wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Improve type-security with phpstan and set up phpunit #1004

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
ea4ba5a
phpstan level 1
alsciende Mar 18, 2026
24f1ce4
phpstan level 2
alsciende Mar 18, 2026
9d6fc9e
phpstan level 3
alsciende Mar 18, 2026
ed41dc8
test containers and test suite
alsciende Mar 18, 2026
53bca0f
removed doctrine-fixtures-bundle ; phpunit is working
alsciende Mar 18, 2026
6e15193
convert deprecated YAML mappings to XML mappings for Doctrine 3.x
alsciende Mar 18, 2026
0a236b4
fix deprecations in sensio_framework_extra and fos_rest
alsciende Mar 19, 2026
333f3c3
remove .phpunit.result.cache
alsciende Apr 12, 2026
4336a2c
use method_exists until php 8 and Stringable
alsciende Apr 12, 2026
38e1cca
add eol at eof
alsciende Apr 12, 2026
32f6274
update 'make test'
alsciende Apr 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
/app/config/parameters.yml
/build/
/phpunit.xml
.phpunit.cache
/var/*
!/var/cache
/var/cache/*
Expand Down
8 changes: 5 additions & 3 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@ install:
php bin/console assets:install --symlink web

phpstan:
php vendor/bin/phpstan --memory-limit=-1 --verbose analyze src --level 0
php vendor/bin/phpstan --memory-limit=-1 --verbose analyze src --level 3

test:
vendor/bin/phpstan analyze src --level 7
phpunit:
php vendor/bin/phpunit

test: phpstan phpunit
5 changes: 5 additions & 0 deletions app/config/config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,8 +126,13 @@ stof_doctrine_extensions:

fos_rest:
routing_loader: false
body_listener: false

nelmio_api_doc:
sandbox:
request_format:
method: accept_header

sensio_framework_extra:
router:
annotations: false
2 changes: 1 addition & 1 deletion app/config/services.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ services:

AppBundle\:
resource: '../../src/AppBundle/*'
exclude: '../../src/AppBundle/{Behavior,DataFixtures,DQL,Entity,Repository,Resources}'
exclude: '../../src/AppBundle/{Behavior,DQL,Entity,Repository,Resources}'

AppBundle\EventListener\DecklistListener:
tags:
Expand Down
2 changes: 1 addition & 1 deletion composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
"twig/twig": "^1.0||^2.0"
},
"require-dev": {
"doctrine/doctrine-fixtures-bundle": "^3.0",
"phpunit/phpunit": "^9.6",
"sensio/generator-bundle": "^3.0",
"symfony/phpunit-bridge": "^3.0"
},
Expand Down
Loading