Skip to content

Commit 7c6666c

Browse files
authored
Remove excessive tests from vector index simplification
1 parent d5db6d7 commit 7c6666c

2 files changed

Lines changed: 1 addition & 20 deletions

File tree

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import {addVectorIndex, disableTTLTemplate, enableTTLTemplate} from '../schemaQueryTemplates';
1+
import {disableTTLTemplate, enableTTLTemplate} from '../schemaQueryTemplates';
22

33
describe('schemaQueryTemplates', () => {
44
describe('enableTTLTemplate', () => {
@@ -20,18 +20,4 @@ describe('schemaQueryTemplates', () => {
2020
expect(template).toContain('ALTER TABLE `table` RESET (TTL);');
2121
});
2222
});
23-
24-
describe('addVectorIndex', () => {
25-
test('only includes distance in vector index settings', () => {
26-
const template = addVectorIndex({path: '/local/table', relativePath: 'table'});
27-
28-
expect(template).toContain('GLOBAL USING vector_kmeans_tree');
29-
expect(template).toContain('WITH (\n distance=cosine\n);');
30-
expect(template).not.toContain('vector_type');
31-
expect(template).not.toContain('vector_dimension');
32-
expect(template).not.toContain('levels');
33-
expect(template).not.toContain('clusters');
34-
expect(template).not.toContain('overlap_clusters');
35-
});
36-
});
3723
});

‎tests/suites/tenant/queryEditor/queryTemplates.test.ts‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -246,11 +246,6 @@ test.describe('Query Templates', () => {
246246

247247
const editorContent = await queryEditor.getEditorContent();
248248
expect(editorContent).toContain('distance=cosine');
249-
expect(editorContent).not.toContain('vector_type');
250-
expect(editorContent).not.toContain('vector_dimension');
251-
expect(editorContent).not.toContain('levels');
252-
expect(editorContent).not.toContain('clusters');
253-
expect(editorContent).not.toContain('overlap_clusters');
254249
});
255250

256251
test('Add fulltext index template contains fulltext_relevance and use_filter_snowball', async ({

0 commit comments

Comments
 (0)