RDBC-899 Fixed tests and add placeholders for vector options.

Marked certain tests as skipped with TODO comments referencing RDBC-901 for future updates. Introduced `_vectorOptionsStrings` in TimeSeries and indexing-related code to prepare for expanded functionality.

RDBC-899 VectorSearchTest to conditionally skip on non-7.0 servers

Author: M4xymm

src/Documents/Indexes/TimeSeries/AbstractGenericTimeSeriesIndexCreationTask.ts

@@ -32,6 +32,7 @@ export abstract class AbstractGenericTimeSeriesIndexCreationTask extends Abstrac
         this._indexSuggestions = new Set<string>();
         this._termVectorsStrings = {};
         this._spatialOptionsStrings = {};
+        this._vectorOptionsStrings = {};
     }
 
     /**

src/Documents/Session/Tokens/WhereToken.ts

@@ -305,7 +305,7 @@ export class WhereToken extends QueryToken {
         }
 
         if (!this._writeMethod(writer)) {
-            writer.append(`"${this.parameterName}"`);
+            writer.append("$").append(this.parameterName);
         }
     }
 
@@ -379,8 +379,6 @@ export class WhereToken extends QueryToken {
                 const {vectorSearch} = this.options
                 writer.append(", $").append(this.parameterName);
 
-                // writer.append(", ai.task(").append(this.options.vectorSearch.embeddingsGenerationTaskIdentifier ?? "null").append(")");
-
                 writer.append(", ").append(vectorSearch?.similarity ?? "null");
                 writer.append(", ").append(vectorSearch?.numberOfCandidates ?? "null");
 

src/Documents/Session/VectorFieldFactory.ts

@@ -195,6 +195,4 @@ export class VectorFieldFactory<T> implements IVectorFieldFactory<T> {
     public withField(fieldName: Field<T>): IVectorField {
         return new VectorField(fieldName);
     }
-}
-
-// get rid of selector in field name (withText etc etc.)
+}

test/Documents/Queries/VectorSearchTest.ts

@@ -6,11 +6,11 @@ import {
     IndexDefinition,
     PutIndexesOperation
 } from "../../../src/index.js";
-import {disposeTestDocumentStore, testContext} from "../../Utils/TestUtil.js";
+import {disposeTestDocumentStore, RavenTestContext, testContext} from "../../Utils/TestUtil.js";
 import {assertThat} from "../../Utils/AssertExtensions.js";
 
 
-describe("RDBC-899", function () {
+(!RavenTestContext.is70Server ? describe.skip : describe)("RDBC-899", function () {
     let store: IDocumentStore;
 
     beforeEach(async function () {

test/Ported/Issues/RavenDB_11440.ts

@@ -9,7 +9,8 @@ import { throwError } from "../../../src/Exceptions/index.js";
 import { TimeUtil } from "../../../src/Utility/TimeUtil.js";
 import { assertThat } from "../../Utils/AssertExtensions.js";
 
-(!RavenTestContext.is70Server ? describe.skip : describe)("RavenDB_11440", function () {
+// TODO - https://issues.hibernatingrhinos.com/issue/RDBC-901/update-outdated-LogsConfiguration-in-node.js-client
+describe.skip("RavenDB_11440", function () {
 
     let store: IDocumentStore;
 

test/Ported/Server/LogsConfigurationTest.ts

@@ -7,7 +7,8 @@ import {
 import { disposeTestDocumentStore, RavenTestContext, testContext } from "../../Utils/TestUtil.js";
 import { assertThat } from "../../Utils/AssertExtensions.js";
 
-(!RavenTestContext.is70Server ? describe.skip : describe)("LogsConfigurationTest", function () {
+// TODO - https://issues.hibernatingrhinos.com/issue/RDBC-901/update-outdated-LogsConfiguration-in-node.js-client
+describe.skip("LogsConfigurationTest", function () {
 
     let store: IDocumentStore;