From ced751318932d8a23e6aa4fbc0cbb35204084cf8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?D=C3=A1vid=20El-Saig?= Date: Fri, 18 Mar 2022 10:49:36 +0100 Subject: [PATCH 1/4] Make ShellFeaturesController AllowAnonymous. --- .../Controllers/ShellFeaturesController.cs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Lombiq.Tests.UI.Shortcuts/Controllers/ShellFeaturesController.cs b/Lombiq.Tests.UI.Shortcuts/Controllers/ShellFeaturesController.cs index e8d28a1e7..6d22994f4 100644 --- a/Lombiq.Tests.UI.Shortcuts/Controllers/ShellFeaturesController.cs +++ b/Lombiq.Tests.UI.Shortcuts/Controllers/ShellFeaturesController.cs @@ -1,4 +1,5 @@ using Lombiq.HelpfulLibraries.Libraries.Mvc; +using Microsoft.AspNetCore.Authorization; using Microsoft.AspNetCore.Mvc; using OrchardCore.Environment.Extensions; using OrchardCore.Environment.Extensions.Features; @@ -10,6 +11,7 @@ namespace Lombiq.Tests.UI.Shortcuts.Controllers; [DevelopmentAndLocalhostOnly] +[AllowAnonymous] public class ShellFeaturesController : Controller { private readonly IShellFeaturesManager _shellFeatureManager; From a72484fef3ab6c36257d7f96c2516744ed3c498c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?D=C3=A1vid=20El-Saig?= Date: Sun, 6 Mar 2022 19:30:08 +0100 Subject: [PATCH 2/4] Fix webroot argument that somehow corrupted the following environment argument. (cherry picked from commit e70011d658f26249a623c683da208dc0e17ee7a5) # Conflicts: # Lombiq.Tests.UI/Services/OrchardCoreInstance.cs --- Lombiq.Tests.UI/Services/OrchardCoreInstance.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs b/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs index 9a36bab9a..4577bdcd1 100644 --- a/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs +++ b/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs @@ -141,7 +141,7 @@ public async Task StartUpAsync() argumentsBuilder = argumentsBuilder .Add("--urls").Add(url) .Add("--contentRoot").Add(_contentRootPath) - .Add("--webroot=").Add(Path.Combine(_contentRootPath, "wwwroot")) + .Add("--webroot=" + Path.Combine(_contentRootPath, "wwwroot")) .Add("--environment").Add("Development"); if (!useExeToExecuteApp) argumentsBuilder = argumentsBuilder.Add(_configuration.AppAssemblyPath); From 74b537af566fcf314c982a2e46b108ffe97c3a6a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?D=C3=A1vid=20El-Saig?= Date: Fri, 18 Mar 2022 12:03:54 +0100 Subject: [PATCH 3/4] Fix this test we just broke. --- Lombiq.Tests.UI.Samples/Tests/BasicTests.cs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Lombiq.Tests.UI.Samples/Tests/BasicTests.cs b/Lombiq.Tests.UI.Samples/Tests/BasicTests.cs index 75fb3ee58..898a33653 100644 --- a/Lombiq.Tests.UI.Samples/Tests/BasicTests.cs +++ b/Lombiq.Tests.UI.Samples/Tests/BasicTests.cs @@ -126,6 +126,9 @@ public Task ApplicationInsightsTrackingShouldBePresent(Browser browser) => ExecuteTestAfterSetupAsync( async context => { + await context.EnableFeatureDirectlyAsync("Lombiq.Privacy.ConsentBanner"); + await context.GoToHomePageAsync(); + // Now there's a bit of a pickle though: The Lombiq Privacy module is also enabled from the test recipe // and shows its privacy consent banner. For tracking to be enabled, even in offline mode, the user // needs to give consent. This is what we do now: From 1ff5bb3883c5cec24c5068657e5319eb6938d7f3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?D=C3=A1vid=20El-Saig?= Date: Mon, 21 Mar 2022 17:38:03 +0100 Subject: [PATCH 4/4] Make argument usage consistent. --- Lombiq.Tests.UI/Services/OrchardCoreInstance.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs b/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs index 4577bdcd1..6ad4fc531 100644 --- a/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs +++ b/Lombiq.Tests.UI/Services/OrchardCoreInstance.cs @@ -141,7 +141,7 @@ public async Task StartUpAsync() argumentsBuilder = argumentsBuilder .Add("--urls").Add(url) .Add("--contentRoot").Add(_contentRootPath) - .Add("--webroot=" + Path.Combine(_contentRootPath, "wwwroot")) + .Add("--webroot").Add(Path.Combine(_contentRootPath, "wwwroot")) .Add("--environment").Add("Development"); if (!useExeToExecuteApp) argumentsBuilder = argumentsBuilder.Add(_configuration.AppAssemblyPath);