Add MDM Transaction Context for further downstream processing giving …

[jkiddo]

…interceptors a better chance of figuring out what happened.

[tadgh]

Approved, recommend adding a changelog in the 7_4_0 folder (or if you want this to merge into the May release, repoint your PR to rel_7_2 and put this change in the 7_2_0 changelog folder.

[dotasek]

The pipelines for this PR failed on MdmOperationPointcutsIT. mergeGoldenResources_withInterceptor_firesHook

I think this may be something intermittent, as it runs correctly locally.

BUT, it does point to a spot we could add test coverage:

Could we add MdmTransactionContext theContext to the params in that test and put in some asserts?

@Hook(Pointcut.MDM_POST_MERGE_GOLDEN_RESOURCES)
				void onUpdate(RequestDetails theDetails, MdmMergeEvent theEvent, MdmTransactionContext theContext) {
					called.getAndSet(true);
					assertEquals("Patient/" + gp1.getIdPart(), theEvent.getFromResource().getId());
					assertEquals("Patient/" + gp2.getIdPart(), theEvent.getToResource().getId());
					assertTrue(theEvent.getFromResource().isGoldenResource() && theEvent.getToResource().isGoldenResource());
				        //FIXME Put some real asserts here
					assertEquals(SomeCondition, theContext.someMethod());
				}