Skip to content
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.

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

Internal variables for generated assert has prefix underscore #68

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

h4kuna
Copy link
Contributor

@h4kuna h4kuna commented Nov 15, 2023

Bug:

I use interface, where parameter is $message. And internal variable rewrite this parameter.

example

    function captureMessage(string $message, \Sentry\Severity $level = null): ?\Sentry\EventId
    {
        ...
        $message = $this->getDebugMessage(); // <------ Here is rewritten incoming parameter

        Assert::assertEquals($expectation->message, $message, $message);

    }

@pionl pionl merged commit c644785 into wrk-flow:main Nov 16, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants