test: update expectations

Author: bevzzz

src/collections/configure/unit.test.ts

@@ -20,7 +20,7 @@ import {
   RerankerVoyageAIConfig,
   VectorConfigCreate,
 } from '../types/index.js';
-import { configure, vectorizer } from './index.js';
+import { configure } from './index.js';
 import {
   InvertedIndexConfigCreate,
   MultiTenancyConfigCreate,
@@ -1007,7 +1007,7 @@ describe('Unit testing of the vectorizer factory class', () => {
       },
       vectorizer: {
         name: 'text2vec-contextionary',
-        config: {},
+        config: undefined,
       },
     });
   });
@@ -1069,8 +1069,8 @@ describe('Unit testing of the vectorizer factory class', () => {
     });
   });
 
-  it.only('should create the correct Text2VecGPT4AllConfig type with all values', () => {
-    const config = vectorizer.text2VecGPT4All({
+  it('should create the correct Text2VecGPT4AllConfig type with all values', () => {
+    const config = configure.vectors.text2VecGPT4All({
       name: 'test',
     });
     expect(config).toEqual<VectorConfigCreate<never, 'test', 'hnsw', 'text2vec-gpt4all'>>({
@@ -1081,7 +1081,7 @@ describe('Unit testing of the vectorizer factory class', () => {
       },
       vectorizer: {
         name: 'text2vec-gpt4all',
-        config: {},
+        config: undefined,
       },
     });
   });

src/collections/configure/vectorizer.ts

@@ -37,20 +37,20 @@ const makeVectorIndex = (opts?: {
     }
     conf = conf
       ? {
-        ...conf,
-        multiVector: conf.multiVector
-          ? {
-            ...conf.multiVector,
-            encoding: conf.multiVector.encoding
-              ? { ...conf.multiVector.encoding, ...opts.encoding }
-              : opts.encoding,
-          }
-          : vectorIndex.multiVector.multiVector({ encoding: opts.encoding }),
-      }
+          ...conf,
+          multiVector: conf.multiVector
+            ? {
+                ...conf.multiVector,
+                encoding: conf.multiVector.encoding
+                  ? { ...conf.multiVector.encoding, ...opts.encoding }
+                  : opts.encoding,
+              }
+            : vectorIndex.multiVector.multiVector({ encoding: opts.encoding }),
+        }
       : {
-        multiVector: vectorIndex.multiVector.multiVector({ encoding: opts.encoding }),
-        type: 'hnsw',
-      };
+          multiVector: vectorIndex.multiVector.multiVector({ encoding: opts.encoding }),
+          type: 'hnsw',
+        };
   }
   if (opts?.quantizer) {
     if (!conf) {
@@ -117,6 +117,8 @@ const formatMulti2VecFields = (
 /** Previously all text-based vectorizers accepted `vectorizeCollectionName` parameter, which was meaningless for some modules and caused others to produce confusing results (see details below). Moving forward, we want to deprecate the usage of this parameter.
  *
  * Collections with `vectorizeCollectionName: true` generate embeddings even if they have no vectorizeable properties. This means all generated embeddings would embed the collection name itself, which makes them rather meaningless.
+ *
+ * @deprecated Use `vectors` instead.
  * */
 export const legacyVectors = {
   /**
@@ -188,16 +190,16 @@ export const legacyVectors = {
           Object.keys(config).length === 0
             ? undefined
             : {
-              ...config,
-              audioFields: audioFields?.map((f) => f.name),
-              depthFields: depthFields?.map((f) => f.name),
-              imageFields: imageFields?.map((f) => f.name),
-              IMUFields: IMUFields?.map((f) => f.name),
-              textFields: textFields?.map((f) => f.name),
-              thermalFields: thermalFields?.map((f) => f.name),
-              videoFields: videoFields?.map((f) => f.name),
-              weights: Object.keys(weights).length === 0 ? undefined : weights,
-            },
+                ...config,
+                audioFields: audioFields?.map((f) => f.name),
+                depthFields: depthFields?.map((f) => f.name),
+                imageFields: imageFields?.map((f) => f.name),
+                IMUFields: IMUFields?.map((f) => f.name),
+                textFields: textFields?.map((f) => f.name),
+                thermalFields: thermalFields?.map((f) => f.name),
+                videoFields: videoFields?.map((f) => f.name),
+                weights: Object.keys(weights).length === 0 ? undefined : weights,
+              },
       },
     });
   },
@@ -227,11 +229,11 @@ export const legacyVectors = {
           Object.keys(config).length === 0
             ? undefined
             : {
-              ...config,
-              imageFields: imageFields?.map((f) => f.name),
-              textFields: textFields?.map((f) => f.name),
-              weights: Object.keys(weights).length === 0 ? undefined : weights,
-            },
+                ...config,
+                imageFields: imageFields?.map((f) => f.name),
+                textFields: textFields?.map((f) => f.name),
+                weights: Object.keys(weights).length === 0 ? undefined : weights,
+              },
       },
     });
   },
@@ -261,11 +263,11 @@ export const legacyVectors = {
           Object.keys(config).length === 0
             ? undefined
             : {
-              ...config,
-              imageFields: imageFields?.map((f) => f.name),
-              textFields: textFields?.map((f) => f.name),
-              weights: Object.keys(weights).length === 0 ? undefined : weights,
-            },
+                ...config,
+                imageFields: imageFields?.map((f) => f.name),
+                textFields: textFields?.map((f) => f.name),
+                weights: Object.keys(weights).length === 0 ? undefined : weights,
+              },
       },
     });
   },
@@ -296,11 +298,11 @@ export const legacyVectors = {
           Object.keys(config).length === 0
             ? undefined
             : {
-              ...config,
-              imageFields: imageFields?.map((f) => f.name),
-              textFields: textFields?.map((f) => f.name),
-              weights: Object.keys(weights).length === 0 ? undefined : weights,
-            },
+                ...config,
+                imageFields: imageFields?.map((f) => f.name),
+                textFields: textFields?.map((f) => f.name),
+                weights: Object.keys(weights).length === 0 ? undefined : weights,
+              },
       },
     });
   },
@@ -400,11 +402,11 @@ export const legacyVectors = {
           Object.keys(config).length === 0
             ? undefined
             : {
-              ...config,
-              imageFields: imageFields?.map((f) => f.name),
-              textFields: textFields?.map((f) => f.name),
-              weights: Object.keys(weights).length === 0 ? undefined : weights,
-            },
+                ...config,
+                imageFields: imageFields?.map((f) => f.name),
+                textFields: textFields?.map((f) => f.name),
+                weights: Object.keys(weights).length === 0 ? undefined : weights,
+              },
       },
     });
   },

src/collections/filters/integration.test.ts

@@ -57,8 +57,7 @@ describe('Testing of the filter class with a simple collection', () => {
           },
         ],
         invertedIndex: weaviate.configure.invertedIndex({ indexTimestamps: true }),
-        vectorizers: weaviate.configure.vectors.text2VecContextionary({
-        }),
+        vectorizers: weaviate.configure.vectors.text2VecContextionary({}),
       })
       .then(() =>
         collection.data.insertMany([