From 8745a73c4fa42e6fc304bc5b53075ac071a918ed Mon Sep 17 00:00:00 2001 From: Mikolaj Boc Date: Mon, 9 Jan 2023 16:12:11 +0100 Subject: [PATCH] Remove the instance.ENV assignment in js test runner MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The assignment crashes with 3.1.25 as instance.ENV is now a read-only property. This used to be assigned to an empty object, so it was useless anyway. Change-Id: I47d4cd831052de514c372adb3f2202d8a0a6e06f Reviewed-by: Morten Johan Sørvig --- util/wasm/batchedtestrunner/qwasmjsruntime.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/util/wasm/batchedtestrunner/qwasmjsruntime.js b/util/wasm/batchedtestrunner/qwasmjsruntime.js index c560f38ea48..b8603f26181 100644 --- a/util/wasm/batchedtestrunner/qwasmjsruntime.js +++ b/util/wasm/batchedtestrunner/qwasmjsruntime.js @@ -176,12 +176,6 @@ export class CompiledModule { instanceParams.monitorRunDependencies = (name) => { }; instanceParams.print = (text) => true && console.log(text); instanceParams.printErr = (text) => true && console.warn(text); - instanceParams.preRun = [ - (instance) => { - const env = {}; - instance.ENV = env; - }, - ]; instanceParams.mainScriptUrlOrBlob = new Blob([this.#js], { type: 'text/javascript',