From f43b9d11caf44396aa7e9bcdd5ed5401f86d2f35 Mon Sep 17 00:00:00 2001 From: Anush008 Date: Wed, 5 Mar 2025 13:54:51 +0530 Subject: [PATCH 1/5] docs: Demonstrate Qdrant version Signed-off-by: Anush008 --- packages/modules/qdrant/src/qdrant-container.test.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/modules/qdrant/src/qdrant-container.test.ts b/packages/modules/qdrant/src/qdrant-container.test.ts index 1af9220ae..2b149818a 100644 --- a/packages/modules/qdrant/src/qdrant-container.test.ts +++ b/packages/modules/qdrant/src/qdrant-container.test.ts @@ -8,7 +8,7 @@ describe("QdrantContainer", () => { // connectQdrantSimple { it("should connect to the client", async () => { - const container = await new QdrantContainer().start(); + const container = await new QdrantContainer("qdrant/qdrant:latest").start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}` }); @@ -22,7 +22,7 @@ describe("QdrantContainer", () => { it("should work with valid API keys", async () => { const apiKey = crypto.randomUUID(); - const container = await new QdrantContainer().withApiKey(apiKey).start(); + const container = await new QdrantContainer("qdrant/qdrant:latest").withApiKey(apiKey).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, apiKey }); @@ -35,7 +35,7 @@ describe("QdrantContainer", () => { it("should fail for invalid API keys", async () => { const apiKey = crypto.randomUUID(); - const container = await new QdrantContainer().withApiKey(apiKey).start(); + const container = await new QdrantContainer("qdrant/qdrant:latest").withApiKey(apiKey).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, @@ -49,7 +49,7 @@ describe("QdrantContainer", () => { // connectQdrantWithConfig { it("should work with config files - valid API key", async () => { - const container = await new QdrantContainer().withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); + const container = await new QdrantContainer("qdrant/qdrant:latest").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, apiKey: "SOME_TEST_KEY" }); @@ -60,7 +60,7 @@ describe("QdrantContainer", () => { // } it("should work with config files - invalid API key", async () => { - const container = await new QdrantContainer().withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); + const container = await new QdrantContainer("qdrant/qdrant:latest").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, From 4131fe4f86fe7391dcb863f2774584f96880605b Mon Sep 17 00:00:00 2001 From: Anush008 Date: Wed, 5 Mar 2025 13:57:35 +0530 Subject: [PATCH 2/5] chore: Bump Qdrant version Signed-off-by: Anush008 --- packages/modules/qdrant/src/qdrant-container.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/modules/qdrant/src/qdrant-container.ts b/packages/modules/qdrant/src/qdrant-container.ts index d4f32a293..df13a37ba 100644 --- a/packages/modules/qdrant/src/qdrant-container.ts +++ b/packages/modules/qdrant/src/qdrant-container.ts @@ -8,7 +8,7 @@ export class QdrantContainer extends GenericContainer { private apiKey: string | undefined; private configFilePath: string | undefined; - constructor(image = "qdrant/qdrant:v1.8.1") { + constructor(image = "qdrant/qdrant:v1.13.4") { super(image); this.withExposedPorts(QDRANT_REST_PORT, QDRANT_GRPC_PORT); this.withWaitStrategy( From f13cc864d62a2f497032f2e4576e99d818ff4144 Mon Sep 17 00:00:00 2001 From: Anush Date: Wed, 5 Mar 2025 14:51:06 +0530 Subject: [PATCH 3/5] chore: Pin to qdrant/qdrant:v1.13.4 --- packages/modules/qdrant/src/qdrant-container.test.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/modules/qdrant/src/qdrant-container.test.ts b/packages/modules/qdrant/src/qdrant-container.test.ts index 2b149818a..f085a0b53 100644 --- a/packages/modules/qdrant/src/qdrant-container.test.ts +++ b/packages/modules/qdrant/src/qdrant-container.test.ts @@ -8,7 +8,7 @@ describe("QdrantContainer", () => { // connectQdrantSimple { it("should connect to the client", async () => { - const container = await new QdrantContainer("qdrant/qdrant:latest").start(); + const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}` }); @@ -22,7 +22,7 @@ describe("QdrantContainer", () => { it("should work with valid API keys", async () => { const apiKey = crypto.randomUUID(); - const container = await new QdrantContainer("qdrant/qdrant:latest").withApiKey(apiKey).start(); + const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withApiKey(apiKey).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, apiKey }); @@ -35,7 +35,7 @@ describe("QdrantContainer", () => { it("should fail for invalid API keys", async () => { const apiKey = crypto.randomUUID(); - const container = await new QdrantContainer("qdrant/qdrant:latest").withApiKey(apiKey).start(); + const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withApiKey(apiKey).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, @@ -49,7 +49,7 @@ describe("QdrantContainer", () => { // connectQdrantWithConfig { it("should work with config files - valid API key", async () => { - const container = await new QdrantContainer("qdrant/qdrant:latest").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); + const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, apiKey: "SOME_TEST_KEY" }); @@ -60,7 +60,7 @@ describe("QdrantContainer", () => { // } it("should work with config files - invalid API key", async () => { - const container = await new QdrantContainer("qdrant/qdrant:latest").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); + const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, From 58229123fb8e216fd1f06af05538a2c9c2349d6c Mon Sep 17 00:00:00 2001 From: Cristian Greco Date: Wed, 5 Mar 2025 09:50:09 +0000 Subject: [PATCH 4/5] Fix lint --- packages/modules/qdrant/src/qdrant-container.test.ts | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packages/modules/qdrant/src/qdrant-container.test.ts b/packages/modules/qdrant/src/qdrant-container.test.ts index f085a0b53..b2e88c692 100644 --- a/packages/modules/qdrant/src/qdrant-container.test.ts +++ b/packages/modules/qdrant/src/qdrant-container.test.ts @@ -49,7 +49,9 @@ describe("QdrantContainer", () => { // connectQdrantWithConfig { it("should work with config files - valid API key", async () => { - const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); + const container = await new QdrantContainer("qdrant/qdrant:v1.13.4") + .withConfigFile(path.resolve(__dirname, "test_config.yaml")) + .start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, apiKey: "SOME_TEST_KEY" }); @@ -60,7 +62,9 @@ describe("QdrantContainer", () => { // } it("should work with config files - invalid API key", async () => { - const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start(); + const container = await new QdrantContainer("qdrant/qdrant:v1.13.4") + .withConfigFile(path.resolve(__dirname, "test_config.yaml")) + .start(); const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, From d5197b5eddd6d3e36b1ff296a898811c7cdd5f18 Mon Sep 17 00:00:00 2001 From: Anush Date: Wed, 5 Mar 2025 19:02:26 +0530 Subject: [PATCH 5/5] fix: Status message qdrant-container.test.ts --- packages/modules/qdrant/src/qdrant-container.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/modules/qdrant/src/qdrant-container.test.ts b/packages/modules/qdrant/src/qdrant-container.test.ts index b2e88c692..4f6257653 100644 --- a/packages/modules/qdrant/src/qdrant-container.test.ts +++ b/packages/modules/qdrant/src/qdrant-container.test.ts @@ -42,7 +42,7 @@ describe("QdrantContainer", () => { apiKey: "INVALID_KEY_" + crypto.randomUUID(), }); - expect(client.getCollections()).rejects.toThrow("Forbidden"); + expect(client.getCollections()).rejects.toThrow("Unauthorized"); await container.stop(); }); @@ -71,7 +71,7 @@ describe("QdrantContainer", () => { apiKey: "INVALID_KEY_" + crypto.randomUUID(), }); - expect(client.getCollections()).rejects.toThrow("Forbidden"); + expect(client.getCollections()).rejects.toThrow("Unauthorized"); await container.stop(); });