feat: add net9.0 support for tests

  - Unit tests and integration tests now target net8.0 and net9.0
  - CI installs .NET 8 and 9 SDK for integration tests

Author: killme2008

.github/workflows/ci.yml

@@ -39,6 +39,7 @@ jobs:
             6.0.x
             7.0.x
             8.0.x
+            9.0.x
 
       - name: Restore
         run: dotnet restore
@@ -91,7 +92,9 @@ jobs:
       - name: Setup .NET
         uses: actions/setup-dotnet@v5
         with:
-          dotnet-version: 8.0.x
+          dotnet-version: |
+            8.0.x
+            9.0.x
 
       - name: Restore
         run: dotnet restore

Directory.Build.props

@@ -1,6 +1,6 @@
 <Project>
   <PropertyGroup>
-    <TargetFrameworks>net6.0;net7.0;net8.0</TargetFrameworks>
+    <TargetFrameworks>net6.0;net7.0;net8.0;net9.0</TargetFrameworks>
     <Nullable>enable</Nullable>
     <ImplicitUsings>enable</ImplicitUsings>
     <LangVersion>latest</LangVersion>

tests/GreptimeDB.Ingester.IntegrationTests/GreptimeClientTests.cs

@@ -26,13 +26,13 @@ public GreptimeClientTests()
         };
     }
 
-    public Task InitializeAsync()
+    public ValueTask InitializeAsync()
     {
         _client = new GreptimeClient(_options);
-        return Task.CompletedTask;
+        return ValueTask.CompletedTask;
     }
 
-    public async Task DisposeAsync()
+    public async ValueTask DisposeAsync()
     {
         if (_client != null)
         {

tests/GreptimeDB.Ingester.IntegrationTests/GreptimeDB.Ingester.IntegrationTests.csproj

@@ -1,11 +1,11 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
+    <TargetFrameworks>net8.0;net9.0</TargetFrameworks>
     <IsPackable>false</IsPackable>
     <IsTestProject>true</IsTestProject>
-    <!-- Allow underscores in test method names -->
-    <NoWarn>$(NoWarn);CA1707</NoWarn>
+    <!-- Allow underscores in test method names; xUnit1051: CancellationToken suggestion -->
+    <NoWarn>$(NoWarn);CA1707;xUnit1051</NoWarn>
   </PropertyGroup>
 
   <ItemGroup>

tests/GreptimeDB.Ingester.Tests/GreptimeDB.Ingester.Tests.csproj

@@ -1,11 +1,11 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
+    <TargetFrameworks>net8.0;net9.0</TargetFrameworks>
     <IsPackable>false</IsPackable>
     <IsTestProject>true</IsTestProject>
-    <!-- Allow underscores in test method names -->
-    <NoWarn>$(NoWarn);CA1707</NoWarn>
+    <!-- Allow underscores in test method names; xUnit1051: CancellationToken suggestion -->
+    <NoWarn>$(NoWarn);CA1707;xUnit1051</NoWarn>
   </PropertyGroup>
 
   <ItemGroup>

tests/GreptimeDB.Ingester.Tests/NameSanitizerTests.cs

@@ -62,13 +62,19 @@ public void Sanitize_EmptyOrNull_ThrowsArgumentException(string? input)
     [Theory]
     [InlineData("table123", "table123")]
     [InlineData("Table123Name", "table123_name")]
-    [InlineData("123table", "123table")]
     public void Sanitize_WithNumbers_PreservesNumbers(string input, string expected)
     {
         var result = NameSanitizer.Sanitize(input);
         result.Should().Be(expected);
     }
 
+    [Fact]
+    public void Sanitize_StartsWithNumber_ThrowsArgumentException()
+    {
+        var act = () => NameSanitizer.Sanitize("123table");
+        act.Should().Throw<ArgumentException>();
+    }
+
     [Fact]
     public void Sanitize_AlreadySnakeCase_NoChange()
     {
@@ -79,10 +85,16 @@ public void Sanitize_AlreadySnakeCase_NoChange()
     [Theory]
     [InlineData("a", "a")]
     [InlineData("A", "a")]
-    [InlineData("_", "_")]
     public void Sanitize_SingleCharacter_Succeeds(string input, string expected)
     {
         var result = NameSanitizer.Sanitize(input);
         result.Should().Be(expected);
     }
+
+    [Fact]
+    public void Sanitize_OnlyUnderscore_ThrowsArgumentException()
+    {
+        var act = () => NameSanitizer.Sanitize("_");
+        act.Should().Throw<ArgumentException>();
+    }
 }