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

chore: update PHPUnit to latest version #93

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SMillerDev
Copy link

No description provided.

@SMillerDev SMillerDev force-pushed the chore/test/update_phpunit branch 3 times, most recently from dfdf9dd to 6b15abd Compare November 8, 2024 12:01
@SMillerDev SMillerDev force-pushed the chore/test/update_phpunit branch from 6b15abd to 928ceae Compare November 19, 2024 09:31
ApnsPHP/Message/Tests/CustomMessageTestBase.php Outdated Show resolved Hide resolved
Comment on lines -73 to 69
->withConsecutive(
->willReturnMap([
[ 'Trying to initialize HTTP/2 backend...' ],
[ 'Initializing HTTP/2 backend with certificate.' ],
[ 'Retry to connect (1/3)...' ],
Copy link

@brianstoop brianstoop Nov 19, 2024

Choose a reason for hiding this comment

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

I get that this works but what exactly does this test now? IIRC, if the ReturnMap array is empty it would also succeed, so what does this add?

Choose a reason for hiding this comment

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

willReturnCallback maybe?

Choose a reason for hiding this comment

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

How does willReturnCallback work here?

Choose a reason for hiding this comment

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

@SMillerDev SMillerDev force-pushed the chore/test/update_phpunit branch 2 times, most recently from 84f360e to 10664ae Compare November 20, 2024 16:43
Comment on lines -73 to 69
->withConsecutive(
->willReturnMap([
[ 'Trying to initialize HTTP/2 backend...' ],
[ 'Initializing HTTP/2 backend with certificate.' ],
[ 'Retry to connect (1/3)...' ],

Choose a reason for hiding this comment

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

willReturnCallback maybe?

@SMillerDev SMillerDev force-pushed the chore/test/update_phpunit branch 2 times, most recently from a844c11 to 4827ac3 Compare December 3, 2024 16:37
ApnsPHP/Push.php Outdated Show resolved Hide resolved
@SMillerDev SMillerDev force-pushed the chore/test/update_phpunit branch from 4827ac3 to 2c2bd3d Compare December 6, 2024 08:59
@SMillerDev SMillerDev force-pushed the chore/test/update_phpunit branch from 2c2bd3d to 6c023cb Compare December 6, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants