Use more string interpolation

Author: bkoelman

src/Examples/DapperExample/TranslationToSql/Builders/SqlQueryBuilder.cs

@@ -306,7 +306,7 @@ private void WriteColumn(ColumnNode column, bool isVirtualColumn, StringBuilder
         {
             foreach (char specialCharacter in SpecialCharactersInLike)
             {
-                safeValue = safeValue.Replace(specialCharacter.ToString(), @"\" + specialCharacter);
+                safeValue = safeValue.Replace(specialCharacter.ToString(), $@"\{specialCharacter}");
             }
         }
 

src/JsonApiDotNetCore/Configuration/ResourceGraphBuilder.cs

@@ -72,8 +72,8 @@ private static void SetRelationshipTypes(ResourceGraph resourceGraph)
 
             if (rightType == null)
             {
-                throw new InvalidConfigurationException($"Resource type '{relationship.LeftType.ClrType}' depends on " +
-                    $"'{rightClrType}', which was not added to the resource graph.");
+                throw new InvalidConfigurationException(
+                    $"Resource type '{relationship.LeftType.ClrType}' depends on '{rightClrType}', which was not added to the resource graph.");
             }
 
             relationship.RightType = rightType;
@@ -129,8 +129,8 @@ private static void ValidateAttributesInDerivedType(ResourceType resourceType)
         {
             if (resourceType.FindAttributeByPublicName(attribute.PublicName) == null)
             {
-                throw new InvalidConfigurationException($"Attribute '{attribute.PublicName}' from base type " +
-                    $"'{resourceType.BaseType.ClrType}' does not exist in derived type '{resourceType.ClrType}'.");
+                throw new InvalidConfigurationException(
+                    $"Attribute '{attribute.PublicName}' from base type '{resourceType.BaseType.ClrType}' does not exist in derived type '{resourceType.ClrType}'.");
             }
         }
     }
@@ -141,8 +141,8 @@ private static void ValidateRelationshipsInDerivedType(ResourceType resourceType
         {
             if (resourceType.FindRelationshipByPublicName(relationship.PublicName) == null)
             {
-                throw new InvalidConfigurationException($"Relationship '{relationship.PublicName}' from base type " +
-                    $"'{resourceType.BaseType.ClrType}' does not exist in derived type '{resourceType.ClrType}'.");
+                throw new InvalidConfigurationException(
+                    $"Relationship '{relationship.PublicName}' from base type '{resourceType.BaseType.ClrType}' does not exist in derived type '{resourceType.ClrType}'.");
             }
         }
     }

src/JsonApiDotNetCore/Middleware/JsonApiRoutingConvention.cs

@@ -101,8 +101,8 @@ public void Apply(ApplicationModel application)
 
                     if (resourceType == null)
                     {
-                        throw new InvalidConfigurationException($"Controller '{controller.ControllerType}' depends on " +
-                            $"resource type '{resourceClrType}', which does not exist in the resource graph.");
+                        throw new InvalidConfigurationException(
+                            $"Controller '{controller.ControllerType}' depends on resource type '{resourceClrType}', which does not exist in the resource graph.");
                     }
 
                     if (_controllerPerResourceTypeMap.TryGetValue(resourceType, out ControllerModel? existingModel))

src/JsonApiDotNetCore/Serialization/Request/Adapters/DocumentInOperationsRequestAdapter.cs

@@ -47,8 +47,7 @@ private void AssertMaxOperationsNotExceeded(IList<AtomicOperationObject?> atomic
         if (atomicOperationObjects.Count > _options.MaximumOperationsPerRequest)
         {
             throw new ModelConversionException(state.Position, "Too many operations in request.",
-                $"The number of operations in this request ({atomicOperationObjects.Count}) is higher " +
-                $"than the maximum of {_options.MaximumOperationsPerRequest}.");
+                $"The number of operations in this request ({atomicOperationObjects.Count}) is higher than the maximum of {_options.MaximumOperationsPerRequest}.");
         }
     }
 

src/JsonApiDotNetCore/Serialization/Request/NotSupportedExceptionExtensions.cs

@@ -48,7 +48,7 @@ public static JsonApiException EnrichSourcePointer(this NotSupportedException ex
                 }
                 else
                 {
-                    error.Source.Pointer = sourcePointer + '/' + error.Source.Pointer;
+                    error.Source.Pointer = $"{sourcePointer}/{error.Source.Pointer}";
                 }
             }
         }

test/DapperTests/UnitTests/LogicalCombinatorTests.cs

@@ -43,7 +43,7 @@ public void Collapses_and_filters()
             conditionRight3
         }.Select(condition => condition.ToString());
 
-        string expectedText = '(' + string.Join(") AND (", terms) + ')';
+        string expectedText = $"({string.Join(") AND (", terms)})";
         result.ToString().Should().Be(expectedText);
     }
 }

test/DapperTests/UnitTests/ParameterNodeTests.cs

@@ -40,6 +40,6 @@ public void Can_format_parameter(object? parameterValue, string formattedValueEx
         string text = parameter.ToString();
 
         // Assert
-        text.Should().Be("@name = " + formattedValueExpected);
+        text.Should().Be($"@name = {formattedValueExpected}");
     }
 }

test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Pagination/PaginationWithTotalCountTests.cs

@@ -592,9 +592,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
             await dbContext.SaveChangesAsync();
         });
 
-        string routePrefix = $"/blogPosts?filter=equals(author.userName,'{account.UserName}')" +
-            "&fields[webAccounts]=userName&include=author&sort=id&foo=bar,baz";
-
+        string routePrefix = $"/blogPosts?filter=equals(author.userName,'{account.UserName}')&fields[webAccounts]=userName&include=author&sort=id&foo=bar,baz";
         string route = $"{routePrefix}&page[number]={pageNumber}";
 
         // Act