In D551#11256, @erik wrote:

2. \NParserCompiler::FindTemplateFiles - recompile all templates from the System Tools section - first convertPathToRelative call is with $path = /core/admin_templates parameter, so no comversion happens - incoming path already is relative. However, result is relative path, so test is passed.

Reverted change from array_merge into += during template parser mass-compilation error displaying.

Added color names in comments.

Improve handling of log levels, that doesn't require coloring.

Cosmetic change.

In D546#11169, @erik wrote:

failed parts 1.10, 1.11. (in both clean install and upgrade cases)

after "use the Clear Templates Cache functionality"
system/cache/core/admin_templates folder is present and has drwxr-xr-x permissions, but system/cache/core/admin_templates/config folder is not present, and system/cache/core/admin_templates/config/config_universal.php file is not present.

under system/cache/core/admin_templates folder are present only incs folder and and tools folder

after "Compiled Template Permission Mode" is changed, system/cache/core/admin_templates/tools folder disappears and system/cache/core/admin_templates/config folder is created, so all other test plan checks are passed.

Use "la_fld_LogServerHostname" phrases (existing) instead of "la_fld_ServerHostname" (non-existing) in the Session Log grid.

In D542#11101, @erik wrote:

All by test plan is ok, but missing phrase for "Server Hostname" field (Session log grid column title).

Extracted Arcanist integration into a separate Differential Revision.

Rename test file/class to make Phabricator understand PHPUnit test results/coverage.

Also try to modify tested code to confirm, that related tests would fail.

Let's implement this the other way around (sender does type casting, but recipient throws an exception on error on any PHP version):