Made all requested fixes. Created tests for kUbitFormatter class.

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

Changed ligic, added unit tests.

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

Fixed the filesize(): stat failed for error when uploading file larger, than 1MB.

In D529#10934, @erik wrote:

Added tests for Core and Custom modules -

unknown_option_tests_v2.patch72 KBDownload

Added tests for "core" and "custom" modules.

Added tests for Core and Custom modules -

Reclassified some tests from Unit to Functional.

Renamed the "kOptionsFormatter" class test for Phabricator to be able to generate coverage report from it.

Found these problems and fixed them (related to tests in the patch):

Added more unit tests (original version created by Erik).

Tested all units, additional bugs found in the In-Commerce module.

1. I've noticed, that the same issue happens also with coupi and api units of the In-Commerce. Please expand the Differential Revision code/test plan to cover them as well.
2. I've noticed, that you're testing, that notice is gone, but not testing that item adding code still works. Please test that as well.
3. I've also renamed the task. Please rename your changelist in the PhpStorm to avoid accidental Differential Revision rename on patch update.

Partially tested (done testing for Core module part).

1. Languages.

1.1. Broken In-Portal clean install due error in lang prefix, preventing language record creation, required in normal installation process.
1.2. Due specific option_sql configuration for PackName and LocalName fields, possible add only record with values, that already exists. So, when no records in Languages table - can't add first language record.

2. Categories - works well in admin interface, but problematic to add through object, because of Priority field options, that are populated only in OnPreSaveCreated event hook.
3. Change logs - sometimes can't create through object same as languages, due specific Prefix and MasterPrefix field options_sql configuration. But in system records are created by direct SQL, so this not a problem. maybe.
4. Users - can't create and update records with US State, because state options are populated too late - after SetFieldsFromHash already set error to the State field.

In D529#10861, @erik wrote:

I do not understand relation between test plan and proposed automation logic.

1. As I understand from test plan, not only required fields should be tested, but fields with specific formatter.
2. As I understand from test plan, should test not only create, but also update.
3. Each unit may be specific, may have dependencies to another units, so proposed automation logic seems is not helpful.

I do not understand relation between test plan and proposed automation logic.