Lombiq · Psichorex · Feb 21, 2024 · Jan 18, 2024 · Jan 19, 2024 · Jan 19, 2024
diff --git a/Lombiq.Tests.UI.AppExtensions/Lombiq.Tests.UI.AppExtensions.csproj b/Lombiq.Tests.UI.AppExtensions/Lombiq.Tests.UI.AppExtensions.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net6.0</TargetFramework>
+    <TargetFramework>net8.0</TargetFramework>
     <IsPublishable>false</IsPublishable>
     <DefaultItemExcludes>$(DefaultItemExcludes);.git*;node_modules\**</DefaultItemExcludes>
   </PropertyGroup>
@@ -33,8 +33,8 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="OrchardCore" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Email.Abstractions" Version="1.7.0" />
+    <PackageReference Include="OrchardCore" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Email.Abstractions" Version="1.8.2" />
   </ItemGroup>
 
 </Project>
diff --git a/Lombiq.Tests.UI.Samples/Helpers/SetupHelpers.cs b/Lombiq.Tests.UI.Samples/Helpers/SetupHelpers.cs
@@ -78,7 +78,7 @@ private static void AssertSetupSuccessful(UITestContext context)
         {
             var validationErrors = context.GetAll(By.ClassName("field-validation-error"));
 
-            if (!validationErrors.Any()) throw;
+            if (validationErrors.Count == 0) throw;
 
             var errors = "\n- " + validationErrors.Select(element => element.Text.Trim()).Join("\n- ");
             throw new AssertionException($"Setup has failed with the following validation errors:{errors}");

diff --git a/Lombiq.Tests.UI.Samples/Lombiq.Tests.UI.Samples.csproj b/Lombiq.Tests.UI.Samples/Lombiq.Tests.UI.Samples.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net6.0</TargetFramework>
+    <TargetFramework>net8.0</TargetFramework>
     <IsPackable>false</IsPackable>
     <!-- Eliminates problem with dotnet publish caused by duplicate .htmlvalidate.json file. See
          https://docs.microsoft.com/en-us/dotnet/core/compatibility/sdk/6.0/duplicate-files-in-output. -->

diff --git a/Lombiq.Tests.UI.Shortcuts/Controllers/ErrorController.cs b/Lombiq.Tests.UI.Shortcuts/Controllers/ErrorController.cs
@@ -2,7 +2,6 @@
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
 using System;
-using System.Diagnostics.CodeAnalysis;
 
 namespace Lombiq.Tests.UI.Shortcuts.Controllers;
 
@@ -11,8 +10,6 @@ public class ErrorController : Controller
 {
     public const string ExceptionMessage = "This action intentionally causes an exception!";
 
-    // This only happens in the CI for some reason.
-    [SuppressMessage("Usage", "CA1822:Mark members as static", Justification = "It's a controller action.")]
     [AllowAnonymous]
     public IActionResult Index() =>
         throw new InvalidOperationException(ExceptionMessage);

diff --git a/Lombiq.Tests.UI.Shortcuts/Lombiq.Tests.UI.Shortcuts.csproj b/Lombiq.Tests.UI.Shortcuts/Lombiq.Tests.UI.Shortcuts.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk.Razor">
 
   <PropertyGroup>
-    <TargetFramework>net6.0</TargetFramework>
+    <TargetFramework>net8.0</TargetFramework>
     <IsPublishable>false</IsPublishable>
     <AddRazorSupportForMvc>true</AddRazorSupportForMvc>
     <DefaultItemExcludes>$(DefaultItemExcludes);.git*;node_modules\**</DefaultItemExcludes>
@@ -34,23 +34,23 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="OrchardCore.Module.Targets" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Tenants" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Themes" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Users" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Media.Abstractions" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Recipes.Abstractions" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Abstractions" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.ResourceManagement.Abstractions" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Workflows" Version="1.7.0" />
+    <PackageReference Include="OrchardCore.Module.Targets" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Tenants" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Themes" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Users" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Media.Abstractions" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Recipes.Abstractions" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Abstractions" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.ResourceManagement.Abstractions" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Workflows" Version="1.8.2" />
   </ItemGroup>
 
   <ItemGroup Condition="'$(NuGetBuild)' != 'true'">
     <ProjectReference Include="..\..\..\src\Libraries\Lombiq.HelpfulLibraries\Lombiq.HelpfulLibraries.OrchardCore\Lombiq.HelpfulLibraries.OrchardCore.csproj" />
   </ItemGroup>
 
   <ItemGroup Condition="'$(NuGetBuild)' == 'true'">
-    <PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="8.1.1-alpha.5.tdeal-16" />
+    <PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="8.1.1-alpha.7.osoe-751" />
   </ItemGroup>
 
 </Project>
diff --git a/Lombiq.Tests.UI.Shortcuts/Manifest.cs b/Lombiq.Tests.UI.Shortcuts/Manifest.cs
@@ -14,15 +14,15 @@
     Category = "Development",
     Description = "WARNING: Only enable this feature in the UI testing environment. Provides shortcuts for common " +
         "operations that UI tests might want to do or check.",
-    Dependencies = new[]
-    {
+    Dependencies =
+    [
         "OrchardCore.ContentManagement",
         "OrchardCore.ContentTypes",
         "OrchardCore.DisplayManagement",
         "OrchardCore.Roles",
         "OrchardCore.Tenants",
         "OrchardCore.Users",
-    }
+    ]
 )]
 
 [assembly: Feature(
@@ -37,19 +37,19 @@
     Name = "Media Cache Purge - Shortcuts - Lombiq UI Testing Toolbox",
     Category = "Development",
     Description = "WARNING: Only enable this feature in the UI testing environment. Provides shortcut for Media Cache Purge.",
-    Dependencies = new[]
-    {
+    Dependencies =
+    [
         "OrchardCore.Media.Cache",
-    }
+    ]
 )]
 
 [assembly: Feature(
     Id = Workflows,
     Name = "Workflows - Shortcuts - Lombiq UI Testing Toolbox",
     Category = "Development",
     Description = "WARNING: Only enable this feature in the UI testing environment. Provides shortcut for Workflows.",
-    Dependencies = new[]
-    {
+    Dependencies =
+    [
         "OrchardCore.Workflows.Http",
-    }
+    ]
 )]
diff --git a/Lombiq.Tests.UI.Tests.UI/Lombiq.Tests.UI.Tests.UI.csproj b/Lombiq.Tests.UI.Tests.UI/Lombiq.Tests.UI.Tests.UI.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net6.0</TargetFramework>
+    <TargetFramework>net8.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
   </PropertyGroup>

diff --git a/Lombiq.Tests.UI/Extensions/BasicOrchardFeaturesTestingUITestContextExtensions.cs b/Lombiq.Tests.UI/Extensions/BasicOrchardFeaturesTestingUITestContextExtensions.cs
@@ -659,9 +659,9 @@ await context
     public static Task ExecuteTestAsync(
         this UITestContext context, string testName, Func<Task> testFunctionAsync)
     {
-        if (context is null) throw new ArgumentNullException(nameof(context));
-        if (testName is null) throw new ArgumentNullException(nameof(testName));
-        if (testFunctionAsync is null) throw new ArgumentNullException(nameof(testFunctionAsync));
+        ArgumentNullException.ThrowIfNull(context);
+        ArgumentNullException.ThrowIfNull(testName);
+        ArgumentNullException.ThrowIfNull(testFunctionAsync);
 
         return ExecuteTestInnerAsync(context, testName, testFunctionAsync);
     }

diff --git a/Lombiq.Tests.UI/Extensions/ExtendedLoggingExtensions.cs b/Lombiq.Tests.UI/Extensions/ExtendedLoggingExtensions.cs
@@ -131,7 +131,7 @@ private static TResult ExecuteSection<TResult>(this UITestContext context, LogSe
             throw new InvalidOperationException("Impossible to reach.");
         });
 
-    private static Task ExecuteSectionAsync(this UITestContext context, LogSection section, Func<Task> functionAsync) =>
+    private static Task<bool> ExecuteSectionAsync(this UITestContext context, LogSection section, Func<Task> functionAsync) =>
         context.ExecuteSectionAsync(
             section,
             async () =>

diff --git a/Lombiq.Tests.UI/Extensions/FormUITestContextExtensions.cs b/Lombiq.Tests.UI/Extensions/FormUITestContextExtensions.cs
@@ -110,8 +110,7 @@ public static void FillInMonacoEditor(
     {
         var script = $@"
             monaco.editor.getEditors().find((element) =>
-                element.getContainerDomNode().id == {JsonConvert.SerializeObject(editorId)})
-            .getModel().setValue({JsonConvert.SerializeObject(text)});";
+                element.getContainerDomNode().id == {JsonConvert.SerializeObject(editorId)}).setValue({JsonConvert.SerializeObject(text)});";
 
         context.ExecuteScript(script);
     }
@@ -125,8 +124,7 @@ public static string GetMonacoEditorText(
     {
         var script = $@"
             return monaco.editor.getEditors().find((element) =>
-                element.getContainerDomNode().id == {JsonConvert.SerializeObject(editorId)})
-            .getModel().getValue();";
+                element.getContainerDomNode().id == {JsonConvert.SerializeObject(editorId)}).getValue();";
 
         return context.ExecuteScript(script) as string;
     }

diff --git a/Lombiq.Tests.UI/Extensions/ShortcutsUITestContextExtensions.cs b/Lombiq.Tests.UI/Extensions/ShortcutsUITestContextExtensions.cs
@@ -342,10 +342,7 @@ public static async Task PurgeMediaCacheDirectlyAsync(this UITestContext context
     public static Task<ApplicationInfo> GetApplicationInfoAsync(this UITestContext context) =>
         context.GetApi().GetApplicationInfoFromApiAsync();
 
-    // This is required to instantiate ILogger<>.
-#pragma warning disable S2094 // Classes should not be empty
     private sealed class ExecuteRecipeShortcut { }
-#pragma warning restore S2094 // Classes should not be empty
 
     /// <summary>
     /// Executes a recipe identified by its name directly.
@@ -646,7 +643,7 @@ internal static async Task WaitInteractiveModeAsync(this UITestContext context)
     }
 
     private static bool IsAdminTheme(IManifestInfo manifest) =>
-        manifest.Tags.Any(tag => tag.EqualsOrdinalIgnoreCase(ManifestConstants.AdminTag));
+        manifest.Tags.Any(tag => tag.EqualsOrdinalIgnoreCase(value: ManifestConstants.AdminTag));
 
     private static async Task<bool> PermissionExistsAsync(
         IEnumerable<IPermissionProvider> permissionProviders, string permissionName) =>

diff --git a/Lombiq.Tests.UI/Extensions/ShouldlyExtensions.cs b/Lombiq.Tests.UI/Extensions/ShouldlyExtensions.cs
@@ -51,7 +51,7 @@ public static void ShouldBeEmptyWhen<TItem, TMessage>(
         JsonSerializerOptions jsonSerializerOptions = null)
     {
         var results = enumerable.Where(condition).ToList();
-        if (!results.Any()) return;
+        if (results.Count == 0) return;
 
         var message = messageTransform == null
             ? JsonSerializer.Serialize(results, jsonSerializerOptions)

diff --git a/Lombiq.Tests.UI/Extensions/TypedRouteUITestContextExtensions.cs b/Lombiq.Tests.UI/Extensions/TypedRouteUITestContextExtensions.cs
@@ -43,11 +43,16 @@ public static string GetRelativeUrlOfAction<TController>(
         this UITestContext context,
         Expression<Action<TController>> actionExpression,
         params (string Key, object Value)[] additionalArguments)
-        where TController : ControllerBase =>
-        TypedRoute
-            .CreateFromExpression(actionExpression, additionalArguments, CreateServiceProvider(context))
+        where TController : ControllerBase
+    {
+        using var serviceProvider = CreateServiceProvider(context);
+        var route = TypedRoute
+            .CreateFromExpression(actionExpression, additionalArguments, serviceProvider)
             .ToString();
 
+        return route;
+    }
+
     /// <summary>
     /// Gets the relative URL generated by <see cref="TypedRoute"/> for the <paramref name="actionExpressionAsync"/> in
     /// the <typeparamref name="TController"/>.
@@ -57,11 +62,17 @@ public static string GetRelativeUrlOfAction<TController>(
         this UITestContext context,
         Expression<Func<TController, Task>> actionExpressionAsync,
         params (string Key, object Value)[] additionalArguments)
-        where TController : ControllerBase =>
-        TypedRoute
-            .CreateFromExpression(actionExpressionAsync.StripResult(), additionalArguments, CreateServiceProvider(context))
+        where TController : ControllerBase
+    {
+        using var serviceProvider = CreateServiceProvider(context);
+
+        var route = TypedRoute
+            .CreateFromExpression(actionExpressionAsync.StripResult(), additionalArguments, serviceProvider)
             .ToString();
 
+        return route;
+    }
+
     /// <summary>
     /// Gets the absolute URL generated by <see cref="TypedRoute"/> for the <paramref name="actionExpression"/> in the
     /// <typeparamref name="TController"/>.
@@ -84,7 +95,7 @@ public static Uri GetAbsoluteUrlOfAction<TController>(
         where TController : ControllerBase =>
         context.GetAbsoluteUri(context.GetRelativeUrlOfAction(actionExpressionAsync, additionalArguments));
 
-    private static IServiceProvider CreateServiceProvider(UITestContext context)
+    private static ServiceProvider CreateServiceProvider(UITestContext context)
     {
         var services = new ServiceCollection();
 

diff --git a/Lombiq.Tests.UI/Extensions/VisualVerificationUITestContextExtensions.cs b/Lombiq.Tests.UI/Extensions/VisualVerificationUITestContextExtensions.cs
@@ -82,7 +82,7 @@ public static void AssertVisualVerificationOnAllResolutions(
 
         if (exceptions.Count == 1) throw exceptions.Single();
 
-        if (exceptions.Any())
+        if (exceptions.Count != 0)
         {
             // The UITestExecutionSession doesn't support AggregateException with multiple inner exceptions, so we just
             // concatenate the exceptions if there are multiple.

diff --git a/Lombiq.Tests.UI/Extensions/WebApplicationInstanceExtensions.cs b/Lombiq.Tests.UI/Extensions/WebApplicationInstanceExtensions.cs
@@ -37,7 +37,7 @@ public static async Task LogsShouldBeEmptyAsync(
                 .SplitByNewLines()
                 .Where(line => line.Contains("|ERROR|"));
 
-            if (permittedErrorLines.Any())
+            if (permittedErrorLines.Count != 0)
             {
                 errorLines = errorLines.Where(line => !permittedErrorLines.Any(line.ContainsOrdinalIgnoreCase));
             }

diff --git a/Lombiq.Tests.UI/Helpers/WebAppConfigHelper.cs b/Lombiq.Tests.UI/Helpers/WebAppConfigHelper.cs
@@ -5,15 +5,17 @@ namespace Lombiq.Tests.UI.Helpers;
 
 public static class WebAppConfigHelper
 {
+    private static readonly string[] Separator = ["src", "test"];
+
     /// <summary>
     /// Retrieves the absolute path to the assembly (DLL) of the application being tested.
     /// </summary>
     /// <param name="webAppName">The web app's project name.</param>
     /// <param name="frameworkFolderName">
-    /// The name of the folder that corresponds to the .NET version in the build output folder (e.g. "net6.0").
+    /// The name of the folder that corresponds to the .NET version in the build output folder (e.g. "net8.0").
     /// </param>
     /// <returns>The absolute path to the assembly (DLL) of the application being tested.</returns>
-    public static string GetAbsoluteApplicationAssemblyPath(string webAppName, string frameworkFolderName = "net6.0")
+    public static string GetAbsoluteApplicationAssemblyPath(string webAppName, string frameworkFolderName = "net8.0")
     {
         string baseDirectory;
 
@@ -24,7 +26,7 @@ public static string GetAbsoluteApplicationAssemblyPath(string webAppName, strin
         else
         {
             var outputFolderContainingPath = Path.Combine(
-                AppContext.BaseDirectory.Split(new[] { "src", "test" }, StringSplitOptions.RemoveEmptyEntries)[0],
+                AppContext.BaseDirectory.Split(Separator, StringSplitOptions.RemoveEmptyEntries)[0],
                 "src",
                 webAppName,
                 "bin");

diff --git a/Lombiq.Tests.UI/Lombiq.Tests.UI.csproj b/Lombiq.Tests.UI/Lombiq.Tests.UI.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net6.0</TargetFramework>
+    <TargetFramework>net8.0</TargetFramework>
     <IsPublishable>false</IsPublishable>
     <DefaultItemExcludes>$(DefaultItemExcludes);.git*</DefaultItemExcludes>
     <!-- With these the driver exe files will be copied to all projects referencing this project. Otherwise all child
@@ -67,15 +67,15 @@
     <PackageReference Include="Azure.Storage.Blobs" Version="12.18.0" />
     <PackageReference Include="Ben.Demystifier" Version="0.4.1" />
     <PackageReference Include="Codeuctivity.ImageSharpCompare" Version="3.0.183" />
-    <PackageReference Include="Microsoft.Extensions.Configuration.EnvironmentVariables" Version="7.0.0" />
-    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="7.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.EnvironmentVariables" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
     <PackageReference Include="Microsoft.SqlServer.DACFx" Version="162.0.52" />
     <PackageReference Include="Microsoft.SqlServer.SqlManagementObjects" Version="170.18.0" />
     <PackageReference Include="Mono.Posix.NETStandard" Version="1.0.0" />
-    <PackageReference Include="OrchardCore.ContentFields" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Logging.NLog" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Abstractions" Version="1.7.0" />
-    <PackageReference Include="OrchardCore.Recipes.Core" Version="1.7.0" />
+    <PackageReference Include="OrchardCore.ContentFields" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Logging.NLog" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Abstractions" Version="1.8.2" />
+    <PackageReference Include="OrchardCore.Recipes.Core" Version="1.8.2" />
     <PackageReference Include="Refit.HttpClientFactory" Version="7.0.0" />
     <PackageReference Include="Sarif.Sdk" Version="4.3.7" />
     <PackageReference Include="Selenium.Axe" Version="4.0.14" />
@@ -102,10 +102,10 @@
   <Import Condition="'$(NuGetBuild)' != 'true'" Project="..\..\..\src\Utilities\Lombiq.Npm.Targets\Lombiq.Npm.Targets.targets" />
 
   <ItemGroup Condition="'$(NuGetBuild)' == 'true'">
-    <PackageReference Include="Lombiq.Tests" Version="2.2.5" />
-    <PackageReference Include="Lombiq.HelpfulLibraries.Cli" Version="8.1.1-alpha.5.tdeal-16" />
-    <PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="8.1.1-alpha.5.tdeal-16" />
-    <PackageReference Include="Lombiq.HelpfulLibraries.Refit" Version="8.1.1-alpha.5.tdeal-16" />
+    <PackageReference Include="Lombiq.Tests" Version="2.2.6-alpha.0.osoe-751" />
+    <PackageReference Include="Lombiq.HelpfulLibraries.Cli" Version="8.1.1-alpha.7.osoe-751" />
+    <PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="8.1.1-alpha.7.osoe-751" />
+    <PackageReference Include="Lombiq.HelpfulLibraries.Refit" Version="8.1.1-alpha.7.osoe-751" />
     <PackageReference Include="Lombiq.Npm.Targets" Version="1.4.0" />
   </ItemGroup>
 

diff --git a/Lombiq.Tests.UI/Models/FailureDumpItem.cs b/Lombiq.Tests.UI/Models/FailureDumpItem.cs
@@ -20,10 +20,7 @@ public FailureDumpItem(
 
     public Task<Stream> GetStreamAsync()
     {
-        if (_disposed)
-        {
-            throw new ObjectDisposedException(nameof(FailureDumpItem));
-        }
+        ObjectDisposedException.ThrowIf(_disposed, this);
 
         return _getStream();
     }

diff --git a/Lombiq.Tests.UI/Models/FailureDumpItemGeneric.cs b/Lombiq.Tests.UI/Models/FailureDumpItemGeneric.cs
@@ -28,10 +28,7 @@ public FailureDumpItemGeneric(
 
     public Task<Stream> GetStreamAsync()
     {
-        if (_disposed)
-        {
-            throw new ObjectDisposedException(nameof(FailureDumpItemGeneric<TContent>));
-        }
+        ObjectDisposedException.ThrowIf(_disposed, this);
 
         if (_content is Stream stream && _getStream == null)
         {

diff --git a/Lombiq.Tests.UI/Models/InstanceCommandLineArgs.cs b/Lombiq.Tests.UI/Models/InstanceCommandLineArgs.cs
@@ -5,7 +5,7 @@ namespace Lombiq.Tests.UI.Models;
 
 public class InstanceCommandLineArgumentsBuilder
 {
-    private readonly List<string> _arguments = new();
+    private readonly List<string> _arguments = [];
 
     public IEnumerable<string> Arguments => _arguments;