Docker: alte Server ohne passenden Host-Namen über die Adresse finden

server/test/dockerHostPool.test.js

@@ -72,8 +72,8 @@ test('Status, Stopp und Befehle gehen an den Host des Matches', async () => {
   await p.stop(r.handle);
   assert.deepEqual(calls.at(-1)[0], 'rm');
   assert.equal(calls.at(-1).at(-1), r.handle.container);
-  // Ein Handle von einem Host, der nicht mehr konfiguriert ist, wird abgelehnt
-  await assert.rejects(p.status({ ...r.handle, hostName: 'gelöscht' }), /nicht \(mehr\) konfiguriert/);
+  // Ein Handle von einem Host, der weder per Name noch per Adresse bekannt ist, wird abgelehnt
+  await assert.rejects(p.status({ ...r.handle, hostName: 'gelöscht', rconHost: '10.9.9.9' }), /nicht \(mehr\) konfiguriert/);
 });
 
 test('Alte Handles ohne Host-Namen laufen auf dem ersten Host', () => {
@@ -96,3 +96,14 @@ test('df-Ausgabe wird in Gesamt- und freien Speicher zerlegt', async () => {
   });
   assert.equal(parseDf('kaputt'), null);
 });
+test('Alter Handle ohne passenden Host-Namen wird über die Adresse dem richtigen Host zugeordnet', async () => {
+  const docker = fakeDocker();
+  const p = new DockerHostPool(options([
+    { ...hostA, rconHost: '10.0.0.1' },
+    { ...hostB, rconHost: '10.0.0.2' },
+  ]), { runDocker: docker.run });
+  await p.stop({ container: 'cs2mv-alt', hostName: 'default', rconHost: '10.0.0.2', port: 27100 });
+  assert.deepEqual(docker.calls.at(-1), ['rm', '-f', 'cs2mv-alt']);
+  assert.equal(p.adapterFor({ hostName: 'default', rconHost: '10.0.0.2' }).options.name, 'b');
+  assert.throws(() => p.adapterFor({ hostName: 'weg', rconHost: '10.9.9.9' }), /nicht \(mehr\) konfiguriert/);
+});