From 1e696d858587aa691f58fdea008743d28e9ee825 Mon Sep 17 00:00:00 2001 From: wujianlin Date: Sat, 10 May 2025 11:45:53 +0800 Subject: [PATCH] The c_utils test case alarm is cleared Signed-off-by: wujianlin --- .../safe_block_queue_benchmark_test.cpp | 4 +-- ...fe_block_queue_tracking_benchmark_test.cpp | 36 +++++++++---------- .../safe_map_benchmark_test.cpp | 4 +-- .../refbase_fuzzer/refbase_fuzzer.cpp | 2 +- .../utils_safe_block_queue_tracking.cpp | 16 ++++----- .../unittest/common/utils_safe_map_test.cpp | 4 +-- 6 files changed, 33 insertions(+), 33 deletions(-) diff --git a/base/test/benchmarktest/safe_block_queue_benchmark_test/safe_block_queue_benchmark_test.cpp b/base/test/benchmarktest/safe_block_queue_benchmark_test/safe_block_queue_benchmark_test.cpp index 0570adc..9ee3ee0 100644 --- a/base/test/benchmarktest/safe_block_queue_benchmark_test/safe_block_queue_benchmark_test.cpp +++ b/base/test/benchmarktest/safe_block_queue_benchmark_test/safe_block_queue_benchmark_test.cpp @@ -137,10 +137,10 @@ auto GetTimeOfSleepTwentyMillisecond() return timeT; } -static void QueuePushFullEquivalent(const int Equivalent, benchmark::State& state) +static void QueuePushFullEquivalent(const int equivalent, benchmark::State& state) { for (unsigned int i = 0; i < QUEUE_SLOTS; i++) { - DemoThreadData::shareQueue.Push(Equivalent); + DemoThreadData::shareQueue.Push(equivalent); } AssertTrue((DemoThreadData::shareQueue.IsFull()), "shareQueue.IsFull() did not equal true.", state); } diff --git a/base/test/benchmarktest/safe_block_queue_tracking_benchmark_test/safe_block_queue_tracking_benchmark_test.cpp b/base/test/benchmarktest/safe_block_queue_tracking_benchmark_test/safe_block_queue_tracking_benchmark_test.cpp index 5fd9acf..f84a48a 100644 --- a/base/test/benchmarktest/safe_block_queue_tracking_benchmark_test/safe_block_queue_tracking_benchmark_test.cpp +++ b/base/test/benchmarktest/safe_block_queue_tracking_benchmark_test/safe_block_queue_tracking_benchmark_test.cpp @@ -157,7 +157,7 @@ void StartThreads(std::thread (&threads)[THREAD_NUM], } template -void joinAllThreads(std::thread (&threads)[N]) +void JoinAllThreads(std::thread (&threads)[N]) { for (auto& t : threads) { if (t.joinable()) { @@ -166,7 +166,7 @@ void joinAllThreads(std::thread (&threads)[N]) } } -void processSharedQueueTasks(DemoThreadData& data) +void ProcessSharedQueueTasks(DemoThreadData& data) { while (!DemoThreadData::shareQueue.IsEmpty()) { data.GetAndOneTaskDone(); @@ -267,10 +267,10 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, testMutilthreadPutAndBlock001)(benc AssertTrue((demoDatas[0].joinStatus), "step 7: demoDatas[0].joinStatus did not equal true.", state); // recover state - joinAllThreads(threads); + JoinAllThreads(threads); joinThread.join(); - processSharedQueueTasks(demoDatas[0]); + ProcessSharedQueueTasks(demoDatas[0]); demoDatas[0].joinStatus = false; } BENCHMARK_LOGD("SafeBlockQueueTracking testMutilthreadPutAndBlock001 end."); @@ -318,8 +318,8 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, PutAndBlockInblankqueue001)(benchma AssertEqual(pushedIn, THREAD_NUM, "pushedIn did not equal THREAD_NUM as expected.", state); AssertEqual(getedOut, THREAD_NUM - QUEUE_SLOTS, "getedOut did not equal THREAD_NUM - QUEUE_SLOTS as expected.", state); - joinAllThreads(threads); - processSharedQueueTasks(demoDatas[0]); + JoinAllThreads(threads); + ProcessSharedQueueTasks(demoDatas[0]); std::this_thread::sleep_for(std::chrono::milliseconds(SLEEP_FOR_HUNDRED_MILLISECOND)); AssertTrue((demoDatas[0].joinStatus), "step 3: demoDatas[0].joinStatus did not equal true.", state); demoDatas[0].joinStatus = false; @@ -371,8 +371,8 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, testPutAndBlockInFullQueue001)(benc AssertEqual(unpushedIn, THREAD_NUM - (i + 1), "unpushedIn did not equal THREAD_NUM - (i + 1) as expected.", state); } - joinAllThreads(threads); - processSharedQueueTasks(demoDatas[0]); + JoinAllThreads(threads); + ProcessSharedQueueTasks(demoDatas[0]); demoDatas[0].joinStatus = false; joinThread.join(); } @@ -418,8 +418,8 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, GetAndBlockInblankqueue001)(benchma AssertEqual(ungetedOut, THREAD_NUM - (i + 1), "ungetedOut did not equal THREAD_NUM - (i + 1) as expected.", state); } - joinAllThreads(threads); - processSharedQueueTasks(demoDatas[0]); + JoinAllThreads(threads); + ProcessSharedQueueTasks(demoDatas[0]); AssertTrue((demoDatas[0].joinStatus), "demoDatas[0].joinStatus did not equal true as expected.", state); demoDatas[0].joinStatus = false; joinThread.join(); @@ -427,10 +427,10 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, GetAndBlockInblankqueue001)(benchma BENCHMARK_LOGD("SafeBlockQueueTracking GetAndBlockInblankqueue001 end."); } -static void QueuePushFullEquivalent(const int Equivalent, benchmark::State& state) +static void QueuePushFullEquivalent(const int equivalent, benchmark::State& state) { for (unsigned int i = 0; i < QUEUE_SLOTS; i++) { - DemoThreadData::shareQueue.Push(Equivalent); + DemoThreadData::shareQueue.Push(equivalent); } AssertTrue((DemoThreadData::shareQueue.IsFull()), "shareQueue.IsFull() did not equal true.", state); } @@ -485,8 +485,8 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, GetAndBlockInfullqueue001)(benchmar AssertEqual(getedOut, THREAD_NUM, "getedOut did not equal THREAD_NUM as expected.", state); AssertEqual(ungetedOut, static_cast(0), "ungetedOut did not equal static_cast(0) as expected.", state); - joinAllThreads(threads); - processSharedQueueTasks(demoDatas[0]); + JoinAllThreads(threads); + ProcessSharedQueueTasks(demoDatas[0]); demoDatas[0].joinStatus = false; joinThread.join(); joinThread2.join(); @@ -536,7 +536,7 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, GetAndBlockInnotfullqueue001)(bench AssertEqual(getedOut, THREAD_NUM, "getedOut did not equal THREAD_NUM as expected.", state); AssertEqual(ungetedOut, static_cast(0), "ungetedOut did not equal static_cast(0) as expected.", state); - joinAllThreads(threads); + JoinAllThreads(threads); AssertTrue((demoDatas[0].joinStatus), "step 3: demoDatas[0].joinStatus did not equal true.", state); demoDatas[0].joinStatus = false; joinThread.join(); @@ -586,8 +586,8 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, PutAndBlockInnotfullqueue001)(bench AssertEqual(unputedin, static_cast(0), "unputedin did not equal static_cast(0) as expected.", state); AssertFalse((demoDatas[0].joinStatus), "step 4: demoDatas[0].joinStatus did not equal false.", state); - joinAllThreads(threads); - processSharedQueueTasks(demoDatas[0]); + JoinAllThreads(threads); + ProcessSharedQueueTasks(demoDatas[0]); std::this_thread::sleep_for(std::chrono::milliseconds(SLEEP_FOR_HUNDRED_MILLISECOND)); AssertTrue((demoDatas[0].joinStatus), "step 5: demoDatas[0].joinStatus did not equal true.", state); joinThread.join(); @@ -637,7 +637,7 @@ BENCHMARK_F(BenchmarkSafeBlockQueueTracking, testMutilthreadConcurrentGetAndPopI for (auto& inThread : threadsin) { inThread.join(); } - processSharedQueueTasks(demoDatas[0]); + ProcessSharedQueueTasks(demoDatas[0]); joinThread.join(); AssertTrue((demoDatas[0].joinStatus), "step 3: demoDatas[0].joinStatus did not equal true.", state); demoDatas[0].joinStatus = false; diff --git a/base/test/benchmarktest/safe_map_benchmark_test/safe_map_benchmark_test.cpp b/base/test/benchmarktest/safe_map_benchmark_test/safe_map_benchmark_test.cpp index fb6fcac..7bf2850 100644 --- a/base/test/benchmarktest/safe_map_benchmark_test/safe_map_benchmark_test.cpp +++ b/base/test/benchmarktest/safe_map_benchmark_test/safe_map_benchmark_test.cpp @@ -278,7 +278,7 @@ BENCHMARK_F(BenchmarkSafeMap, testUtilsNormalFeatureEraseAndClear001)(benchmark: * @tc.name: testUtilsNormalFeatureIterate001 * @tc.desc: Using Iterate to change the second parameter of SafeMap */ -void callback(const std::string str, int& value) +void Callback(const std::string str, int& value) { value++; } @@ -294,7 +294,7 @@ BENCHMARK_F(BenchmarkSafeMap, testUtilsNormalFeatureIterate001)(benchmark::State demoData.Insert("B", INSERT_TWO); demoData.Insert("C", INSERT_THREE); demoData.Insert("D", INSERT_FOUR); - demoData.Iterate(callback); + demoData.Iterate(Callback); AssertEqual(demoData.Size(), INSERT_FOUR, "demoData.Size() did not equal INSERT_FOUR as expected.", state); diff --git a/base/test/fuzztest/refbase_fuzzer/refbase_fuzzer.cpp b/base/test/fuzztest/refbase_fuzzer/refbase_fuzzer.cpp index 6157d59..52f48f0 100644 --- a/base/test/fuzztest/refbase_fuzzer/refbase_fuzzer.cpp +++ b/base/test/fuzztest/refbase_fuzzer/refbase_fuzzer.cpp @@ -354,7 +354,7 @@ public: void OnLastWeakRef(const void *objectIda) override { - FUZZ_LOGI("OnLastWeakRef is called"); + FUZZ_LOGI("OnLastWeakRef is called"); } }; diff --git a/base/test/unittest/common/utils_safe_block_queue_tracking.cpp b/base/test/unittest/common/utils_safe_block_queue_tracking.cpp index 25103a0..09e5f49 100644 --- a/base/test/unittest/common/utils_safe_block_queue_tracking.cpp +++ b/base/test/unittest/common/utils_safe_block_queue_tracking.cpp @@ -418,10 +418,10 @@ HWTEST_F(UtilsSafeBlockQueueTracking, testMutilthreadConcurrentGetAndBlockInblan joinThread.join(); } -static void QueuePushFullEquivalent(const int Equivalent) +static void QueuePushFullEquivalent(const int equivalent) { for (unsigned int i = 0; i < QUEUE_SLOTS; i++) { - DemoThreadData::shareQueue.Push(Equivalent); + DemoThreadData::shareQueue.Push(equivalent); } ASSERT_TRUE(DemoThreadData::shareQueue.IsFull()); } @@ -488,8 +488,8 @@ HWTEST_F(UtilsSafeBlockQueueTracking, testMutilthreadConcurrentGetAndBlockInfull ASSERT_EQ(getedOut, THREAD_NUM); ASSERT_EQ(ungetedOut, static_cast(0)); - for (auto& t : threads) { - t.join(); + for (auto& thd : threads) { + thd.join(); } while (!DemoThreadData::shareQueue.IsEmpty()) { @@ -680,12 +680,12 @@ HWTEST_F(UtilsSafeBlockQueueTracking, testMutilthreadConcurrentGetAndPopInfullqu ASSERT_FALSE(demoDatas[0].joinStatus); demoDatas[0].joinStatus = false; - for (auto& t : threadsout) { - t.join(); + for (auto& thdout : threadsout) { + thdout.join(); } - for (auto& t : threadsin) { - t.join(); + for (auto& thdin : threadsin) { + thdin.join(); } while (!DemoThreadData::shareQueue.IsEmpty()) { diff --git a/base/test/unittest/common/utils_safe_map_test.cpp b/base/test/unittest/common/utils_safe_map_test.cpp index 9c78dbf..d628016 100644 --- a/base/test/unittest/common/utils_safe_map_test.cpp +++ b/base/test/unittest/common/utils_safe_map_test.cpp @@ -199,7 +199,7 @@ HWTEST_F(UtilsSafeMap, testUtilsNormalFeatureEraseAndClear001, TestSize.Level0) * @tc.name: testUtilsNormalFeatureIterate001 * @tc.desc: Using Iterate to change the second parameter of SafeMap */ -void callback(const std::string str, int& value) +void Callback(const std::string str, int& value) { value++; } @@ -213,7 +213,7 @@ HWTEST_F(UtilsSafeMap, testUtilsNormalFeatureIterate001, TestSize.Level0) demoData.Insert("B", 2); demoData.Insert("C", 3); demoData.Insert("D", 4); - demoData.Iterate(callback); + demoData.Iterate(Callback); ASSERT_EQ(demoData.Size(), 4); ASSERT_EQ(demoData.ReadVal("A"), 2); -- Gitee