Skip to content

Commit aa04c92

Browse files
committed
test: use different port binding for backup/unit.test.ts
Seems like Node 18 does not automatically 'resolve' port collisions, so 2 test running Express servers on the same ports will interfere with one another.
1 parent 67b675e commit aa04c92

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/collections/backup/unit.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,8 @@ describe('Mock testing of backup cancellation', () => {
109109
let mock: CancelMock;
110110

111111
beforeAll(async () => {
112-
mock = await CancelMock.use('1.27.0', 8958, 8959);
113-
client = await weaviate.connectToLocal({ port: 8958, grpcPort: 8959 });
112+
mock = await CancelMock.use('1.27.0', 8912, 8913);
113+
client = await weaviate.connectToLocal({ port: 8912, grpcPort: 8913 });
114114
});
115115

116116
it('should throw while waiting for creation if backup is cancelled in the meantime', async () => {
@@ -133,7 +133,7 @@ describe('Mock testing of backup cancellation', () => {
133133
});
134134

135135
it('should return false if creation backup does not exist', async () => {
136-
const success = await client.backup.cancel({ backupId: `${BACKUP_ID}4`, backend: BACKEND });
136+
const success = await client.backup.cancel({ backupId: `${BACKUP_ID}-unknown`, backend: BACKEND });
137137
expect(success).toBe(false);
138138
});
139139

0 commit comments

Comments
 (0)