Unit test for FGB error in #14331

[eileenmcnaughton]

Overview

Adds unit test to demonstrate that without #14331 we get an error

Before

No test cover

After

Test cover

Technical Details

@seamuslee001 despite the temptation I couldn't justify fixing #14331 without a test as I wouldn't have accepted a FGB fix from someone else without one & really if it was just a case of fixing without a test I would have done it 2 months ago https://lab.civicrm.org/dev/core/issues/754 since it would have been a 5 minute job

Comments

[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]

Merging as it affects tests and is a test for another PR

[eileenmcnaughton]

@seamuslee001 hmm - that should have failed - it failed locally without the other patch

[seamuslee001]

@eileenmcnaughton it did i have merged and kicked off tests on the one

[eileenmcnaughton]

@seamuslee001 ah right - I thought you'd merge in the reverse order - but no - you are on it!