Rider Refactor

A final refactor for all of the changes made

src/Moq.INavigationService/MockNavigationService.cs

@@ -2,43 +2,44 @@
 namespace Moq;
 
 /// <summary>
-/// Mock Navigation Service, Capable of mocking the <see cref="PageNavigationService"/> during a unit test.
+/// Mock Navigation Service, Capable of mocking the <see cref="PageNavigationService" /> during a unit
+/// test.
 /// </summary>
 public class MockNavigationService : Mock<INavigationService>, INavigationService, IRegistryAware
 {
 	#region INavigationService
 
-	/// <inheritdoc/>
+	/// <inheritdoc />
 	public Task<INavigationResult> GoBackAsync(INavigationParameters parameters)
 	{
 		return Object.GoBackAsync(parameters);
 	}
 
-	/// <inheritdoc/>
+	/// <inheritdoc />
 	public Task<INavigationResult> GoBackAsync(string viewName, INavigationParameters parameters)
 	{
 		return Object.GoBackAsync(viewName, parameters);
 	}
 
-	/// <inheritdoc/>
+	/// <inheritdoc />
 	public Task<INavigationResult> GoBackToAsync(string name, INavigationParameters parameters)
 	{
 		return Object.GoBackToAsync(name, parameters);
 	}
 
-	/// <inheritdoc/>
+	/// <inheritdoc />
 	public Task<INavigationResult> GoBackToRootAsync(INavigationParameters parameters)
 	{
 		return Object.GoBackToRootAsync(parameters);
 	}
 
-	/// <inheritdoc/>
+	/// <inheritdoc />
 	public Task<INavigationResult> NavigateAsync(Uri uri, INavigationParameters parameters)
 	{
 		return Object.NavigateAsync(uri, parameters);
 	}
 
-	/// <inheritdoc/>
+	/// <inheritdoc />
 	public Task<INavigationResult> SelectTabAsync(string name, INavigationParameters parameters)
 	{
 		return Object.SelectTabAsync(name, parameters);
@@ -48,7 +49,7 @@ public Task<INavigationResult> SelectTabAsync(string name, INavigationParameters
 
 	#region IRegistryAware
 
-	/// <inheritdoc/>
+	/// <inheritdoc />
 	public IViewRegistry Registry { get; } = new MockViewRegistry();
 
 	#endregion IRegistryAware

src/Moq.INavigationService/SetupNavigationExtensions.cs

@@ -3,7 +3,7 @@
 namespace Moq;
 
 /// <summary>
-/// Setup Methods For The Mock <see cref="INavigationService"/>
+/// Setup Methods For The Mock <see cref="INavigationService" />
 /// </summary>
 public static class SetupNavigationExtensions
 {
@@ -26,7 +26,8 @@ public static void SetupAllNavigationReturns(this Mock<INavigationService> navig
 	}
 
 	/// <summary>
-	/// This setup method will ensure all navigation calls will return a failed <see cref="INavigationResult"/> with the given exception.
+	/// This setup method will ensure all navigation calls will return a failed
+	/// <see cref="INavigationResult" /> with the given exception.
 	/// </summary>
 	/// <param name="navigationServiceMock">The mock navigation service</param>
 	/// <param name="exception">The exception that caused the navigation failure</param>
@@ -45,7 +46,7 @@ public static void SetupAllNavigationFails(this Mock<INavigationService> navigat
 	}
 
 	/// <summary>
-	/// Setup Navigation For The Given <see cref="Uri"/>
+	/// Setup Navigation For The Given <see cref="Uri" />
 	/// </summary>
 	/// <param name="navigationServiceMock">The mock navigation service</param>
 	/// <param name="destination">The destination uri</param>
@@ -56,7 +57,7 @@ public static ISetup<INavigationService, Task<INavigationResult>> SetupNavigatio
 	}
 
 	/// <summary>
-	/// Setup Navigation For The Given <see cref="Uri"/> &amp; <see cref="INavigationParameters"/>
+	/// Setup Navigation For The Given <see cref="Uri" /> &amp; <see cref="INavigationParameters" />
 	/// </summary>
 	/// <param name="navigationServiceMock">The mock navigation service</param>
 	/// <param name="destination">The destination uri</param>
@@ -73,7 +74,10 @@ public static ISetup<INavigationService, Task<INavigationResult>> SetupNavigatio
 	/// <param name="navigationServiceMock">The mock navigation service</param>
 	/// <param name="destination">The destination string</param>
 	/// <returns>Moq Setup</returns>
-	/// <exception cref="NotSupportedException">This will be thrown if the string is not a valid <see cref="Uri"/></exception>
+	/// <exception cref="NotSupportedException">
+	/// This will be thrown if the string is not a valid
+	/// <see cref="Uri" />
+	/// </exception>
 	public static ISetup<INavigationService, Task<INavigationResult>> SetupNavigation(this Mock<INavigationService> navigationServiceMock, string destination)
 	{
 		if (!Uri.TryCreate(destination, UriKind.RelativeOrAbsolute, out var destinationUri))
@@ -85,13 +89,16 @@ public static ISetup<INavigationService, Task<INavigationResult>> SetupNavigatio
 	}
 
 	/// <summary>
-	/// Setup Navigation For The Given Destination String &amp; <see cref="INavigationParameters"/>
+	/// Setup Navigation For The Given Destination String &amp; <see cref="INavigationParameters" />
 	/// </summary>
 	/// <param name="navigationServiceMock">The mock navigation service</param>
 	/// <param name="destination">The destination string</param>
 	/// <param name="navigationParameters">The expected parameters</param>
 	/// <returns>Moq Setup</returns>
-	/// <exception cref="NotSupportedException">This will be thrown if the string is not a valid <see cref="Uri"/></exception>
+	/// <exception cref="NotSupportedException">
+	/// This will be thrown if the string is not a valid
+	/// <see cref="Uri" />
+	/// </exception>
 	public static ISetup<INavigationService, Task<INavigationResult>> SetupNavigation(this Mock<INavigationService> navigationServiceMock, string destination, INavigationParameters navigationParameters)
 	{
 		if (!Uri.TryCreate(destination, UriKind.RelativeOrAbsolute, out var destinationUri))

src/Moq.INavigationService/VerifyNavigationExtensions.cs

@@ -4,7 +4,7 @@
 namespace Moq;
 
 /// <summary>
-/// Setup Methods For The Mock <see cref="INavigationService"/>
+/// Setup Methods For The Mock <see cref="INavigationService" />
 /// </summary>
 public static class VerifyNavigationExtensions
 {

tests/Moq.INavigationService.Tests/EquivalenceHelperTests.cs

@@ -26,66 +26,65 @@ public void AreEquivalent_Should_DetermineCorrectly(object? a, object? b, bool e
 	{
 		{
 			// Both are null, expect true
-			null,
-			null,
-			true
+			null, null, true
 		},
 		{
 			// Both object are Uri, same value, expect true
-			new Uri("navigation://NavigationPage/HomePage"),
-			new Uri("navigation://NavigationPage/HomePage"),
-			true
+			new Uri("navigation://NavigationPage/HomePage"), new Uri("navigation://NavigationPage/HomePage"), true
 		},
 		{
 			// One object is Uri, other is null, expect false
-			new Uri("navigation://NavigationPage/HomePage"),
-			null,
-			false
+			new Uri("navigation://NavigationPage/HomePage"), null, false
 		},
 		{
 			// One object is Uri, other is null, expect false
-			null,
-			new Uri("navigation://NavigationPage/HomePage"),
-			false
+			null, new Uri("navigation://NavigationPage/HomePage"), false
 		},
 		{
 			// Both object are Uri, different values, expect false
-			new Uri("navigation://NavigationPage/HomePage"),
-			new Uri("https://www.github.com"),
-			false
+			new Uri("navigation://NavigationPage/HomePage"), new Uri("https://www.github.com"), false
 		},
 		{
 			// Both object are NavigationParameters, same values, expect true
-			new NavigationParameters()
+			new NavigationParameters
 			{
-				{ "KeyOne", "Hello World" },
+				{
+					"KeyOne", "Hello World"
+				},
 			},
-			new NavigationParameters()
+			new NavigationParameters
 			{
-				{ "KeyOne", "Hello World" },
+				{
+					"KeyOne", "Hello World"
+				},
 			},
 			true
 		},
 		{
 			// Both object are NavigationParameters, different values, expect false
-			new NavigationParameters()
+			new NavigationParameters
 			{
-				{ "KeyOne", "Hello World" },
+				{
+					"KeyOne", "Hello World"
+				},
 			},
-			new NavigationParameters()
+			new NavigationParameters
 			{
-				{ "ThisIsANumber", 1928 },
+				{
+					"ThisIsANumber", 1928
+				},
 			},
 			false
 		},
 		{
 			// One object is INavigationParameters, other is null, expect false
-			new NavigationParameters()
+			new NavigationParameters
 			{
-				{ "KeyOne", "Hello World" },
+				{
+					"KeyOne", "Hello World"
+				},
 			},
-			null,
-			false
-		}
+			null, false
+		},
 	};
 }