[wasm] cleanup testing and host support in templatest and tests (dotnet#73785)

* wip

* wip

* wip

* Single instance in create method.

* withMainAssembly

* withAsyncFlushOnExit

* Fix exporting INTERNAL api.

Move all legacy API object creation to dotnet.es6.post.js where module export is created.

* fix release build

* Move virtualWorkingDirectory initialization to the create method, so it is used in both scenarios (run & create).

* Fix applicationArguments in templates.

* Update WBT.

* Fix ENVIRONMENT_IS_*.

* Move pthreadPoolSize to MonoConfig (from internal).

Fix merge error.

* Fix ConsolePublishAndRun.

* Update readme.md in templates.

* Update WasmAppHost to always pass application arguments as query string.

Add API for parsing applications arguments passed by the WasmAppHost.

Co-authored-by: Marek Fišera <mara@neptuo.com>

Author: pavelsavara

src/installer/pkg/sfx/Microsoft.NETCore.App/Directory.Build.props

@@ -240,6 +240,7 @@
     <PlatformManifestFileEntry Include="dotnet.es6.pre.js" IsNative="true" />
     <PlatformManifestFileEntry Include="dotnet.es6.lib.js" IsNative="true" />
     <PlatformManifestFileEntry Include="dotnet.es6.post.js" IsNative="true" />
+    <PlatformManifestFileEntry Include="dotnet.es6.extpost.js" IsNative="true" />
     <PlatformManifestFileEntry Include="corebindings.c" IsNative="true" />
     <PlatformManifestFileEntry Include="driver.c" IsNative="true" />
     <PlatformManifestFileEntry Include="pinvoke.c" IsNative="true" />

src/mono/sample/wasm/browser-bench/Console/Makefile

@@ -9,6 +9,6 @@ endif
 PROJECT_NAME=Wasm.Console.Bench.Sample.csproj
 CONSOLE_DLL=Wasm.Console.Bench.Sample.dll
 MAIN_JS=test-main.js
-ARGS=--run $(CONSOLE_DLL)
+ARGS=
 
 run: run-console

src/mono/sample/wasm/browser/main.js

@@ -1,21 +1,4 @@
-import createDotnetRuntime from './dotnet.js'
-
-function wasm_exit(exit_code, reason) {
-    /* Set result in a tests_done element, to be read by xharness in runonly CI test */
-    const tests_done_elem = document.createElement("label");
-    tests_done_elem.id = "tests_done";
-    tests_done_elem.innerHTML = exit_code.toString();
-    if (exit_code) tests_done_elem.style.background = "red";
-    document.body.appendChild(tests_done_elem);
-
-    if (reason) console.error(reason);
-    console.log(`WASM EXIT ${exit_code}`);
-}
-
-/**
- * @type {import('../../../wasm/runtime/dotnet').CreateDotnetRuntimeType}
- */
-const createDotnetRuntimeTyped = createDotnetRuntime;
+import { dotnet, exit } from './dotnet.js'
 
 function add(a, b) {
     return a + b;
@@ -24,29 +7,35 @@ function add(a, b) {
 function sub(a, b) {
     return a - b;
 }
+
 try {
-    const { runtimeBuildInfo, setModuleImports, getAssemblyExports, runMain } = await createDotnetRuntimeTyped({
-        configSrc: "./mono-config.json",
-        onConfigLoaded: (config) => {
-            // This is called during emscripten `dotnet.wasm` instantiation, after we fetched config.
-            console.log('user code Module.onConfigLoaded');
-            // config is loaded and could be tweaked before the rest of the runtime startup sequence
-            config.environmentVariables["MONO_LOG_LEVEL"] = "debug"
-        },
-        preInit: () => { console.log('user code Module.preInit'); },
-        preRun: () => { console.log('user code Module.preRun'); },
-        onRuntimeInitialized: () => {
-            console.log('user code Module.onRuntimeInitialized');
-            // here we could use API passed into this callback
-            // Module.FS.chdir("/");
-        },
-        onDotnetReady: () => {
-            // This is called after all assets are loaded.
-            console.log('user code Module.onDotnetReady');
-        },
-        postRun: () => { console.log('user code Module.postRun'); },
-    });
-    
+    const { runtimeBuildInfo, setModuleImports, getAssemblyExports, runMain, getConfig } = await dotnet
+        .withConsoleForwarding()
+        .withElementOnExit()
+        .withModuleConfig({
+            configSrc: "./mono-config.json",
+            onConfigLoaded: (config) => {
+                // This is called during emscripten `dotnet.wasm` instantiation, after we fetched config.
+                console.log('user code Module.onConfigLoaded');
+                // config is loaded and could be tweaked before the rest of the runtime startup sequence
+                config.environmentVariables["MONO_LOG_LEVEL"] = "debug"
+            },
+            preInit: () => { console.log('user code Module.preInit'); },
+            preRun: () => { console.log('user code Module.preRun'); },
+            onRuntimeInitialized: () => {
+                console.log('user code Module.onRuntimeInitialized');
+                // here we could use API passed into this callback
+                // Module.FS.chdir("/");
+            },
+            onDotnetReady: () => {
+                // This is called after all assets are loaded.
+                console.log('user code Module.onDotnetReady');
+            },
+            postRun: () => { console.log('user code Module.postRun'); },
+        })
+        .create();
+
+
     // at this point both emscripten and monoVM are fully initialized.
     // we could use the APIs returned and resolved from createDotnetRuntime promise
     // both exports are receiving the same object instances
@@ -60,17 +49,18 @@ try {
         }
     });
 
-    const exports = await getAssemblyExports("Wasm.Browser.Sample.dll");
+    const config = getConfig();
+    const exports = await getAssemblyExports(config.mainAssemblyName);
     const meaning = exports.Sample.Test.TestMeaning();
     console.debug(`meaning: ${meaning}`);
     if (!exports.Sample.Test.IsPrime(meaning)) {
         document.getElementById("out").innerHTML = `${meaning} as computed on dotnet ver ${runtimeBuildInfo.productVersion}`;
         console.debug(`ret: ${meaning}`);
     }
 
-    let exit_code = await runMain("Wasm.Browser.Sample.dll", []);
-    wasm_exit(exit_code);
+    let exit_code = await runMain(config.mainAssemblyName, []);
+    exit(exit_code);
 }
 catch (err) {
-    wasm_exit(2, err);
+    exit(2, err);
 }

src/mono/sample/wasm/console-node/Wasm.Console.Node.Sample.csproj

@@ -6,7 +6,7 @@
 
     <GenerateRunScriptForSample Condition="'$(ArchiveTests)' == 'true'">true</GenerateRunScriptForSample>
     <ExpectedExitCode>2</ExpectedExitCode>
-    <RunScriptCommand>$(ExecXHarnessCmd) wasm test --app=. --engine=NodeJS --engine-arg=--stack-trace-limit=1000 --js-file=main.mjs --output-directory=$(XHarnessOutput) --expected-exit-code $(ExpectedExitCode) -- --run $(MSBuildProjectName).dll</RunScriptCommand>
+    <RunScriptCommand>$(ExecXHarnessCmd) wasm test --app=. --engine=NodeJS --engine-arg=--stack-trace-limit=1000 --js-file=main.mjs --output-directory=$(XHarnessOutput) --expected-exit-code $(ExpectedExitCode)</RunScriptCommand>
   </PropertyGroup>
 
   <ItemGroup>

src/mono/sample/wasm/console-node/main.mjs

@@ -1,18 +1,6 @@
-// @ts-check
-// @ts-ignore
-import createDotnetRuntime from './dotnet.js'
-import process from 'process'
+import { dotnet } from './dotnet.js'
 
-/**
- * @type {import('../../../wasm/runtime/dotnet').CreateDotnetRuntimeType}
- */
-const createDotnetRuntimeTyped = createDotnetRuntime;
-
-const { runMainAndExit } = await createDotnetRuntimeTyped({
-    disableDotnet6Compatibility: true,
-    configSrc: "./mono-config.json",
-});
-
-const app_args = process.argv.slice(2);
-const dllName = "Wasm.Console.Node.Sample.dll";
-await runMainAndExit(dllName, app_args);
+dotnet
+    .withDiagnosticTracing(false)
+    .withApplicationArguments("dotnet", "is", "great!")
+    .run()

src/mono/sample/wasm/console-v8/Wasm.Console.V8.Sample.csproj

@@ -1,17 +1,13 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <WasmCopyAppZipToHelixTestDir Condition="'$(ArchiveTests)' == 'true'">true</WasmCopyAppZipToHelixTestDir>
-    <WasmMainJSPath>v8shim.cjs</WasmMainJSPath>
+    <WasmMainJSPath>main.mjs</WasmMainJSPath>
     <WasmGenerateRunV8Script>true</WasmGenerateRunV8Script>
 
     <GenerateRunScriptForSample Condition="'$(ArchiveTests)' == 'true'">true</GenerateRunScriptForSample>
-    <RunScriptCommand>$(ExecXHarnessCmd) wasm test --app=. --engine=V8 --engine-arg=--stack-trace-limit=1000 --js-file=v8shim.cjs --output-directory=$(XHarnessOutput) -- --run $(MSBuildProjectName).dll</RunScriptCommand>
+    <RunScriptCommand>$(ExecXHarnessCmd) wasm test --app=. --engine=V8 --engine-arg=--stack-trace-limit=1000 --engine-arg=--module --js-file=main.mjs --output-directory=$(XHarnessOutput) -- --run $(MSBuildProjectName).dll</RunScriptCommand>
   </PropertyGroup>
 
-  <ItemGroup>
-    <WasmExtraFilesToDeploy Include="main.mjs" />
-  </ItemGroup>
-
   <PropertyGroup>
     <_SampleProject>Wasm.Console.V8.Sample.csproj</_SampleProject>
     <_SampleAssembly>Wasm.Console.V8.Sample.dll</_SampleAssembly>

src/mono/sample/wasm/console-v8/main.mjs

@@ -1,11 +1,6 @@
-import createDotnetRuntime from './dotnet.js'
+import { dotnet } from './dotnet.js'
 
-const dllName = "Wasm.Console.V8.Sample.dll";
-const app_args = Array.from(arguments);
-
-async function main() {
-    const { runMainAndExit } = await createDotnetRuntime();
-    await runMainAndExit(dllName, app_args);
-}
-
-main();
+dotnet
+    .withDiagnosticTracing(false)
+    .withApplicationArguments(...arguments)
+    .run()

src/mono/sample/wasm/console-v8/v8shim.cjs

@@ -13,7 +13,6 @@ internal sealed class BrowserArguments
 {
     public string? HTMLPath { get; private set; }
     public bool? ForwardConsoleOutput { get; private set; }
-    public bool UseQueryStringToPassArguments { get; private set; }
     public string[] AppArgs { get; init; }
     public CommonConfiguration CommonConfig { get; init; }
 
@@ -27,8 +26,7 @@ public BrowserArguments(CommonConfiguration commonConfig)
 
     private OptionSet GetOptions() => new OptionSet
     {
-        { "forward-console", "Forward JS console output", v => ForwardConsoleOutput = true },
-        { "use-query-string-for-args", "Use query string to pass arguments (Default: false)", v => UseQueryStringToPassArguments = true }
+        { "forward-console", "Forward JS console output", v => ForwardConsoleOutput = true }
     };
 
     public void ParseJsonProperties(IDictionary<string, JsonElement>? properties)
@@ -37,8 +35,6 @@ public void ParseJsonProperties(IDictionary<string, JsonElement>? properties)
             HTMLPath = htmlPathElement.GetString();
         if (properties?.TryGetValue("forward-console", out JsonElement forwardConsoleElement) == true)
             ForwardConsoleOutput = forwardConsoleElement.GetBoolean();
-        if (properties?.TryGetValue("use-query-string-for-args", out JsonElement useQueryElement) == true)
-            UseQueryStringToPassArguments = useQueryElement.GetBoolean();
     }
 
     public void Validate()

src/mono/wasm/host/BrowserArguments.cs

@@ -76,8 +76,7 @@ private async Task RunAsync(ILoggerFactory loggerFactory, CancellationToken toke
                                                                            _args.CommonConfig.HostProperties.WebServerPort,
                                                                            token);
 
-        string[] fullUrls = BuildUrls(serverURLs,
-                                      _args.UseQueryStringToPassArguments ? _args.AppArgs : Array.Empty<string>());
+        string[] fullUrls = BuildUrls(serverURLs, _args.AppArgs);
         Console.WriteLine();
         foreach (string url in fullUrls)
             Console.WriteLine($"App url: {url}");