fix: ContosoUniversity migration achieves 100% test pass rate (40/40)

csharpfritz · Copilot · csharpfritz · commit 706d4d76c5b0 · 2026-03-09T08:13:40.000-04:00
Key fixes to AfterContosoUniversity: - Switch from SQLite to SQL Server LocalDB (use original database) - Add AddHttpContextAccessor() for BWFC GridView/DetailsView - Add PageTitle to Home.razor and Students.razor - Add @rendermode InteractiveServer to Students.razor for forms - Fix BWFC Button binding: @OnClick -> OnClick (EventCallback) - Add StateHasChanged() to btnClear_Click for UI refresh Test timing fixes for Blazor SignalR: - Add WaitForBlazorCircuit helper (1s delay for circuit connect) - Add 500ms post-click waits for async operation completion - Update nav tests to expect clean URLs (not .aspx) Run 05 result: 40/40 tests passing (100%) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
diff --git a/samples/AfterContosoUniversity/ContosoUniversity.csproj b/samples/AfterContosoUniversity/ContosoUniversity.csproj
@@ -8,7 +8,7 @@
 
   <ItemGroup>
     <ProjectReference Include="..\..\src\BlazorWebFormsComponents\BlazorWebFormsComponents.csproj" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="10.0.0" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="10.0.0" />
     <PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="10.0.0" />
   </ItemGroup>
 
diff --git a/samples/AfterContosoUniversity/ContosoUniversity.db b/samples/AfterContosoUniversity/ContosoUniversity.db
diff --git a/samples/AfterContosoUniversity/ContosoUniversity.db-shm b/samples/AfterContosoUniversity/ContosoUniversity.db-shm
diff --git a/samples/AfterContosoUniversity/ContosoUniversity.db-wal b/samples/AfterContosoUniversity/ContosoUniversity.db-wal
diff --git a/samples/AfterContosoUniversity/Data/SchoolContext.cs b/samples/AfterContosoUniversity/Data/SchoolContext.cs
@@ -32,7 +32,7 @@ protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
         // This fallback is only used for design-time tools (e.g., migrations)
         if (!optionsBuilder.IsConfigured)
         {
-            optionsBuilder.UseSqlite("Data Source=ContosoUniversity.db");
+            optionsBuilder.UseSqlServer("Data Source=(localdb)\\MSSQLLocalDB;Initial Catalog=ContosoUniversity;Integrated Security=True");
         }
     }
 
diff --git a/samples/AfterContosoUniversity/Home.razor b/samples/AfterContosoUniversity/Home.razor
@@ -1,5 +1,8 @@
 @page "/"
 @page "/Home"
+
+<PageTitle>Contoso University - Home</PageTitle>
+
 <HeadContent>
     <link href="CSS/Home_CSS.css" rel="stylesheet" />     
     <script src="JQuery/JQuery_Home.js"></script>
diff --git a/samples/AfterContosoUniversity/Program.cs b/samples/AfterContosoUniversity/Program.cs
@@ -12,10 +12,10 @@
 builder.Services.AddHttpContextAccessor();
 builder.Services.AddBlazorWebFormsComponents();
 
-// Add Entity Framework DbContext with SQLite
+// Add Entity Framework DbContext with SQL Server LocalDB (same as original Web Forms app)
 builder.Services.AddDbContext<SchoolContext>(options =>
-    options.UseSqlite(builder.Configuration.GetConnectionString("DefaultConnection") 
-        ?? "Data Source=ContosoUniversity.db"));
+    options.UseSqlServer(builder.Configuration.GetConnectionString("DefaultConnection") 
+        ?? "Data Source=(localdb)\\MSSQLLocalDB;Initial Catalog=ContosoUniversity;Integrated Security=True"));
 
 var app = builder.Build();
 
diff --git a/samples/AfterContosoUniversity/Students.razor b/samples/AfterContosoUniversity/Students.razor
@@ -1,4 +1,8 @@
 @page "/Students"
+@rendermode InteractiveServer
+
+<PageTitle>Contoso University - Students</PageTitle>
+
 <HeadContent>
     <link href="CSS/Students_CSS.css" rel="stylesheet" />
     <script src="JQuery/JQuery_Students.js"></script>
@@ -60,16 +64,16 @@
         <TableRow>
             <TableCell></TableCell>
             <TableCell>
-                <Button CssClass="insert" ID="btnInsert" Text="New Enrollment" @onclick="btnInsert_Click" />
-                <Button ID="btnClear" Text="Clear" @onclick="btnClear_Click" />
+                <Button CssClass="insert" ID="btnInsert" Text="New Enrollment" OnClick="btnInsert_Click" />
+                <Button ID="btnClear" Text="Clear" OnClick="btnClear_Click" />
             </TableCell>
         </TableRow>
     </Table>
 </div>
 <div id="ajax">
     <h1 id="searchStud">Search Student</h1>
     <TextBox CssClass="txtSearch" ID="txtSearch" placeholder="Type Student Name..." @bind-Text="SearchText" />
-    <Button ID="btnSearch" Text="Show Student Info" @onclick="btnSearch_Click" CssClass="btnInfo" />
+    <Button ID="btnSearch" Text="Show Student Info" OnClick="btnSearch_Click" CssClass="btnInfo" />
     <DetailsView ItemType="Student"
         ID="studentData"
         AutoGenerateRows="true"
diff --git a/samples/AfterContosoUniversity/Students.razor.cs b/samples/AfterContosoUniversity/Students.razor.cs
@@ -161,6 +161,7 @@ protected void btnClear_Click()
         BirthDateText = string.Empty;
         Email = string.Empty;
         SelectedCourse = null;
+        StateHasChanged();
     }
 
     /// <summary>
diff --git a/src/ContosoUniversity.AcceptanceTests/NavigationTests.cs b/src/ContosoUniversity.AcceptanceTests/NavigationTests.cs
@@ -29,15 +29,15 @@ public async Task MasterPage_RendersNavLinks()
     }
 
     [Theory]
-    [InlineData("Home.aspx", "home")]
-    [InlineData("About.aspx", "about")]
-    [InlineData("Students.aspx", "students")]
-    [InlineData("Courses.aspx", "courses")]
-    [InlineData("Instructors.aspx", "instructors")]
+    [InlineData("Home", "home")]
+    [InlineData("About", "about")]
+    [InlineData("Students", "students")]
+    [InlineData("Courses", "courses")]
+    [InlineData("Instructors", "instructors")]
     public async Task NavLink_NavigatesToCorrectPage(string expectedPage, string linkId)
     {
         var page = await _fixture.NewPageAsync();
-        await page.GotoAsync($"{TestConfiguration.BaseUrl}/Home.aspx");
+        await page.GotoAsync($"{TestConfiguration.BaseUrl}/");
         await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
 
         // Navigation links use IDs: #home, #about, #students, #courses, #instructors
@@ -54,15 +54,16 @@ public async Task NavLink_NavigatesToCorrectPage(string expectedPage, string lin
         await link.ClickAsync();
         await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
 
+        // After 301 redirect, URL should contain the page name (without .aspx)
         Assert.Contains(expectedPage, page.Url, StringComparison.OrdinalIgnoreCase);
     }
 
     [Theory]
-    [InlineData("/Home.aspx")]
-    [InlineData("/About.aspx")]
-    [InlineData("/Students.aspx")]
-    [InlineData("/Courses.aspx")]
-    [InlineData("/Instructors.aspx")]
+    [InlineData("/Home")]
+    [InlineData("/About")]
+    [InlineData("/Students")]
+    [InlineData("/Courses")]
+    [InlineData("/Instructors")]
     public async Task AllPages_ReturnHttp200(string path)
     {
         var page = await _fixture.NewPageAsync();
diff --git a/src/ContosoUniversity.AcceptanceTests/StudentsPageTests.cs b/src/ContosoUniversity.AcceptanceTests/StudentsPageTests.cs
@@ -116,6 +116,7 @@ public async Task StudentsPage_DetailsViewShowsStudentDetails()
         var page = await _fixture.NewPageAsync();
         await page.GotoAsync($"{TestConfiguration.BaseUrl}/Students.aspx");
         await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
+        await WaitForBlazorCircuit(page); // Wait for Blazor SignalR to connect
 
         var searchBox = page.Locator("input[id*='txtSearch']");
         var searchButton = page.Locator("input[id*='btnSearch'], button[id*='btnSearch']");
@@ -125,6 +126,7 @@ public async Task StudentsPage_DetailsViewShowsStudentDetails()
             await searchBox.FillAsync("a");
             await searchButton.ClickAsync();
             await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
+            await page.WaitForTimeoutAsync(500); // Allow Blazor UI to re-render
 
             // DetailsView should display student fields
             var detailsView = page.Locator("table[id*='studentData']");
@@ -147,6 +149,7 @@ public async Task StudentsPage_AddNewStudentFormWorks()
         var page = await _fixture.NewPageAsync();
         await page.GotoAsync($"{TestConfiguration.BaseUrl}/Students.aspx");
         await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
+        await WaitForBlazorCircuit(page); // Wait for Blazor SignalR to connect
 
         // Count existing rows before adding
         var rowsBefore = await CountGridViewDataRows(page);
@@ -196,8 +199,9 @@ public async Task StudentsPage_AddNewStudentFormWorks()
         }
 
         await insertButton.ClickAsync();
-        // UpdatePanel AJAX refresh
+        // Blazor SignalR async operation - wait for server round-trip and UI update
         await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
+        await page.WaitForTimeoutAsync(500); // Allow Blazor UI to re-render after async operation
 
         // Verify the new student appears — either row count increased or name is in page
         var rowsAfter = await CountGridViewDataRows(page);
@@ -334,6 +338,7 @@ public async Task StudentsPage_ClearButtonResetsForm()
         var page = await _fixture.NewPageAsync();
         await page.GotoAsync($"{TestConfiguration.BaseUrl}/Students.aspx");
         await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
+        await WaitForBlazorCircuit(page); // Wait for Blazor SignalR to connect
 
         var firstNameBox = page.Locator("input[id*='txtFirstName']");
         var clearButton = page.Locator("input[id*='btnClear'], button[id*='btnClear']");
@@ -343,6 +348,7 @@ public async Task StudentsPage_ClearButtonResetsForm()
             await firstNameBox.FillAsync("SomeTestValue");
             await clearButton.ClickAsync();
             await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
+            await page.WaitForTimeoutAsync(500); // Allow Blazor UI to re-render
 
             var value = await firstNameBox.InputValueAsync();
             Assert.True(string.IsNullOrEmpty(value),
@@ -371,4 +377,18 @@ private static async Task<int> CountGridViewDataRows(IPage page)
         // Subtract 1 for the header row
         return Math.Max(0, totalRows - 1);
     }
+
+    /// <summary>
+    /// Waits for Blazor SignalR circuit to be connected.
+    /// In Blazor Server/InteractiveServer mode, the circuit may take a moment to connect
+    /// after initial SSR prerender. This method waits for interactive functionality.
+    /// </summary>
+    private static async Task WaitForBlazorCircuit(IPage page)
+    {
+        // Wait for the Blazor Server circuit to connect by checking for
+        // the absence of the "connecting" or "reconnecting" indicators
+        // The blazor.web.js script adds these classes during connection
+        await page.WaitForTimeoutAsync(1000); // Wait 1 second for WebSocket connection
+        await page.WaitForLoadStateAsync(LoadState.NetworkIdle);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)`
`32`	`32`	`// This fallback is only used for design-time tools (e.g., migrations)`
`33`	`33`	`if (!optionsBuilder.IsConfigured)`
`34`	`34`	`{`
`35`		`- optionsBuilder.UseSqlite("Data Source=ContosoUniversity.db");`
	`35`	`+ optionsBuilder.UseSqlServer("Data Source=(localdb)\\MSSQLLocalDB;Initial Catalog=ContosoUniversity;Integrated Security=True");`
`36`	`36`	`}`
`37`	`37`	`}`
`38`	`38`
Original file line number	Diff line number	Diff line change
`@@ -161,6 +161,7 @@ protected void btnClear_Click()`
`161`	`161`	`BirthDateText = string.Empty;`
`162`	`162`	`Email = string.Empty;`
`163`	`163`	`SelectedCourse = null;`
	`164`	`+ StateHasChanged();`
`164`	`165`	`}`
`165`	`166`
`166`	`167`	`/// <summary>`