Please also auto-reformat method DocBlock and add missing/remove unnecessary parts to solve any CS-related issues.

Please replace the exception message to:

The $kg argument isn't a number.

Please also auto-reformat method body (few changes really) to solve any CS-related issues.

Please replace the exception message to:

Either the $pounds or $ounces argument isn't a number.

add test (the kUnitFormatterTest test class) to cover at least this change (non-numeric values in $major/$minor are treated as 0 in pound2kg conversion code when provided via kUnitFormatter)

add test (the kUnitFormatterTest test class) to cover at least this change (non-numeric $value is treated as 0 in kg2pounds conversion code when provided via kUnitFormatter)

Please split this into different tests (use data provider as necessary):

• checking valid conversion logic
• checking, that exception is thrown on incorrect parameter value (for each supported parameter, that is protected by exception)

Please:

1. replace the 'InvalidArgumentException' with the InvalidArgumentException::class
2. also check the exception message

• set key to the 'pounds_only=false'
• replace the kUtil::POUND_TO_KG * 1.5 with the final number

• dataset contents isn't displayed for a failed test and that's why we'd better use dataset array keys to show user friendly value
• no need to replicate method internals in test

• set key to the 'pounds_only=true'
• replace the kUtil::POUND_TO_KG * 1.5 with the final number

• dataset contents isn't displayed for a failed test and that's why we'd better use dataset array keys to show user friendly value
• no need to replicate method internals in test

Please split this into different tests (use data provider as necessary):

• checking valid conversion logic
• checking, that exception is thrown on incorrect parameter value (for each supported parameter, that is protected by exception)

Please:

1. replace the 'InvalidArgumentException' with the InvalidArgumentException::class
2. also check the exception message

Please add missing data provider entry, that would cover case, when pounds=0, but ounces<>0, but is a valid number.

In all added tests use assertEquals when comparing numbers instead of assertSame.

Comparing 0 to null would pass with assertEquals.

Please also add the 4th case, when both pounds and ounces are zero.

Ah, I've misplaced the method names.

I've meant use assertSame (works as a === b) instead of assertEquals (works as a == b) to prevent 0 === null comparison from passing the test.