[release/9.0] [browser] Make boot json file name configurable (#108708)

* [browser] Make boot json file name configurable

* Fix test based on state in 9

* Avoid need for change in SDK repo

---------

Co-authored-by: Larry Ewing <[email protected]>
Co-authored-by: Jeff Schwartz <[email protected]>

Author: 3 people

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -199,6 +199,10 @@ Copyright (c) .NET Foundation. All rights reserved.
       <_WasmFingerprintAssets Condition="'$(_WasmFingerprintAssets)' == ''">false</_WasmFingerprintAssets>
       <_WasmFingerprintDotnetJs>$(WasmFingerprintDotnetJs)</_WasmFingerprintDotnetJs>
       <_WasmFingerprintDotnetJs Condition="'$(_WasmFingerprintDotnetJs)' == ''">false</_WasmFingerprintDotnetJs>
+      <_WasmBootConfigFileName>$(WasmBootConfigFileName)</_WasmBootConfigFileName>
+      <_WasmBootConfigFileName Condition="'$(_WasmBootConfigFileName)' == ''">blazor.boot.json</_WasmBootConfigFileName>
+      <_WasmPublishBootConfigFileName Condition="'$(_WasmBootConfigFileName)' == 'blazor.boot.json'">blazor.publish.boot.json</_WasmPublishBootConfigFileName>
+      <_WasmPublishBootConfigFileName Condition="'$(_WasmPublishBootConfigFileName)' == ''">publish.$(_WasmBootConfigFileName)</_WasmPublishBootConfigFileName>
 
       <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
       <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -327,7 +331,7 @@ Copyright (c) .NET Foundation. All rights reserved.
 
   <Target Name="_GenerateBuildWasmBootJson" DependsOnTargets="$(GenerateBuildWasmBootJsonDependsOn)">
     <PropertyGroup>
-      <_WasmBuildBootJsonPath>$(IntermediateOutputPath)blazor.boot.json</_WasmBuildBootJsonPath>
+      <_WasmBuildBootJsonPath>$(IntermediateOutputPath)$(_WasmBootConfigFileName)</_WasmBuildBootJsonPath>
     </PropertyGroup>
 
     <ItemGroup>
@@ -392,13 +396,13 @@ Copyright (c) .NET Foundation. All rights reserved.
     </ItemGroup>
 
     <PropertyGroup>
-      <_WasmBuildBootJsonPath>$(IntermediateOutputPath)blazor.boot.json</_WasmBuildBootJsonPath>
+      <_WasmBuildBootJsonPath>$(IntermediateOutputPath)$(_WasmBootConfigFileName)</_WasmBuildBootJsonPath>
     </PropertyGroup>
 
     <ItemGroup>
       <_BuildWasmBootJson
         Include="$(_WasmBuildBootJsonPath)"
-        RelativePath="_framework/blazor.boot.json" />
+        RelativePath="_framework/$(_WasmBootConfigFileName)" />
     </ItemGroup>
 
     <DefineStaticWebAssets
@@ -589,8 +593,8 @@ Copyright (c) .NET Foundation. All rights reserved.
   <Target Name="_AddPublishWasmBootJsonToStaticWebAssets" DependsOnTargets="GeneratePublishWasmBootJson">
     <ItemGroup>
       <_PublishWasmBootJson
-        Include="$(IntermediateOutputPath)blazor.publish.boot.json"
-        RelativePath="_framework/blazor.boot.json" />
+        Include="$(IntermediateOutputPath)$(_WasmPublishBootConfigFileName)"
+        RelativePath="_framework/$(_WasmBootConfigFileName)" />
     </ItemGroup>
 
     <DefineStaticWebAssets
@@ -659,7 +663,7 @@ Copyright (c) .NET Foundation. All rights reserved.
       DebugLevel="$(WasmDebugLevel)"
       LinkerEnabled="$(PublishTrimmed)"
       CacheBootResources="$(BlazorCacheBootResources)"
-      OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
+      OutputPath="$(IntermediateOutputPath)$(_WasmPublishBootConfigFileName)"
       ConfigurationFiles="@(_WasmPublishConfigFile)"
       LazyLoadedAssemblies="@(BlazorWebAssemblyLazyLoad)"
       InvariantGlobalization="$(InvariantGlobalization)"
@@ -679,7 +683,7 @@ Copyright (c) .NET Foundation. All rights reserved.
       FingerprintAssets="$(_WasmFingerprintAssets)" />
 
     <ItemGroup>
-      <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />
+      <FileWrites Include="$(IntermediateOutputPath)$(_WasmPublishBootConfigFileName)" />
     </ItemGroup>
 
   </Target>

src/mono/wasm/Wasm.Build.Tests/AssertWasmSdkBundleOptions.cs

@@ -13,7 +13,7 @@ public record AssertWasmSdkBundleOptions(
     string BinFrameworkDir,
     string? PredefinedIcudt,
     GlobalizationMode GlobalizationMode = GlobalizationMode.Sharded,
-    string BootJsonFileName = "blazor.boot.json",
+    string BootConfigFileName = "blazor.boot.json",
     NativeFilesType ExpectedFileType = NativeFilesType.FromRuntimePack,
     RuntimeVariant RuntimeType = RuntimeVariant.SingleThreaded,
     bool ExpectFingerprintOnDotnetJs = false,
@@ -29,7 +29,7 @@ public record AssertWasmSdkBundleOptions(
                GlobalizationMode: GlobalizationMode,
                ExpectedFileType: ExpectedFileType,
                RuntimeType: RuntimeType,
-               BootJsonFileName: BootJsonFileName,
+               BootJsonFileName: BootConfigFileName,
                ExpectFingerprintOnDotnetJs: ExpectFingerprintOnDotnetJs,
                ExpectSymbolsFile: ExpectSymbolsFile,
                AssertIcuAssets: AssertIcuAssets,

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorBuildOptions.cs

@@ -11,6 +11,7 @@ public record BlazorBuildOptions
     string Config,
     NativeFilesType ExpectedFileType = NativeFilesType.FromRuntimePack,
     string TargetFramework = BuildTestBase.DefaultTargetFrameworkForBlazor,
+    string BootConfigFileName = "blazor.boot.json",
     bool IsPublish = false,
     bool WarnAsError = true,
     bool ExpectSuccess = true,

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmProjectProvider.cs

@@ -16,6 +16,7 @@ public BlazorWasmProjectProvider(ITestOutputHelper _testOutput, string? _project
     public void AssertBundle(BlazorBuildOptions options)
         => AssertBundle(new AssertWasmSdkBundleOptions(
                 Config: options.Config,
+                BootConfigFileName: options.BootConfigFileName,
                 IsPublish: options.IsPublish,
                 TargetFramework: options.TargetFramework,
                 BinFrameworkDir: options.BinFrameworkDir ?? FindBinFrameworkDir(options.Config, options.IsPublish, options.TargetFramework),

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -60,6 +60,7 @@ protected void BuildProject(
         RuntimeVariant runtimeType = RuntimeVariant.SingleThreaded,
         bool assertAppBundle = true,
         bool expectSuccess = true,
+        string bootConfigFileName = "blazor.boot.json",
         params string[] extraArgs)
     {
         (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(
@@ -68,6 +69,7 @@ protected void BuildProject(
             BinFrameworkDir: binFrameworkDir,
             RuntimeType: runtimeType,
             AssertAppBundle: assertAppBundle,
+            BootConfigFileName: bootConfigFileName,
             ExpectSuccess: expectSuccess), extraArgs);
         if (expectSuccess)
         {
@@ -83,12 +85,14 @@ protected void PublishProject(
         string configuration,
         RuntimeVariant runtimeType = RuntimeVariant.SingleThreaded,
         bool assertAppBundle = true,
+        string bootConfigFileName = "blazor.boot.json",
         params string[] extraArgs)
     {
         (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(
             Id: Id,
             Config: configuration,
             RuntimeType: runtimeType,
+            BootConfigFileName: bootConfigFileName,
             AssertAppBundle: assertAppBundle), extraArgs);
         result.EnsureSuccessful();
     }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/ModuleConfigTests.cs

@@ -74,4 +74,33 @@ public async Task OutErrOverrideWorks()
             "Emscripten err override doesn't work"
         );
     }
+
+    [Theory]
+    [InlineData("Release", true)]
+    [InlineData("Release", false)]
+    public async Task OverrideBootConfigName(string config, bool isPublish)
+    {
+        CopyTestAsset("WasmBasicTestApp", $"OverrideBootConfigName", "App");
+
+        string[] extraArgs = ["-p:WasmBootConfigFileName=boot.json"];
+        if (isPublish)
+            PublishProject(config, bootConfigFileName: "boot.json", extraArgs: extraArgs);
+        else
+            BuildProject(config, bootConfigFileName: "boot.json", extraArgs: extraArgs);
+
+        var runOptions = new RunOptions(
+            Configuration: config,
+            TestScenario: "OverrideBootConfigName"
+        );
+        var result = await (isPublish
+            ? RunSdkStyleAppForPublish(runOptions)
+            : RunSdkStyleAppForBuild(runOptions)
+        );
+
+        Assert.Collection(
+            result.TestOutput,
+            m => Assert.Equal("ConfigSrc: boot.json", m),
+            m => Assert.Equal("Managed code has run", m)
+        );
+    }
 }

src/mono/wasm/testassets/WasmBasicTestApp/App/OverrideBootConfigNameTest.cs

@@ -0,0 +1,10 @@
+using System.Runtime.InteropServices.JavaScript;
+
+public partial class OverrideBootConfigNameTest
+{
+    [JSExport]
+    public static void Run()
+    {
+        TestOutput.WriteLine("Managed code has run");
+    }
+}

src/mono/wasm/testassets/WasmBasicTestApp/App/wwwroot/main.js

@@ -126,9 +126,12 @@ switch (testCase) {
         };
         dotnet.withConfig({ maxParallelDownloads: maxParallelDownloads });
         break;
+    case "OverrideBootConfigName":
+        dotnet.withConfigSrc("boot.json");
+        break;
 }
 
-const { setModuleImports, getAssemblyExports, getConfig, INTERNAL } = await dotnet.create();
+const { setModuleImports, Module, getAssemblyExports, getConfig, INTERNAL } = await dotnet.create();
 const config = getConfig();
 const exports = await getAssemblyExports(config.mainAssemblyName);
 const assemblyExtension = Object.keys(config.resources.coreAssembly)[0].endsWith('.wasm') ? ".wasm" : ".dll";
@@ -206,6 +209,11 @@ try {
             exports.MemoryTest.Run();
             exit(0);
             break;
+            break;
+        case "OverrideBootConfigName":
+            testOutput("ConfigSrc: " + Module.configSrc);
+            exports.OverrideBootConfigNameTest.Run();
+            exit(0);
         default:
             console.error(`Unknown test case: ${testCase}`);
             exit(3);