Add test for participant receipts, super minor consistency fix.

[eileenmcnaughton]

Overview

This is mostly adding testing for participant receipts. The one change to core code is to move the assignment
of contactID to the template from a specific path into the shared code. This parameter is not required by
the templates but it is somewhat common for sites to want to have it assigned to templates for
customisation which is presumably why it's there. Since it is there in one path we should either remove or
have it in all & for a field like contactID all makes sense. Tests lock in that var

Before

Less tests

After

More tests

Technical Details

Comments

I'm mostly trying to build up the test suite here so we can have another go at moving events to create pending + create payment.

I think we should remove $contributeMode from the tpl next

@mattwire @kcristiano

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