Avoid conflict of mock class name to fix test failure (opensearch-pro…

…ject#1819)

Rename MockIndex to FaissMockIndex in faiss_wrapper_unit_test.cpp to avoid interference during unit tests. Also, rename MockIDMap to FaissMockIdMap for consistency. The MockIndex class is defined in tests/mocks/faiss_index_mock.h

Signed-off-by: Heemin Kim <heemin@amazon.com>

jni/tests/faiss_wrapper_unit_test.cpp

@@ -25,12 +25,12 @@ using ::testing::NiceMock;
 
 using idx_t = faiss::idx_t;
 
-struct MockIndex : faiss::IndexHNSW {
-    explicit MockIndex(idx_t d) : faiss::IndexHNSW(d, 32) {
+struct FaissMockIndex : faiss::IndexHNSW {
+    explicit FaissMockIndex(idx_t d) : faiss::IndexHNSW(d, 32) {
     }
 };
 
-struct MockIdMap : faiss::IndexIDMap {
+struct FaissMockIdMap : faiss::IndexIDMap {
     mutable idx_t nCalled{};
     mutable const float *xCalled{};
     mutable int kCalled{};
@@ -40,7 +40,7 @@ struct MockIdMap : faiss::IndexIDMap {
     mutable const faiss::SearchParametersHNSW *paramsCalled{};
     mutable faiss::RangeSearchResult *resCalled{};
 
-    explicit MockIdMap(MockIndex *index) : faiss::IndexIDMapTemplate<faiss::Index>(index) {
+    explicit FaissMockIdMap(FaissMockIndex *index) : faiss::IndexIDMapTemplate<faiss::Index>(index) {
     }
 
     void search(
@@ -108,8 +108,8 @@ class FaissWrappeterParametrizedTestFixture : public testing::TestWithParam<Quer
     }
 
 protected:
-    MockIndex index_;
-    MockIdMap id_map_;
+    FaissMockIndex index_;
+    FaissMockIdMap id_map_;
 };
 
 class FaissWrapperParametrizedRangeSearchTestFixture : public testing::TestWithParam<RangeSearchTestInput> {
@@ -119,8 +119,8 @@ class FaissWrapperParametrizedRangeSearchTestFixture : public testing::TestWithP
     }
 
 protected:
-    MockIndex index_;
-    MockIdMap id_map_;
+    FaissMockIndex index_;
+    FaissMockIdMap id_map_;
 };
 
 namespace query_index_test {
@@ -369,4 +369,3 @@ namespace range_search_test {
         )
     );
 }
-