Bump Roslynator.Analyzers from 4.16.1 to 5.0.0 (#19)

* Bump Roslynator.Analyzers from 4.16.1 to 5.0.0

---
updated-dependencies:
- dependency-name: Roslynator.Analyzers
  dependency-version: 5.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: analyzers
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: migrated editorconfig rules to Roslynator.Analyzers 5.0.0 and fixed/suppressed new found violations

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: David Liebeherr <info@rent-a-developer.de>

Author: dependabot[bot]

.editorconfig

@@ -1,4 +1,6 @@
-[*.cs]
+root = true
+
+[*.cs]
 max_line_length = 120
 charset = utf-8
 
@@ -173,3 +175,18 @@ dotnet_diagnostic.S4456.severity = none
 
 [*.ps1]
 end_of_line = crlf
+
+[{tests,benchmarks}/**/*.cs]
+
+dotnet_diagnostic.S2344.severity = none
+dotnet_diagnostic.S3010.severity = none
+dotnet_diagnostic.S4144.severity = none
+dotnet_diagnostic.S3459.severity = none
+dotnet_diagnostic.S3453.severity = none
+dotnet_diagnostic.S1144.severity = none
+dotnet_diagnostic.S1172.severity = none
+dotnet_diagnostic.S3963.severity = none
+dotnet_diagnostic.S6562.severity = none
+dotnet_diagnostic.S2925.severity = none
+dotnet_diagnostic.S5034.severity = none
+dotnet_diagnostic.S2696.severity = none

Directory.Build.props

@@ -33,7 +33,7 @@
 			<PrivateAssets>all</PrivateAssets>
 			<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
 		</PackageReference>
-		<PackageReference Include="Roslynator.Analyzers" Version="4.16.1">
+		<PackageReference Include="Roslynator.Analyzers" Version="5.0.0">
 			<PrivateAssets>all</PrivateAssets>
 			<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
 		</PackageReference>

benchmarks/DbConnectionPlus.Benchmarks/Benchmarks.Query_Dynamic.cs

@@ -53,7 +53,7 @@ public List<dynamic> Query_Dynamic_Command()
                 ["ByteValue"] = dataReader.GetByte(ordinal++),
                 ["CharValue"] = dataReader.GetChars(ordinal++, 0, charBuffer, 0, 1) == 1
                     ? charBuffer[0]
-                    : throw new(),
+                    : throw new InvalidOperationException(),
                 ["DateTimeValue"] = DateTime.Parse(dataReader.GetString(ordinal++), CultureInfo.InvariantCulture),
                 ["DecimalValue"] = Decimal.Parse(dataReader.GetString(ordinal++), CultureInfo.InvariantCulture),
                 ["DoubleValue"] = dataReader.GetDouble(ordinal++),
@@ -62,7 +62,7 @@ public List<dynamic> Query_Dynamic_Command()
                 ["Int32Value"] = (Int32)dataReader.GetInt64(ordinal++),
                 ["Int64Value"] = dataReader.GetInt64(ordinal++),
                 ["SingleValue"] = dataReader.GetFloat(ordinal++),
-                ["StringValue"] = dataReader.GetString(ordinal++)
+                ["StringValue"] = dataReader.GetString(ordinal)
             };
 
             entities.Add(new DataRow(dictionary));

benchmarks/DbConnectionPlus.Benchmarks/Benchmarks.cs

@@ -77,7 +77,7 @@ private static BenchmarkEntity ReadEntity(IDataReader dataReader)
             BooleanValue = dataReader.GetInt64(ordinal++) == 1,
             BytesValue = (Byte[])dataReader.GetValue(ordinal++),
             ByteValue = dataReader.GetByte(ordinal++),
-            CharValue = dataReader.GetChars(ordinal++, 0, charBuffer, 0, 1) == 1 ? charBuffer[0] : throw new(),
+            CharValue = dataReader.GetChars(ordinal++, 0, charBuffer, 0, 1) == 1 ? charBuffer[0] : throw new InvalidOperationException(),
             DateTimeValue = DateTime.Parse(dataReader.GetString(ordinal++), CultureInfo.InvariantCulture),
             DecimalValue = Decimal.Parse(dataReader.GetString(ordinal++), CultureInfo.InvariantCulture),
             DoubleValue = dataReader.GetDouble(ordinal++),

tests/DbConnectionPlus.IntegrationTests/DatabaseAdapters/EntityManipulator.InsertEntityTests.cs

@@ -109,7 +109,7 @@ await this.CallApi(
             TestContext.Current.CancellationToken
         );
 
-        this.Connection.QueryFirst<MappingTestEntityAttributes>($"SELECT * FROM {Q("MappingTestEntity")}")
+        (await this.Connection.QueryFirstAsync<MappingTestEntityAttributes>($"SELECT * FROM {Q("MappingTestEntity")}"))
             .Should().BeEquivalentTo(
                 entity,
                 options => options.Using<String>(context => context.Subject.Should().BeNull())
@@ -137,7 +137,7 @@ await this.CallApi(
             TestContext.Current.CancellationToken
         );
 
-        this.Connection.QueryFirst<MappingTestEntityFluentApi>($"SELECT * FROM {Q("MappingTestEntity")}")
+        (await this.Connection.QueryFirstAsync<MappingTestEntityFluentApi>($"SELECT * FROM {Q("MappingTestEntity")}"))
             .Should().BeEquivalentTo(
                 entity,
                 options => options.Using<String>(context => context.Subject.Should().BeNull())
@@ -160,7 +160,7 @@ await this.CallApi(
             TestContext.Current.CancellationToken
         );
 
-        this.Connection.QueryFirst<MappingTestEntity>($"SELECT * FROM {Q("MappingTestEntity")}")
+        (await this.Connection.QueryFirstAsync<MappingTestEntity>($"SELECT * FROM {Q("MappingTestEntity")}"))
             .Should().BeEquivalentTo(entity);
     }
 

tests/DbConnectionPlus.IntegrationTests/DatabaseAdapters/EntityManipulator.UpdateEntityTests.cs

@@ -195,7 +195,7 @@ await this.CallApi(
             TestContext.Current.CancellationToken
         );
 
-        this.Connection.QueryFirst<MappingTestEntityAttributes>($"SELECT * FROM {Q("MappingTestEntity")}")
+        (await this.Connection.QueryFirstAsync<MappingTestEntityAttributes>($"SELECT * FROM {Q("MappingTestEntity")}"))
             .Should().BeEquivalentTo(
                 updatedEntity,
                 options => options.Using<String>(context => context.Subject.Should().BeNull())
@@ -225,7 +225,7 @@ await this.CallApi(
             TestContext.Current.CancellationToken
         );
 
-        this.Connection.QueryFirst<MappingTestEntityFluentApi>($"SELECT * FROM {Q("MappingTestEntity")}")
+        (await this.Connection.QueryFirstAsync<MappingTestEntityFluentApi>($"SELECT * FROM {Q("MappingTestEntity")}"))
             .Should().BeEquivalentTo(
                 updatedEntity,
                 options => options.Using<String>(context => context.Subject.Should().BeNull())
@@ -271,7 +271,7 @@ await this.CallApi(
             TestContext.Current.CancellationToken
         );
 
-        this.Connection.QueryFirst<MappingTestEntity>($"SELECT * FROM {Q("MappingTestEntity")}")
+        (await this.Connection.QueryFirstAsync<MappingTestEntity>($"SELECT * FROM {Q("MappingTestEntity")}"))
             .Should().BeEquivalentTo(updatedEntity);
     }
 

tests/DbConnectionPlus.IntegrationTests/DbConnectionExtensions.QueryFirstOrDefaultTests.cs

@@ -200,7 +200,7 @@ Boolean useAsyncApi
         dataRow
             .Should().NotBeNull();
 
-        dataRow!["Id"]
+        dataRow["Id"]
             .Should().Be(entityIds[0]);
 
         this.ExistsTemporaryTableInDb(temporaryTableName)

tests/DbConnectionPlus.IntegrationTests/DbConnectionExtensions.QuerySingleOrDefaultTests.cs

@@ -221,7 +221,7 @@ Boolean useAsyncApi
         dataRow
             .Should().NotBeNull();
 
-        dataRow!["Id"]
+        dataRow["Id"]
             .Should().Be(entityId);
 
         this.ExistsTemporaryTableInDb(temporaryTableName)

tests/DbConnectionPlus.IntegrationTests/DbConnectionPlus.IntegrationTests.csproj

@@ -20,7 +20,7 @@
 		<PackageReference Include="Npgsql" Version="10.0.3" />
 		<PackageReference Include="NSubstitute" Version="5.3.0" />
 		<PackageReference Include="Oracle.ManagedDataAccess.Core" Version="23.26.300" />
-		<PackageReference Include="System.Linq.Async" Version="7.0.1" />
+		<PackageReference Include="System.Linq.AsyncEnumerable" Version="10.0.11" />
 		<!--
 			The four database servers the suite tests against. Testcontainers starts each container on demand and
 			removes it when the run ends - see TestDatabase/Containers/. Testcontainers.XunitV3, not

tests/DbConnectionPlus.IntegrationTests/IntegrationTestsBase.cs

@@ -238,7 +238,7 @@ SELECT 1
              FROM   {Q(metadata.TableName)}
              WHERE  {String.Join(
                      " AND ",
-                     [.. keyProperties.Select(p => $"{Q(p.ColumnName)} = {P(p.PropertyName)}")]
+                     keyProperties.Select(p => $"{Q(p.ColumnName)} = {P(p.PropertyName)}").ToList()
                  )}
              """,
             keyProperties.Select(p => (p.PropertyName, p.PropertyGetter!(entity))).ToArray()!
@@ -387,7 +387,9 @@ private void InterceptDbCommand(DbCommand command, IReadOnlyList<InterpolatedTem
     /// <returns>A <see cref="CancellationToken" /> that will be cancelled after 100 milliseconds.</returns>
     protected static CancellationToken CreateCancellationTokenThatIsCancelledAfter100Milliseconds()
     {
+#pragma warning disable S2930
         var cancellationTokenSource = new CancellationTokenSource();
+#pragma warning restore S2930
         cancellationTokenSource.CancelAfter(100);
         return cancellationTokenSource.Token;
     }
@@ -397,7 +399,9 @@ protected static CancellationToken CreateCancellationTokenThatIsCancelledAfter10
     /// <summary>
     /// The connection to the test database for the currently running integration test.
     /// </summary>
+#pragma warning disable S2743
     private static readonly AsyncLocal<DbConnection> currentTestDatabaseConnection = new();
+#pragma warning restore S2743
 
     /// <summary>
     /// The database adapter for the test database of the currently running integration test.
@@ -406,5 +410,7 @@ protected static CancellationToken CreateCancellationTokenThatIsCancelledAfter10
     /// The adapter rather than the provider: an interface that declares a static abstract member - which
     /// <see cref="ITestDatabaseProvider.StartDatabaseAsync" /> is - cannot be used as a type argument.
     /// </remarks>
+#pragma warning disable S2743
     private static readonly AsyncLocal<IDatabaseAdapter> currentDatabaseAdapter = new();
+#pragma warning restore S2743
 }