fix(#151, #188): fixed issue where collections vars in tests were not being updated

packages/bruno-electron/src/ipc/network/index.js

@@ -72,7 +72,7 @@ const getSize = (data) => {
   return 0;
 };
 
-const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
+const registerNetworkIpc = (mainWindow) => {
   // handler for sending http request
   ipcMain.handle('send-http-request', async (event, item, collectionUid, collectionPath, environment, collectionVariables) => {
     const cancelTokenUid = uuid();
@@ -216,6 +216,7 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
 
       // run assertions
       const assertions = get(request, 'assertions');
+      if(assertions && assertions.length) {
         const assertRuntime = new AssertRuntime();
         const results = assertRuntime.runAssertions(assertions, request, response, envVars, collectionVariables, collectionPath);
 
@@ -226,9 +227,11 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
           requestUid,
           collectionUid
         });
+      }
 
       // run tests
       const testFile = item.draft ? get(item.draft, 'request.tests') : get(item, 'request.tests');
+      if(testFile && testFile.length) {
         const testRuntime = new TestRuntime();
         const testResults = testRuntime.runTests(testFile, request, response, envVars, collectionVariables, collectionPath);
 
@@ -240,6 +243,14 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
           collectionUid
         });
 
+        mainWindow.webContents.send('main:script-environment-update', {
+          envVariables: testResults.envVariables,
+          collectionVariables: testResults.collectionVariables,
+          requestUid,
+          collectionUid
+        });
+      }
+
       deleteCancelToken(cancelTokenUid);
 
       return {
@@ -263,6 +274,7 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
       if(error && error.response) {
         // run assertions
         const assertions = get(request, 'assertions');
+        if(assertions && assertions.length) {
           const assertRuntime = new AssertRuntime();
           const results = assertRuntime.runAssertions(assertions, request, error.response, envVars, collectionVariables, collectionPath);
     
@@ -273,9 +285,11 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
             requestUid,
             collectionUid
           });
+        }
   
         // run tests
         const testFile = item.draft ? get(item.draft, 'request.tests') : get(item, 'request.tests');
+        if(testFile && testFile.length) {
           const testRuntime = new TestRuntime();
           const testResults = testRuntime.runTests(testFile, request, error.response, envVars, collectionVariables, collectionPath);
     
@@ -287,6 +301,14 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
             collectionUid
           });
 
+          mainWindow.webContents.send('main:script-environment-update', {
+            envVariables: testResults.envVariables,
+            collectionVariables: testResults.collectionVariables,
+            requestUid,
+            collectionUid
+          });
+        }
+  
         return {
           status: error.response.status,
           statusText: error.response.statusText,
@@ -507,6 +529,7 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
 
           // run tests
           const testFile = item.draft ? get(item.draft, 'request.tests') : get(item, 'request.tests');
+          if(testFile && testFile.length) {
             const testRuntime = new TestRuntime();
             const testResults = testRuntime.runTests(testFile, request, response, envVars, collectionVariables, collectionPath);
 
@@ -516,6 +539,13 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
               ...eventData
             });
 
+            mainWindow.webContents.send('main:script-environment-update', {
+              envVariables: testResults.envVariables,
+              collectionVariables: testResults.collectionVariables,
+              collectionUid
+            });
+          }
+
           mainWindow.webContents.send('main:run-folder-event', {
             type: 'response-received',
             ...eventData,
@@ -562,6 +592,7 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
 
             // run tests
             const testFile = item.draft ? get(item.draft, 'request.tests') : get(item, 'request.tests');
+            if(testFile && testFile.length) {
               const testRuntime = new TestRuntime();
               const testResults = testRuntime.runTests(testFile, request, error.response, envVars, collectionVariables, collectionPath);
 
@@ -571,6 +602,13 @@ const registerNetworkIpc = (mainWindow, watcher, lastOpenedCollections) => {
                 ...eventData
               });
 
+              mainWindow.webContents.send('main:script-environment-update', {
+                envVariables: testResults.envVariables,
+                collectionVariables: testResults.collectionVariables,
+                collectionUid
+              });
+            }
+
             // if we get a response from the server, we consider it as a success
             mainWindow.webContents.send('main:run-folder-event', {
               type: 'response-received',