Re: Small test case fixes / observations (php-5.2.6_rc4)
| From: | Josie Messa | Date: | Mon, 07 Apr 2008 09:22:16 +0000 |
| Subject: | Re: Small test case fixes / observations (php-5.2.6_rc4) | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-36833@lists.php.net to get a copy of this message | ||
On Sun, Apr 6, 2008 at 10:44 PM, Christian Hoffmann <hoffie@gentoo.org>
wrote:
> Heya,
>
> while building and testing a minimal version of PHP and as such disabling
> the session extension, I discovered that
> ext/standard/tests/general_functions/bug44394_2.phpt fails without the
> session extension being enabled. I created a patch to fix that [1].
>
> On a sidenote, some tests seem to break on amd64 systems:
> ext/standard/tests/array/array_slice_variation6.phpt:
> Apparently there is a real bug (or difference in behavior between 32bit
> and 64bit) in the code here, and the test revealed it. See [2]
>
> ext/standard/tests/array/array_slice_variation2.phpt
> ext/standard/tests/array/array_slice_variation3.phpt:
> Iteration 7 is different on 32bit and 64bit [3][4]. I don't know if this
> behavior is intended, for now I simply added a skipif for non-32bit [5][6].
Ah, I wrote these tests on a 32 bit machine so I didn't pick up any changes
in behaviour. I don't think a SKIPIF section should be added as I wouldn't
expect there to be a difference here at all, and adding a SKIPIF section
means that the problem will get ignored, I'd raise a bug instead.
>
>
> It would be nice to get the first patch (session-related), and maybe the
> 64bit-related skipifs as well, committed.
>
> All testing / patching has been done on/against php-5.2.6RC4.
>
> [1]
>
> http://home.hoffie.info/static/003_test-standard-bug44394_2-skipif.patch
> [2] http://home.hoffie.info/static/array_slice_variation6.diff
> [3] http://home.hoffie.info/static/array_slice_variation2.diff
> [4] http://home.hoffie.info/static/array_slice_variation3.diff
> [5]
>
> http://home.hoffie.info/static/005_test-standard-array_slice_variation2-32bitonly.patch
> [6]
>
> http://home.hoffie.info/static/004_test-standard-array_slice_variation3-32bitonly.patch
>
> Thanks,
> --
> Christian Hoffmann
>
Thanks,
Josie