-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ogma-core
: Spec2Copilot backend does not perform variable substitution in top-level property names
#120
Comments
Change Manager: Confirmed that the issue exists. |
Technical Lead: Confirmed that the issue should be addressed. |
Technical Lead: Issue scheduled for fixing in Ogma 1.3. Fix assigned to: @ivanperez-keera. |
When using property names as arguments to trigger, the Spec2Copilot backend does not apply the same variable/name substitution that was applied in their definition, thus referring in the call to trigger to an undefined argument. This commit modifies the generation of trigger names in Copilot to be able to apply the same name subsitution map, so that the stream name is resolved.
Implementor: Solution implemented, review requested. |
Change Manager: Verified that:
|
Change Manager: Implementation ready to be merged. |
Description
When using property names as arguments to
trigger
, the Spec2Copilot backend does not apply the same variable/name substitution that was applied in their definition, thus referring in the call totrigger
to an undefined argument.Type
Additional context
None.
Requester
Method to check presence of bug
Running the backend with one of the examples available in ogma produces code that does not compile.
Expected result
When executed, the code produced by the
Spec2Copilot
backend does not report to an error due to a trigger stream identifier being undefined.Desired result
When executed, the code produced by the
Spec2Copilot
backend does not report to an error due to a trigger stream identifier being undefined.Proposed solution
Modify the definition of
Spec2Copilot
backend to apply the name substitution function to trigger argument names.Further notes
None.
The text was updated successfully, but these errors were encountered: