Page MenuHomePhabricator

Provide additional Arcanist PHP 8.1 fixes
ClosedPublic

Authored by epriestley on Dec 10 2021, 12:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 5, 7:35 AM
Unknown Object (File)
Fri, May 3, 12:58 AM
Unknown Object (File)
Thu, May 2, 1:43 PM
Unknown Object (File)
Thu, May 2, 3:05 AM
Unknown Object (File)
Tue, Apr 30, 8:18 AM
Unknown Object (File)
Sun, Apr 28, 5:58 AM
Unknown Object (File)
Sun, Apr 28, 4:12 AM
Unknown Object (File)
Sun, Apr 28, 12:37 AM
Subscribers
None

Details

Summary

Ref T13588. I pointed my local php at PHP 8.1 and this is what I've hit so far; all these cases seem very unlikely to have any subtle behavior.

Test Plan

Ran various arc workflows under PHP 8.1.

Diff Detail

Repository
rARC Arcanist
Branch
len2
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 25606
Build 35418: Run Core Tests
Build 35417: arc lint + arc unit

Unit TestsFailed

TimeTest
0 msArcanistBundleTestCase::testEncodeBase85
EXCEPTION (RuntimeException): Implicit conversion from float 777.0705882352942 to int loses precision #0 /Users/epriestley/dev/core/lib/arcanist/src/parser/ArcanistBundle.php(1025): PhutilErrorHandler::handleError(8192, 'Implicit conver...', '/Users/epriestl...', 1025) #1 /Users/epriestley/dev/core/lib/arcanist/src/parser/__tests__/ArcanistBundleTestCase.php(983): ArcanistBundle::newBase85Data('\x00\x01\x02\x03\x04\x05\x06\x07\x08\t\n\v\f\r\x0E...', '\n', '64bit')
0 msArcanistBundleTestCase::testGitRepository
EXCEPTION (RuntimeException): ltrim(): Passing null to parameter #1 ($string) of type string is deprecated #0 [internal function]: PhutilErrorHandler::handleError(8192, 'ltrim(): Passin...', '/Users/epriestl...', 31) #1 /Users/epriestley/dev/core/lib/arcanist/src/filesystem/PhutilDirectoryFixture.php(31): ltrim(NULL, '/')
0 msArcanistBundleTestCase::testTabEncoding
EXCEPTION (RuntimeException): strlen(): Passing null to parameter #1 ($string) of type string is deprecated #0 /Users/epriestley/dev/core/lib/arcanist/src/parser/ArcanistDiffParser.php(928): PhutilErrorHandler::handleError(8192, 'strlen(): Passi...', '/Users/epriestl...', 928) #1 /Users/epriestley/dev/core/lib/arcanist/src/parser/ArcanistDiffParser.php(810): ArcanistDiffParser->parseChangeset(Object(ArcanistDiffChange))
0 msArcanistDiffParserTestCase::testGitRenames
EXCEPTION (RuntimeException): preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated #0 [internal function]: PhutilErrorHandler::handleError(8192, 'preg_match(): P...', '/Users/epriestl...', 586) #1 /Users/epriestley/dev/core/lib/arcanist/src/parser/ArcanistDiffParser.php(586): preg_match('@^(?P<new>new) ...', NULL, NULL)
0 msArcanistDiffParserTestCase::testParser
EXCEPTION (RuntimeException): strlen(): Passing null to parameter #1 ($string) of type string is deprecated #0 /Users/epriestley/dev/core/lib/arcanist/src/parser/ArcanistDiffParser.php(928): PhutilErrorHandler::handleError(8192, 'strlen(): Passi...', '/Users/epriestl...', 928) #1 /Users/epriestley/dev/core/lib/arcanist/src/parser/ArcanistDiffParser.php(348): ArcanistDiffParser->parseChangeset(Object(ArcanistDiffChange))
View Full Test Results (12 Failed · 772 Passed · 64 Skipped)

Event Timeline

  • Fix unit tests and a handful of other errors.
  • Also fix service profiler test case.
This revision was not accepted when it landed; it landed in state Needs Review.Dec 10 2021, 12:42 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.