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>
Wed, 18 Apr 2018 23:12:06 +0000 (23:12 +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

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 ],