Fix reconfigure.replication and add unit test for its usage

Author: tsmith023

src/collections/configure/index.ts

@@ -3,6 +3,7 @@ import {
   InvertedIndexConfigUpdate,
   MultiTenancyConfigCreate,
   ReplicationConfigCreate,
+  ReplicationConfigUpdate,
   ReplicationDeletionStrategy,
   ShardingConfigCreate,
   VectorConfigUpdate,
@@ -234,7 +235,7 @@ const reconfigure = {
     asyncEnabled?: boolean;
     deletionStrategy?: ReplicationDeletionStrategy;
     factor?: number;
-  }): ReplicationConfigCreate => {
+  }): ReplicationConfigUpdate => {
     return {
       asyncEnabled: options.asyncEnabled,
       deletionStrategy: options.deletionStrategy,

src/collections/configure/types/base.ts

@@ -140,7 +140,7 @@ export type ReplicationConfigCreate = RecursivePartial<ReplicationConfig>;
 
 export type ReplicationConfigUpdate = {
   asyncEnabled?: boolean;
-  deletionStragey?: ReplicationDeletionStrategy;
+  deletionStrategy?: ReplicationDeletionStrategy;
   factor?: number;
 };
 

src/collections/configure/unit.test.ts

@@ -14,17 +14,18 @@ import {
   ModuleConfig,
   VectorConfigCreate,
 } from '../types/index.js';
-import { configure } from './index.js';
+import { configure, reconfigure } from './index.js';
 import {
   InvertedIndexConfigCreate,
   MultiTenancyConfigCreate,
   ReplicationConfigCreate,
+  ReplicationConfigUpdate,
   ShardingConfigCreate,
   VectorIndexConfigFlatCreate,
   VectorIndexConfigHNSWCreate,
 } from './types/index.js';
 
-describe('Unit testing of the configure factory class', () => {
+describe('Unit testing of the configure & reconfigure factory classes', () => {
   it('should create the correct InvertedIndexConfig type with all values', () => {
     const config = configure.invertedIndex({
       bm25b: 0.5,
@@ -76,7 +77,7 @@ describe('Unit testing of the configure factory class', () => {
     });
   });
 
-  it('should create the correct ReplicationConfig type with all values', () => {
+  it('should create the correct ReplicationConfigCreate type with all values', () => {
     const config = configure.replication({
       asyncEnabled: true,
       deletionStrategy: 'DeleteOnConflict',
@@ -89,6 +90,19 @@ describe('Unit testing of the configure factory class', () => {
     });
   });
 
+  it('should create the correct ReplicationConfigUpdate type with all values', () => {
+    const config = reconfigure.replication({
+      asyncEnabled: true,
+      deletionStrategy: 'DeleteOnConflict',
+      factor: 2,
+    });
+    expect(config).toEqual<ReplicationConfigUpdate>({
+      asyncEnabled: true,
+      deletionStrategy: 'DeleteOnConflict',
+      factor: 2,
+    });
+  });
+
   it('should create the correct ShardingConfig type with all values', () => {
     const config = configure.sharding({
       virtualPerPhysical: 256,