[REF] Add in Unit test to demonstrate error when trying to import a c… #24814

seamuslee001 · 2022-10-25T21:40:50Z

…ampaign id int rather than name or label of the campaign

Overview

@eileenmcnaughton as discussed this shows the problem when importing a contribution with campaign id rather than the campaign label

No test

Test

civibot · 2022-10-25T21:40:52Z

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

…ampaign id int rather than name or label of the campaign

seamuslee001 · 2022-10-25T23:04:09Z

Test fails as expected without #24815

andrew-cormick-dockery · 2022-10-25T23:26:23Z

I'd add tests for:

seamuslee001 · 2022-10-25T23:38:14Z

Jenkins re test this please

seamuslee001 · 2022-10-26T00:25:24Z

Jenkins re test this please

civibot bot added the 5.55 label Oct 25, 2022

[REF] Add in Unit test to demonstrate error when trying to import a c…

d3f5fbe

…ampaign id int rather than name or label of the campaign

seamuslee001 force-pushed the failing_test_campaign_id branch from 7ee3819 to d3f5fbe Compare October 25, 2022 21:41

eileenmcnaughton added the merge on pass label Oct 25, 2022

eileenmcnaughton merged commit f864ecd into civicrm:5.55 Oct 26, 2022

eileenmcnaughton deleted the failing_test_campaign_id branch October 26, 2022 02:01