Skip to content

Commit ef6da81

Browse files
committed
Run headers_without_mbstring.phpt instead of always skipping it
The test skips whenever ext/mbstring is present, which is every job in the CI matrix, so the non-mbstring header encoding path had no coverage at all. That is how its expectations came to encode a chunking bug unnoticed. Add an --INI-- section disabling mb_substr()/mb_strlen(). Unlike the -i command line flag, --INI-- also applies to --SKIPIF--, so the existing skip condition now acts as a fallback: the test runs when the functions can be disabled, and still skips if they cannot.
1 parent 1eeb05f commit ef6da81

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

tests/headers_without_mbstring.phpt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
--TEST--
22
Multi-test for headers encoding using base64 and quoted-printable
3+
--INI--
4+
disable_functions=mb_substr,mb_strlen
35
--SKIPIF--
46
<?php
57
if (function_exists('mb_substr') && function_exists('mb_strlen')) {

0 commit comments

Comments
 (0)