API: Remove 1e3>1 test case from ApiBaseTest
authorTimo Tijhof <krinklemail@gmail.com>
Wed, 18 Apr 2018 23:09:53 +0000 (00:09 +0100)
committerKrinkle <krinklemail@gmail.com>
Thu, 19 Apr 2018 01:22:10 +0000 (01:22 +0000)
Follows-up d3da5e08d35f, which broke PHP 7.1 and PHP 7.2 test jobs.

This isn't about logic in ApiBase, but about PHP's str-to-int
logic which actually varies between PHP5 and PHP7.

One maps it to 1000, the other as strict int 1 (with non-numerical
text chopped off, the same as it would for 1foo).

If we want to support 1e3 as part of the API itself, then we'd
need to parse it ourselves and then add the test back.

Bug: T192425
Change-Id: I6fc43f29425749a1abe785fefa312db6a98ea7cd
(cherry picked from commit 9ded02ec8e255ee6b9ee213b278b1f9a86147051)

tests/phpunit/includes/api/ApiBaseTest.php

index dbedfc5..4bffc74 100644 (file)
@@ -1159,7 +1159,6 @@ class ApiBaseTest extends ApiTestCase {
                $integerTests = [
                        [ '+1', 1 ],
                        [ '-1', -1 ],
-                       [ '1e3', 1 ],
                        [ '1.5', 1 ],
                        [ '-1.5', -1 ],
                        [ '1abc', 1 ],