Skip to content

ext/json: applied fixers to improve test robustness - #23029

Merged
Girgias merged 1 commit into
php:masterfrom
NickSdot:test/hardening/ext-json
Aug 22, 2026
Merged

ext/json: applied fixers to improve test robustness#23029
Girgias merged 1 commit into
php:masterfrom
NickSdot:test/hardening/ext-json

Conversation

@NickSdot

@NickSdot NickSdot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Asserts error class; extracted from: #22799

@NickSdot
NickSdot force-pushed the test/hardening/ext-json branch from 25372da to 5fe4302 Compare August 4, 2026 14:03
@NickSdot
NickSdot marked this pull request as ready for review August 4, 2026 15:12
@NickSdot
NickSdot requested a review from bukka as a code owner August 4, 2026 15:12
Comment thread ext/json/tests/009.phpt
@@ -11,4 +11,3 @@ echo "Done\n";
--EXPECT--
string(7) "[1,2,3]"
Done

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This (and following) removes useless second empty line.

@NickSdot
NickSdot force-pushed the test/hardening/ext-json branch 3 times, most recently from f4635b4 to 684f514 Compare August 19, 2026 14:08
@NickSdot
NickSdot force-pushed the test/hardening/ext-json branch from 684f514 to ebb32fa Compare August 21, 2026 18:25
@Girgias
Girgias merged commit 60adb50 into php:master Aug 22, 2026
17 of 18 checks passed
@NickSdot
NickSdot deleted the test/hardening/ext-json branch August 22, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants