dev/core#2568 Enotice fix + test

[eileenmcnaughton]

Same as #20178 but with a test
(and a couple of changes to support that)

Overview

dev/core#2568 Enotice fix + test

Before

This is mostly about adding the tests for dev/core#2568 - it might be nicer to merge that one to actually fix the bug

After

Test added

Technical Details

@demeritcowboy so this adjusts another call to use getIDs() - once again I don't feel like I can replicate the issue we were hitting before

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).

[eileenmcnaughton]

@demeritcowboy are you able to review this? I have a follow up - but it's just a test that builds off this test - which I'll put up when this is merged

[demeritcowboy]

jenkins retest this please (since some relates to #20187)

[demeritcowboy]

This seems ok. Tried a couple variations.
The setContributionIds() is only used for tests. The other replacements seem ok.

[eileenmcnaughton]

thanks @demeritcowboy