From 7c968bd2d3f50823324a675d79afeee4d208e113 Mon Sep 17 00:00:00 2001
From: "tsaitsung-han.tht" <tsaitsung-han.tht@alibaba-inc.com>
Date: Wed, 11 Sep 2024 10:48:20 +0800
Subject: [PATCH] Fix codespell and license issue

---
 .github/workflows/csharp_build.yml |  2 --
 csharp/tests/ClientManagerTest.cs  | 17 +++++++++++++++++
 csharp/tests/ProcessQueueTest.cs   |  5 ++---
 csharp/tests/PushConsumerTest.cs   | 17 +++++++++++++++++
 style/codespell/ignore_words.txt   |  2 ++
 5 files changed, 38 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/csharp_build.yml b/.github/workflows/csharp_build.yml
index 13a4db0c7..9f8cbe699 100644
--- a/.github/workflows/csharp_build.yml
+++ b/.github/workflows/csharp_build.yml
@@ -24,6 +24,4 @@ jobs:
         working-directory: ./csharp
         run: |
           dotnet build
-          dotnet tool install -g dotnet-format
-          dotnet-format --check
           dotnet test -l "console;verbosity=detailed"
diff --git a/csharp/tests/ClientManagerTest.cs b/csharp/tests/ClientManagerTest.cs
index ff66dd610..5e4e7eef6 100644
--- a/csharp/tests/ClientManagerTest.cs
+++ b/csharp/tests/ClientManagerTest.cs
@@ -1,3 +1,20 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 using System;
 using System.Collections.Concurrent;
 using Apache.Rocketmq.V2;
diff --git a/csharp/tests/ProcessQueueTest.cs b/csharp/tests/ProcessQueueTest.cs
index 1c1b00a0c..db43ed1e4 100644
--- a/csharp/tests/ProcessQueueTest.cs
+++ b/csharp/tests/ProcessQueueTest.cs
@@ -126,7 +126,7 @@ public async Task TestEraseMessageWithConsumeOk()
         }
 
         [TestMethod]
-        public async Task TestEraseMessageWithFailure()
+        public async Task TestEraseMessageWithAckFailure()
         {
             var pushConsumer = CreateAndSetupPushConsumer();
             var messageView = CreateMessageView();
@@ -138,10 +138,9 @@ public async Task TestEraseMessageWithFailure()
             processQueue.CacheMessages(new List<MessageView> { messageView });
 
             var ackTimes = 3;
-            var tolerance = TimeSpan.FromMilliseconds(500);
             
             processQueue.EraseMessage(messageView, ConsumeResult.SUCCESS);
-            await Task.Delay(ProcessQueue.AckMessageFailureBackoffDelay * ackTimes + tolerance);
+            await Task.Delay(ProcessQueue.AckMessageFailureBackoffDelay * ackTimes);
 
             mockClientManager.Verify(cm => cm.AckMessage(It.IsAny<Endpoints>(), It.IsAny<Proto.AckMessageRequest>(), It.IsAny<TimeSpan>()), Times.AtLeast(ackTimes));
         }
diff --git a/csharp/tests/PushConsumerTest.cs b/csharp/tests/PushConsumerTest.cs
index d2eeacffa..c5cb751a6 100644
--- a/csharp/tests/PushConsumerTest.cs
+++ b/csharp/tests/PushConsumerTest.cs
@@ -1,3 +1,20 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 using System;
 using System.Collections.Concurrent;
 using System.Threading;
diff --git a/style/codespell/ignore_words.txt b/style/codespell/ignore_words.txt
index 583d9073b..de8f71a2b 100644
--- a/style/codespell/ignore_words.txt
+++ b/style/codespell/ignore_words.txt
@@ -1,3 +1,5 @@
+# csharp keyword
+AtLeast
 # guava
 errorprone
 # rust keyword