Optimize redundant mock scripts evaluation

[w-ess]

Prerequisites

• I have written a descriptive issue title
• I have searched existing bugs to ensure a similar bug has not already been created

Description

Redundant script evaluation

Steps to reproduce

Given a mock with a single custom script in the body. Calling its respective method, the script will be evaluated twice.

Expected behavior

The script should be evaluated once.

Screenshots

Additional context

No response

[natenho]

Hello @w-ess , could you provide more details? how are you noticing the duplication? I've just sent an e-mail so we can work on your case

[w-ess]

Hi, example sent by e-mail

[natenho]

The behavior described in this issue is actually by design and is caused by multiple template transformations occurring along the mocking pipeline, leading to re-evaluation of script snippets.

For instance, given the mock in the provided screenshot, the script <#= TesteObterLInk.ObterLinl() #> will be executed at 2 distinct moments:

1. Before deciding which mock to return, it needs to transform each template to evaluate the "condition" field of each one:
   

   Mockaco/src/Mockaco.AspNetCore/Templating/Request/RequestConditionMatcher.cs

   Line 47 in c3dcc72

   var template = await _templateTransformer.Transform(mock.RawTemplate, conditionMatcherScriptContext);

Given that the condition is dynamic, the mock must not be returned when it is evaluated to false. This is step could be optimized to partially evaluate the template, looking only for the condition field

2. After deciding which mock to return, we need to effectively produce the output:
   

   Mockaco/src/Mockaco.AspNetCore/Middlewares/RequestMatchingMiddleware.cs

   Line 58 in c3dcc72

   var template = await templateTransformer.TransformAndSetVariables(mock.RawTemplate, scriptContext);

This stage leads to re-evaluation of mock scripts, this time with all context variables available (like route variables).

There is also a third possible moment where the script might be evaluated again, when using the callback template:

Mockaco/src/Mockaco.AspNetCore/Middlewares/CallbackMiddleware.cs

Line 62 in c3dcc72

var template = await templateTransformer.Transform(mockacoContext.Mock.RawTemplate, scriptContext);

@w-ess I'll change the title of the issue and consider this optimization as an improvement instead of an actual bug.