Refactor: Make IndexHtmlFragments properties immutable

jsakamoto · jsakamoto · commit 8dbe23082952 · 2026-05-01T07:43:37.000+09:00
diff --git a/BlazorWasmPreRendering.Build.Shared/IndexHtmlFragments.cs b/BlazorWasmPreRendering.Build.Shared/IndexHtmlFragments.cs
@@ -2,9 +2,11 @@
 
 public class IndexHtmlFragments
 {
-    public string FirstPart { get; set; } = "";
-    public string MiddlePart { get; set; } = "";
-    public string LastPart { get; set; } = "";
+    public string FirstPart { get; }
+
+    public string MiddlePart { get; }
+
+    public string LastPart { get; }
 
     public IndexHtmlFragments(string firstPart, string middlePart, string lastPart)
     {
diff --git a/BlazorWasmPreRendering.Build.Shared/ServerSideRenderingOptions.cs b/BlazorWasmPreRendering.Build.Shared/ServerSideRenderingOptions.cs
@@ -22,7 +22,7 @@ public class ServerSideRenderingOptions
 
     public RenderMode RenderMode { get; init; }
 
-    public IndexHtmlFragments IndexHtmlFragments { get; set; } = new("", "", "");
+    public IndexHtmlFragments IndexHtmlFragments { get; init; } = new("", "", "");
 
     public bool DeleteLoadingContents { get; init; }
 
diff --git a/BlazorWasmPreRendering.Build.Test/BuildProgramTests.cs b/BlazorWasmPreRendering.Build.Test/BuildProgramTests.cs
@@ -73,11 +73,11 @@ public void StoreOptionsToEnvironment_Test()
             DeleteLoadingContents = true,
             Environment = "Production",
             EmulateAuthMe = false,
-            IndexHtmlFragments = {
-                FirstPart = "<html>\r\n<head>\r\n",
-                LastPart = "</body>\r\n</html>",
-                MiddlePart = "</head>\r\n<body>\r\n",
-            },
+            IndexHtmlFragments = new(
+                firstPart: "<html>\r\n<head>\r\n",
+                lastPart: "</body>\r\n</html>",
+                middlePart: "</head>\r\n<body>\r\n"
+            ),
             Locales = { "en", "ja" },
             MiddlewareDllsDir = "C:\\project\\obj\\Release\\net6.0\\middleware",
             MiddlewarePackages = {
@@ -126,11 +126,11 @@ public void StoreOptionsToEnvironment_NoMiddlewarePackages_Test()
             DeleteLoadingContents = false,
             Environment = "Development",
             EmulateAuthMe = true,
-            IndexHtmlFragments = {
-                FirstPart = "<html>\r\n<head>\r\n",
-                LastPart = "</body>\r\n</html>",
-                MiddlePart = "</head>\r\n<body>\r\n"
-            },
+            IndexHtmlFragments = new(
+                firstPart: "<html>\r\n<head>\r\n",
+                lastPart: "</body>\r\n</html>",
+                middlePart: "</head>\r\n<body>\r\n"
+            ),
             MiddlewareDllsDir = "C:\\project\\obj\\Release\\net6.0\\middleware",
             RenderMode = RenderMode.Static,
             RootComponentTypeName = "Project.App",

Original file line number	Diff line number	Diff line change
`@@ -2,9 +2,11 @@`
`2`	`2`
`3`	`3`	`public class IndexHtmlFragments`
`4`	`4`	`{`
`5`		`- public string FirstPart { get; set; } = "";`
`6`		`- public string MiddlePart { get; set; } = "";`
`7`		`- public string LastPart { get; set; } = "";`
	`5`	`+ public string FirstPart { get; }`
	`6`	`+`
	`7`	`+ public string MiddlePart { get; }`
	`8`	`+`
	`9`	`+ public string LastPart { get; }`
`8`	`10`
`9`	`11`	`public IndexHtmlFragments(string firstPart, string middlePart, string lastPart)`
`10`	`12`	`{`