[phpunit test only] - Add one additional data integrity check to ensure that after Edit Fin…

[KarinG]

…ancial Items add up to total_amount at Contribution level.

Overview

This is a small follow up - add/on to: #12629

Before

Test is checking line_items, tax_amount and total_amount all add up [after editing a Membership that has Line Items and Sales Tax associated with it].

After

In addition test is now also checking that the Financial Items add up to total_amount at the contribution level [after editing a Membership that has Line Items and Sales Tax associated with it].

Comments

Bigger picture: one of our clients has been hit with intermittent regressions on the Edit Contribution screen (if the Contribution involves Line Items and Sales Tax) - this will help ensure that what's working currently is locked in.

All tests are passing on my local - over to Jenkins.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

Adding Merge on pass as only affects unit tests and the assumption here is correct IMO

[KarinG]

Hi and Merci - @seamuslee001

[seamuslee001]

Jenkins re test this please