Pre-request variables are not being interpolated within request body when running tests via CLI

[greaterking]

I have checked the following:

• I use the newest version of bruno.
• I've searched existing issues and found nothing related to my issue.

Describe the bug

When running tests via CLI if pre-request variables used within the request body(json) never get interpolated and are submitted as such {{some_variable}} in your request.

This does not happen within the bruno the app this ONLY happens when running test via bruno CLI

Might be slightly related to...

Issues:

• [Bug] Variables in the body with multipat form type. #352
• Basic Auth header not sent #329

PRs:

• fix(#352): Variable interpolation for multipart form body #1217 (PR)

Environment

.bru file to reproduce the bug

No response

Screenshots/Live demo link

Create a test and set some vars

Use them with in the body

Define some assertions using those variables

Every tests using those variables will fail

Instead of the variable values being sent with the request the template literal is what ends up being saved (in my create example anyway)

By the time assertions are run the variable are working properly again but whats returned as the response will be wrong since the string literal was saved and not it's value.

Currently I'm working around this with this script in my collection so its run with every request

// Check if this is running via command line
// (This property only exists when tests are run via bruno app)
// note: wish there was a native way of doing this ... bru.executionMode() ..'CLI' || 'App'
function isCLI() {
  return typeof req.historyLogger === "undefined"; 
}

// When running tests from the command line ensure
// all variables are set before any requests are made
function initializeCliPreRequestVars() {
  
  if (!isCLI()) return;
  
  const {req: {requestVariables}} = req;

  for (const variableKey in requestVariables) {
      bru.setVar(variableKey, requestVariables[variableKey]);
  }
}

initializeCliPreRequestVars();

[Nightbr]

Same here and also Vars from Pre Request are now (it was working before 1.20) not accessible in Pre Request Script.

const myVar = bru.getVar('my_var'); -> always return undefined

Related: #2759

Fixed: replace bru.getVar('my_var') with bru.getRequestVar('my_var').

Just need to update the doc: https://docs.usebruno.com/scripting/javascript-reference#collection-variables and everything will be clear ;)

[HenriqueRech]

I'm having the same issue.
I'm trying to use the collections our team built in the Bruno App to automate some tests using the cli and its reporters, but right now it doesn't seem to be interpolating any of the collection.bru, folder.bru or the Request vars:pre-request.

I noticed that the bruno-cli var interpolation doesn't seem to use any of the request object variables into account, like the bruno-electron interpolation does.

bruno/packages/bruno-cli/src/runner/interpolate-vars.js

Lines 16 to 18 in ac67c4c

	const interpolateVars = (request, envVars = {}, runtimeVariables = {}, processEnvVars = {}) => {
	// we clone envVars because we don't want to modify the original object
	envVars = cloneDeep(envVars);

bruno/packages/bruno-electron/src/ipc/network/interpolate-vars.js

Lines 16 to 22 in ac67c4c

	const interpolateVars = (request, envVariables = {}, runtimeVariables = {}, processEnvVars = {}) => {
	const globalEnvironmentVariables = request?.globalEnvironmentVariables || {};
	const collectionVariables = request?.collectionVariables || {};
	const folderVariables = request?.folderVariables || {};
	const requestVariables = request?.requestVariables || {};
	// we clone envVars because we don't want to modify the original object
	envVariables = cloneDeep(envVariables);

[chriswinter90]

This is definitely still an issue in the bruno-cli runner.

Requests and tests work fine in the electron app.

As @HenriqueRech pointed out. The interpolate-vars.js was modified with @lohxt1 's fix for multipart form data variables but doesn't look like those changes also made their way into the cli.

[thdk]

I'm not using cli but the electron app and notice that variables set in pre request Vars tab also don't override variables which are already set at runtime.