fix: move persisting user to catch statement #3871
main.yml
on: push
Coding Guidelines Check
29s
FE Asset Linting
28s
FE asset formatting check
19s
PHPMD
12s
Static Code Analysis
33s
Run frontend type check
29s
Run frontend unit- and functional-tests
1m 1s
Matrix: Run E2E-tests via Cypress
Annotations
9 errors and 1 warning
|
FE Asset Linting
Process completed with exit code 1.
|
|
FE Asset Linting:
src/Resources/src/stores/participationsStore.ts#L213
'getProfileId' is defined but never used
|
|
Static Code Analysis
Process completed with exit code 2.
|
|
InvalidOperand:
src/Mealz/MealBundle/Tests/Controller/ParticipantControllerTest.php#L95
src/Mealz/MealBundle/Tests/Controller/ParticipantControllerTest.php:95:45: InvalidOperand: Cannot concatenate with a int|null (see https://psalm.dev/058)
|
|
InvalidArgument:
src/Mealz/MealBundle/Tests/Controller/ParticipantControllerTest.php#L91
src/Mealz/MealBundle/Tests/Controller/ParticipantControllerTest.php:91:47: InvalidArgument: Argument 1 of App\Mealz\MealBundle\Tests\Controller\ParticipantControllerTest::getUserProfile expects int|null, but 'alice.meals' provided (see https://psalm.dev/004)
|
|
InvalidArgument:
src/Mealz/MealBundle/Service/GuestParticipationService.php#L96
src/Mealz/MealBundle/Service/GuestParticipationService.php:96:25: InvalidArgument: Argument 1 of App\Mealz\UserBundle\Entity\Profile::setId expects int|null, but non-empty-string provided (see https://psalm.dev/004)
|
|
Coding Guidelines Check
Process completed with exit code 8.
|
|
Run frontend unit- and functional-tests
Process completed with exit code 1.
|
|
tests/unit/stores/participationsStore.spec.ts > Test participationsStore > should return the profile id:
src/Resources/tests/unit/stores/participationsStore.spec.ts#L135
TypeError: getProfileId is not a function
❯ tests/unit/stores/participationsStore.spec.ts:135:20
|
|
FE Asset Linting:
src/Resources/src/components/NavBar.vue#L65
Attribute "v-show" should go before ":key"
|