Changed to "new()" syntax where possible and changelog update

Author: egil

CHANGELOG.md

@@ -14,6 +14,14 @@ List of new features.
 ### Changed
 List of changes in existing functionality.
 
+- bUnit's mock IJSRuntime has been moved to an "always on" state by default, in strict mode, and is now available through `TestContext`'s `JSInterop` property. This makes it possible for first party Blazor components like the `<Virtualize>` component, which depend on JSInterop, to "just work" in tests.
+
+  **Compatible with previous releases:** To get the same effect as calling `Services.AddMockJSRuntime()` in beta-11, which used to add the mock IJSRuntime in "loose" mode, you now just need to change the mode of the already on JSInterop, i.e. `ctx.JSInterop.Mode = JSRuntimeMode.Loose`.
+
+  **Inspect registered handlers:** Since the new design allows registering invoke handlers in the context of the `TestContext`, you might need to get already registered handlers in your individual tests. This can be done with the `TryGetInvokeHandler()` method, that will return handler that can handle the parameters passed to it. E.g. to get a handler for a `IJSRuntime.InvokaAsync<string>("getValue")`, call `ctx.JSInterop.TryGetInvokeHandler<string>("getValue")`.
+
+  Learn more [issue #237](https://github.com/egil/bUnit/issues/237). By [@egil](https://github.com/egil) in [#247](https://github.com/egil/bUnit/pull/247).
+
 ### Removed
 List of now removed features.
 

src/bunit.core/ComponentParameterCollectionBuilder.cs

@@ -26,7 +26,7 @@ public sealed class ComponentParameterCollectionBuilder<TComponent> where TCompo
 				.OfType<ParameterAttribute>()
 				.Any(x => x.CaptureUnmatchedValues);
 
-		private readonly ComponentParameterCollection _parameters = new ComponentParameterCollection();
+		private readonly ComponentParameterCollection _parameters = new();
 
 		/// <summary>
 		/// Creates an instance of the <see cref="ComponentParameterCollectionBuilder{TComponent}"/>.

src/bunit.core/Rendering/RenderTreeFrameCollection.cs

@@ -8,7 +8,7 @@ namespace Bunit.Rendering
 	/// </summary>
 	public sealed class RenderTreeFrameCollection
 	{
-		private readonly Dictionary<int, ArrayRange<RenderTreeFrame>> _currentRenderTree = new Dictionary<int, ArrayRange<RenderTreeFrame>>();
+		private readonly Dictionary<int, ArrayRange<RenderTreeFrame>> _currentRenderTree = new();
 
 		/// <summary>
 		/// Gets the <see cref="ArrayRange{RenderTreeFrame}"/> associated with the <paramref name="componentId"/>.

src/bunit.core/Rendering/TestRenderer.cs

@@ -14,11 +14,11 @@ namespace Bunit.Rendering
 	/// </summary>
 	public partial class TestRenderer : Renderer, ITestRenderer
 	{
-		private readonly object _renderTreeAccessLock = new object();
+		private readonly object _renderTreeAccessLock = new();
 		private readonly ILogger _logger;
 		private readonly IRenderedComponentActivator _activator;
 		private Exception? _unhandledException;
-		private readonly Dictionary<int, IRenderedFragmentBase> _renderedComponents = new Dictionary<int, IRenderedFragmentBase>();
+		private readonly Dictionary<int, IRenderedFragmentBase> _renderedComponents = new();
 
 		/// <inheritdoc/>
 		public override Dispatcher Dispatcher { get; } = Dispatcher.CreateDefault();

src/bunit.web/Diffing/DiffMarkupFormatter.cs

@@ -13,7 +13,7 @@ public class DiffMarkupFormatter : PrettyMarkupFormatter, IMarkupFormatter
 		/// <summary>
 		/// Gets an instance of the <see cref="DiffMarkupFormatter"/>.
 		/// </summary>
-		public new static readonly DiffMarkupFormatter Instance = new DiffMarkupFormatter();
+		public new static readonly DiffMarkupFormatter Instance = new();
 
 		/// <summary>
 		/// Creates an instance of the <see cref="DiffMarkupFormatter"/>.

src/bunit.web/EventDispatchExtensions/Key.cs

@@ -566,7 +566,7 @@ public static implicit operator KeyboardEventArgs(Key key)
 		}
 
 		// This has to be placed last since it is referencing other static fields, that must be initialized first.
-		private static readonly Dictionary<(string value, string code), Key> PredefinedKeys = new Dictionary<(string value, string code), Key>
+		private static readonly Dictionary<(string value, string code), Key> PredefinedKeys = new()
 		{
 			{ (Key.Backspace.Value, Key.Backspace.Code), Key.Backspace },
 			{ (Key.Tab.Value, Key.Tab.Code), Key.Tab },

src/bunit.web/EventDispatchExtensions/TriggerEventDispatchExtensions.cs

@@ -16,8 +16,8 @@ namespace Bunit
 	/// </summary>
 	public static class TriggerEventDispatchExtensions
 	{
-		private static readonly HashSet<string> NonBubblingEvents = new HashSet<string> { "onabort", "onblur", "onchange", "onerror", "onfocus", "onload", "onloadend", "onloadstart", "onmouseenter", "onmouseleave", "onprogress", "onreset", "onscroll", "onsubmit", "onunload", "ontoggle", "ondomnodeinsertedintodocument", "ondomnoderemovedfromdocument" };
-		private static readonly HashSet<string> DisabledEventNames = new HashSet<string> { "onclick", "ondblclick", "onmousedown", "onmousemove", "onmouseup" };
+		private static readonly HashSet<string> NonBubblingEvents = new() { "onabort", "onblur", "onchange", "onerror", "onfocus", "onload", "onloadend", "onloadstart", "onmouseenter", "onmouseleave", "onprogress", "onreset", "onscroll", "onsubmit", "onunload", "ontoggle", "ondomnodeinsertedintodocument", "ondomnoderemovedfromdocument" };
+		private static readonly HashSet<string> DisabledEventNames = new() { "onclick", "ondblclick", "onmousedown", "onmousemove", "onmouseup" };
 
 		/// <summary>
 		/// Raises the event <paramref name="eventName"/> on the element <paramref name="element"/>

src/bunit.web/JSInterop/BunitJSInterop.cs

@@ -13,8 +13,8 @@ namespace Bunit
 	/// </summary>
 	public class BunitJSInterop
 	{
-		private readonly Dictionary<string, List<JSRuntimeInvocation>> _invocations = new Dictionary<string, List<JSRuntimeInvocation>>();
-		private readonly Dictionary<string, List<object>> _handlers = new Dictionary<string, List<object>>();
+		private readonly Dictionary<string, List<JSRuntimeInvocation>> _invocations = new();
+		private readonly Dictionary<string, List<object>> _handlers = new();
 
 		/// <summary>
 		/// Gets a dictionary of all <see cref="List{JSRuntimeInvocation}"/> this mock has observed.

src/bunit.web/RazorTesting/Fixture.cs

@@ -11,7 +11,7 @@ namespace Bunit
 	/// <inheritdoc/>
 	public class Fixture : FixtureBase<Fixture>
 	{
-		private readonly Dictionary<string, IRenderedFragment> _renderedFragments = new Dictionary<string, IRenderedFragment>();
+		private readonly Dictionary<string, IRenderedFragment> _renderedFragments = new();
 		private IReadOnlyList<FragmentBase>? _testData;
 
 		private IReadOnlyList<FragmentBase> TestData

src/bunit.web/Rendering/BunitHtmlParser.cs

@@ -24,7 +24,7 @@ public sealed class BunitHtmlParser : IDisposable
 
 		private readonly IBrowsingContext _context;
 		private readonly IHtmlParser _htmlParser;
-		private readonly List<IDocument> _documents = new List<IDocument>();
+		private readonly List<IDocument> _documents = new();
 
 		/// <summary>
 		/// Creates an instance of the parser with a AngleSharp context