Refactor bindings parallel flow tests (#326)

Author: alonre24

src/VecSim/algorithms/brute_force/brute_force.h

@@ -57,6 +57,18 @@ class BruteForceIndex : public VecSimIndexAbstract<DistType> {
 
     inline vecsim_stl::vector<VectorBlock *> getVectorBlocks() const { return vectorBlocks; }
     virtual ~BruteForceIndex();
+#ifdef BUILD_TESTS
+    /**
+     * @brief Used for testing - store vector(s) data associated with a given label. This function
+     * copies the vector(s)' data buffer(s) and place it in the output vector
+     *
+     * @param label
+     * @param vectors_output empty vector to be modified, should store the blob(s) associated with
+     * the label.
+     */
+    virtual void getDataByLabel(labelType label,
+                                std::vector<std::vector<DataType>> &vectors_output) const = 0;
+#endif
 
 protected:
     // Private internal function that implements generic single vector insertion.

src/VecSim/algorithms/brute_force/brute_force_multi.h

@@ -33,7 +33,8 @@ class BruteForceIndex_Multi : public BruteForceIndex<DataType, DistType> {
             new (this->allocator) vecsim_stl::unique_results_container(cap, this->allocator));
     }
 #ifdef BUILD_TESTS
-    void GetDataByLabel(labelType label, std::vector<std::vector<DataType>> &vectors_output) {
+    void getDataByLabel(labelType label,
+                        std::vector<std::vector<DataType>> &vectors_output) const override {
 
         auto ids = labelToIdsLookup.find(label);
 

src/VecSim/algorithms/brute_force/brute_force_single.h

@@ -32,7 +32,8 @@ class BruteForceIndex_Single : public BruteForceIndex<DataType, DistType> {
 
     inline size_t indexLabelCount() const override { return this->count; }
 #ifdef BUILD_TESTS
-    void GetDataByLabel(labelType label, std::vector<std::vector<DataType>> &vectors_output) {
+    void getDataByLabel(labelType label,
+                        std::vector<std::vector<DataType>> &vectors_output) const override {
 
         auto id = labelToIdLookup.at(label);
 

src/VecSim/algorithms/hnsw/hnsw.h

@@ -235,6 +235,19 @@ class HNSWIndex : public VecSimIndexAbstract<DistType>,
     // inline priority queue getter that need to be implemented by derived class
     virtual inline candidatesLabelsMaxHeap<DistType> *getNewMaxPriorityQueue() const = 0;
 
+#ifdef BUILD_TESTS
+    /**
+     * @brief Used for testing - store vector(s) data associated with a given label. This function
+     * copies the vector(s)' data buffer(s) and place it in the output vector
+     *
+     * @param label
+     * @param vectors_output empty vector to be modified, should store the blob(s) associated with
+     * the label.
+     */
+    virtual void getDataByLabel(labelType label,
+                                std::vector<std::vector<DataType>> &vectors_output) const = 0;
+#endif
+
 protected:
     // inline label to id setters that need to be implemented by derived class
     virtual inline std::unique_ptr<vecsim_stl::abstract_results_container>

src/VecSim/algorithms/hnsw/hnsw_multi.h

@@ -42,7 +42,8 @@ class HNSWIndex_Multi : public HNSWIndex<DataType, DistType> {
         : HNSWIndex<DataType, DistType>(input, params, allocator, version),
           label_lookup_(this->max_elements_, allocator) {}
 
-    void GetDataByLabel(labelType label, std::vector<std::vector<DataType>> &vectors_output) {
+    void getDataByLabel(labelType label,
+                        std::vector<std::vector<DataType>> &vectors_output) const override {
 
         auto ids = label_lookup_.find(label);
 

src/VecSim/algorithms/hnsw/hnsw_single.h

@@ -35,7 +35,8 @@ class HNSWIndex_Single : public HNSWIndex<DataType, DistType> {
         : HNSWIndex<DataType, DistType>(input, params, allocator, version),
           label_lookup_(this->max_elements_, allocator) {}
 
-    void GetDataByLabel(labelType label, std::vector<std::vector<DataType>> &vectors_output) {
+    void getDataByLabel(labelType label,
+                        std::vector<std::vector<DataType>> &vectors_output) const override {
 
         auto id = label_lookup_.at(label);