Skip to content

Commit

Permalink
QA-536: move san file handling in its own class; fix scope
Browse files Browse the repository at this point in the history
  • Loading branch information
dothebart committed May 13, 2024
1 parent 3685a7a commit c71d4e4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions js/client/modules/@arangodb/testutils/process-utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -845,7 +845,7 @@ function executeAndWait (cmd, args, options, valgrindTest, rootDir, coreCheck =
getStructure: function() { return {}; }
};
}

let sh;
let res = {};
if (platform.substr(0, 3) === 'win' && !options.disableMonitor) {
res = executeExternal(cmd, args, false, coverageEnvironment());
Expand Down Expand Up @@ -877,7 +877,7 @@ function executeAndWait (cmd, args, options, valgrindTest, rootDir, coreCheck =
}
} else {
// V8 executeExternalAndWait thinks that timeout is in ms, so *1000
let sh = new sanHandler(cmd.replace(/.*\//, ''), options.sanOptions, options.isSan, options.extremeVerbosity);
sh = new sanHandler(cmd.replace(/.*\//, ''), options.sanOptions, options.isSan, options.extremeVerbosity);
sh.detectLogfiles(instanceInfo.rootDir, instanceInfo.rootDir);
sh.setSanOptions();
res = executeExternalAndWait(cmd, args, false, timeout * 1000, coverageEnvironment());
Expand Down

0 comments on commit c71d4e4

Please sign in to comment.