MB-61170: Release all allocated resources in memcached_statistics_tests

Initialize the required services before running the tests and shut
them down before terminating.

Change-Id: If76f400455d817924f0e1f54d12b5281801446d8
Reviewed-on: https://review.couchbase.org/c/kv_engine/+/207214
Tested-by: Build Bot <[email protected]>
Reviewed-by: Vesko Karaganev <[email protected]>

Author: trondn

Diff for: tests/statistics/CMakeLists.txt

@@ -1,12 +1,12 @@
 cb_add_test_executable(memcached_statistics_test
-        prometheus_test.cc
-        collector_test.cc)
+        collector_test.cc
+        main.cc
+        prometheus_test.cc)
 kv_enable_pch(memcached_statistics_test)
 cb_enable_unity_build(memcached_statistics_test)
 target_link_libraries(memcached_statistics_test
         PRIVATE
                 GTest::gmock
-                GTest::gtest_main
                 memcached_daemon
                 memcached_logger
                 statistics)

Diff for: tests/statistics/main.cc

@@ -0,0 +1,32 @@
+/*
+ *     Copyright 2024-Present Couchbase, Inc.
+ *
+ *   Use of this software is governed by the Business Source License included
+ *   in the file licenses/BSL-Couchbase.txt.  As of the Change Date specified
+ *   in that file, in accordance with the Business Source License, use of this
+ *   software will be governed by the Apache License, Version 2.0, included in
+ *   the file licenses/APL2.txt.
+ */
+
+#include <daemon/external_auth_manager_thread.h>
+#include <daemon/mcaudit.h>
+#include <folly/portability/GTest.h>
+#include <logger/logger.h>
+
+int main(int argc, char** argv) {
+    ::testing::InitGoogleTest(&argc, argv);
+
+    cb::logger::createBlackholeLogger();
+
+    externalAuthManager = std::make_unique<ExternalAuthManagerThread>();
+    initialize_audit();
+
+    int ret = RUN_ALL_TESTS();
+
+    externalAuthManager->shutdown();
+    externalAuthManager->waitForState(Couchbase::ThreadState::Zombie);
+    externalAuthManager.reset();
+    shutdown_audit();
+    cb::logger::shutdown();
+    return ret;
+}

Diff for: tests/statistics/prometheus_test.cc

@@ -33,12 +33,6 @@
 #include <chrono>
 #include <vector>
 
-void PrometheusStatTest::SetUpTestSuite() {
-    if (!externalAuthManager) {
-        externalAuthManager = std::make_unique<ExternalAuthManagerThread>();
-    }
-}
-
 PrometheusStatTest::EndpointMetrics PrometheusStatTest::getMetrics() const {
     EndpointMetrics metrics;
 
@@ -53,7 +47,6 @@ PrometheusStatTest::EndpointMetrics PrometheusStatTest::getMetrics() const {
 TEST_F(PrometheusStatTest, auditStatsNotPerBucket) {
     // confirm audit stats are not labelled with a bucket - they are actually
     // global stats.
-    initialize_audit();
     auto metrics = getMetrics();
     using namespace cb::stats;
     using namespace ::testing;

Diff for: tests/statistics/prometheus_test.h

@@ -32,7 +32,5 @@ class PrometheusStatTest : public CollectorTest {
         StatMap metering;
     };
 
-    static void SetUpTestSuite();
-
     EndpointMetrics getMetrics() const;
 };