Resolved a regression of `template_methods` working when overloaded by library steps

[steven-terrana]

PR Details

fixes #291

a bug was introduced by #283 that caused a regression in template methods working as expected.

Description

PR #283 introduced a bug where the TemplatePrimitiveCollector was not being persisted after each TemplatePrimitiveInjector.

This resulted in TemplateMethodInjector not being able to access the TemplatePrimitives that had been created by LibraryStepInjector.

The fix was to update the TemplatePrimitiveInjector.invoke$ method so that the updated TemplatePrimitiveCollector is persisted on the WorkflowRun after each injector is invoked.

How Has This Been Tested

reproduced the bug both manually and in unit tests. confirmed that the change fixes both.

Types of Changes

• [x] Added Unit Testing
• [ ] Docs change / refactoring / dependency upgrade
• [x] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• [x] I have added the appropriate label for this PR
• [x] If necessary, I have updated the documentation accordingly.
• [x] All new and existing tests passed.