E2E test for enhanced nav auto-disabling itself for non-Blazor endpoints

SteveSandersonMS · SteveSandersonMS · commit b4db2f5bc95b · 2023-09-11T10:09:03.000+01:00
diff --git a/src/Components/test/E2ETest/ServerRenderingTests/EnhancedNavigationTest.cs b/src/Components/test/E2ETest/ServerRenderingTests/EnhancedNavigationTest.cs
@@ -8,6 +8,7 @@
 using Xunit.Abstractions;
 using Components.TestServer.RazorComponents;
 using OpenQA.Selenium;
+using OpenQA.Selenium.Support.Extensions;
 
 namespace Microsoft.AspNetCore.Components.E2ETests.ServerRenderingTests;
 
@@ -307,6 +308,18 @@ public void RefreshWithForceReloadDoesFullPageReload(string renderMode)
         Assert.EndsWith("/nav", Browser.Url);
     }
 
+    [Fact]
+    public void EnhancedNavNotUsedForNonBlazorDestinations()
+    {
+        Navigate($"{ServerPathBase}/nav");
+        Browser.Equal("Hello", () => Browser.Exists(By.TagName("h1")).Text);
+        Assert.Equal("object", Browser.ExecuteJavaScript<string>("return typeof Blazor")); // Blazor JS is loaded
+
+        Browser.Exists(By.TagName("nav")).FindElement(By.LinkText("Non-Blazor HTML page")).Click();
+        Browser.Equal("This is a non-Blazor endpoint", () => Browser.Exists(By.TagName("h1")).Text);
+        Assert.Equal("undefined", Browser.ExecuteJavaScript<string>("return typeof Blazor")); // Blazor JS is NOT loaded
+    }
+
     private static bool IsElementStale(IWebElement element)
     {
         try
diff --git a/src/Components/test/testassets/Components.TestServer/RazorComponentEndpointsStartup.cs b/src/Components/test/testassets/Components.TestServer/RazorComponentEndpointsStartup.cs
@@ -145,6 +145,13 @@ private static void MapEnhancedNavigationEndpoints(IEndpointRouteBuilder endpoin
         endpoints.MapGet("redirect/nonblazor/get", PerformRedirection);
         endpoints.MapPost("redirect/nonblazor/post", PerformRedirection);
 
+        // Used when testing enhanced navigation to non-Blazor endpoints
+        endpoints.Map("/nav/non-blazor-html-response", async (HttpResponse response) =>
+        {
+            response.ContentType = "text/html";
+            await response.WriteAsync("<html><body><h1>This is a non-Blazor endpoint</h1><p>That's all</p></body></html>");
+        });
+
         static Task PerformRedirection(HttpRequest request, HttpResponse response)
         {
             response.Redirect(request.Query["external"] == "true"
diff --git a/src/Components/test/testassets/Components.TestServer/RazorComponents/Shared/EnhancedNavLayout.razor b/src/Components/test/testassets/Components.TestServer/RazorComponents/Shared/EnhancedNavLayout.razor
@@ -7,6 +7,7 @@
     <NavLink href="nav/give-404-with-content">Error page with 404 content</NavLink> |
     <NavLink href="nav/nonexistent-page">Error page with no content</NavLink> |
     <NavLink href="nav/non-html-response">Non-HTML page</NavLink> |
+    <NavLink href="nav/non-blazor-html-response">Non-Blazor HTML page</NavLink> |
     <NavLink href="nav/scroll-to-hash#some-content">Scroll to hash</NavLink> |
     <NavLink href="nav/throw-while-streaming">Error while streaming</NavLink> |
     <NavLink href="nav/interactive-component-navigation/server">Interactive component navigation (server)</NavLink> |
