Skip to content

Added Symfony Logger assertion (dontSeeDeprecations)#206

Merged
TavoNiievez merged 1 commit intoCodeception:mainfrom
TavoNiievez:logger-trait
Feb 20, 2025
Merged

Added Symfony Logger assertion (dontSeeDeprecations)#206
TavoNiievez merged 1 commit intoCodeception:mainfrom
TavoNiievez:logger-trait

Conversation

@TavoNiievez
Copy link
Copy Markdown
Member

@TavoNiievez TavoNiievez commented Feb 17, 2025 •
edited
Loading 

       $I->amOnPage('/home');
       $I->dontSeeDeprecations();
 Step  Don't see deprecations
 Fail  Found 2 deprecation messages in the log:
  - Since doctrine/doctrine-bundle 2.12: The default value of "doctrine.orm.controller_resolver.auto_mapping" will be changed from `true` to `false`...
  - Since doctrine/doctrine-bundle 2.13: Enabling the controller resolver automapping feature has been deprecated....
Failed asserting that an array is empty.

@TavoNiievez
Copy link
Copy Markdown
Member Author

TavoNiievez commented Feb 17, 2025

cc. @xEdelweiss @ThomasLandauer

ThomasLandauer
ThomasLandauer suggested changes Feb 17, 2025
View reviewed changes
src/Codeception/Module/Symfony/LoggerAssertionsTrait.php Outdated
trait LoggerAssertionsTrait
{
/**
* Asserts that there are no deprecation messages in the log.
Copy link
Copy Markdown
Member

@ThomasLandauer ThomasLandauer Feb 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Asserts that there are no deprecation messages in Symfony's log.

Copy link
Copy Markdown
Member Author

@TavoNiievez TavoNiievez Feb 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed, thank you :)

@TavoNiievez TavoNiievez merged commit 12fdf09 into Codeception:main Feb 20, 2025
11 checks passed
@TavoNiievez TavoNiievez deleted the logger-trait branch April 18, 2025 16:37
@TavoNiievez TavoNiievez added this to the 3.5.1 milestone Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ThomasLandauer ThomasLandauer ThomasLandauer requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.5.1

Development

Successfully merging this pull request may close these issues.

2 participants

@TavoNiievez @ThomasLandauer