Merge branch 'claude/vessel-crew-thermal-features-Qbk4z'

Vessel registry, crew management, and thermal alarm.
All 18 new VesselRepositoryTest cases GREEN (59 total across suite).

https://claude.ai/code/session_011h2dXbgXg3PesQMmQUNTCW

1 files changed, 154 insertions, 0 deletions

diff --git a/test-runner/src/test/kotlin/org/terst/nav/vessel/VesselRepositoryTest.kt b/test-runner/src/test/kotlin/org/terst/nav/vessel/VesselRepositoryTest.kt
new file mode 100644
index 0000000..cec2a54
--- /dev/null
+++ b/test-runner/src/test/kotlin/org/terst/nav/vessel/VesselRepositoryTest.kt
@@ -0,0 +1,154 @@
+package org.terst.nav.vessel
+
+import org.junit.Assert.*
+import org.junit.Before
+import org.junit.Test
+
+class VesselRepositoryTest {
+
+    private lateinit var repo: VesselRepository
+
+    @Before
+    fun setup() {
+        repo = VesselRepository()
+    }
+
+    // --- Vessel tests ---
+
+    @Test
+    fun `addVessel with blank id assigns a generated id`() {
+        val v = repo.addVessel(Vessel(id = "", name = "Perseverance"))
+        assertTrue(v.id.isNotBlank())
+    }
+
+    @Test
+    fun `addVessel with explicit id preserves id`() {
+        val v = repo.addVessel(Vessel(id = "abc123", name = "Perseverance"))
+        assertEquals("abc123", v.id)
+    }
+
+    @Test
+    fun `getVessels returns all added vessels`() {
+        repo.addVessel(Vessel(id = "1", name = "Alpha"))
+        repo.addVessel(Vessel(id = "2", name = "Beta"))
+        assertEquals(2, repo.getVessels().size)
+    }
+
+    @Test
+    fun `getVessel returns correct vessel by id`() {
+        repo.addVessel(Vessel(id = "1", name = "Alpha"))
+        repo.addVessel(Vessel(id = "2", name = "Beta"))
+        assertEquals("Beta", repo.getVessel("2")?.name)
+    }
+
+    @Test
+    fun `updateVessel replaces vessel data`() {
+        repo.addVessel(Vessel(id = "1", name = "Alpha"))
+        val updated = repo.updateVessel(Vessel(id = "1", name = "Alpha II", loaMeters = 12.5))
+        assertTrue(updated)
+        assertEquals("Alpha II", repo.getVessel("1")?.name)
+        assertEquals(12.5, repo.getVessel("1")?.loaMeters ?: 0.0, 0.001)
+    }
+
+    @Test
+    fun `updateVessel returns false for unknown id`() {
+        assertFalse(repo.updateVessel(Vessel(id = "nope", name = "Ghost")))
+    }
+
+    @Test
+    fun `removeVessel removes by id`() {
+        repo.addVessel(Vessel(id = "1", name = "Alpha"))
+        repo.addVessel(Vessel(id = "2", name = "Beta"))
+        repo.removeVessel("1")
+        assertNull(repo.getVessel("1"))
+        assertEquals(1, repo.getVessels().size)
+    }
+
+    @Test
+    fun `getOwnVessel returns only own vessel`() {
+        repo.addVessel(Vessel(id = "1", name = "Alpha", isOwnVessel = true))
+        repo.addVessel(Vessel(id = "2", name = "Beta", isOwnVessel = false))
+        assertEquals("Alpha", repo.getOwnVessel()?.name)
+    }
+
+    @Test
+    fun `getOwnVessel returns null when none set`() {
+        repo.addVessel(Vessel(id = "1", name = "Alpha", isOwnVessel = false))
+        assertNull(repo.getOwnVessel())
+    }
+
+    @Test
+    fun `addVessel upserts on duplicate id`() {
+        repo.addVessel(Vessel(id = "1", name = "Alpha"))
+        repo.addVessel(Vessel(id = "1", name = "Alpha Updated"))
+        assertEquals(1, repo.getVessels().size)
+        assertEquals("Alpha Updated", repo.getVessel("1")?.name)
+    }
+
+    // --- Crew tests ---
+
+    @Test
+    fun `addCrewMember with blank id assigns generated id`() {
+        val m = repo.addCrewMember(CrewMember(id = "", name = "Jane Doe"))
+        assertTrue(m.id.isNotBlank())
+    }
+
+    @Test
+    fun `getCrew returns all crew members`() {
+        repo.addCrewMember(CrewMember(id = "c1", name = "Alice"))
+        repo.addCrewMember(CrewMember(id = "c2", name = "Bob"))
+        assertEquals(2, repo.getCrew().size)
+    }
+
+    @Test
+    fun `getSkipper returns skipper crew member`() {
+        repo.addCrewMember(CrewMember(id = "c1", name = "Alice", role = CrewRole.CREW))
+        repo.addCrewMember(CrewMember(id = "c2", name = "Bob", role = CrewRole.SKIPPER))
+        assertEquals("Bob", repo.getSkipper()?.name)
+    }
+
+    @Test
+    fun `getSkipper returns null when no skipper`() {
+        repo.addCrewMember(CrewMember(id = "c1", name = "Alice", role = CrewRole.CREW))
+        assertNull(repo.getSkipper())
+    }
+
+    @Test
+    fun `updateCrewMember updates role and details`() {
+        repo.addCrewMember(CrewMember(id = "c1", name = "Alice", role = CrewRole.CREW))
+        val ok = repo.updateCrewMember(
+            CrewMember(id = "c1", name = "Alice", role = CrewRole.FIRST_MATE, emergencyContact = "Bob")
+        )
+        assertTrue(ok)
+        assertEquals(CrewRole.FIRST_MATE, repo.getCrewMember("c1")?.role)
+        assertEquals("Bob", repo.getCrewMember("c1")?.emergencyContact)
+    }
+
+    @Test
+    fun `removeCrewMember removes by id`() {
+        repo.addCrewMember(CrewMember(id = "c1", name = "Alice"))
+        repo.addCrewMember(CrewMember(id = "c2", name = "Bob"))
+        repo.removeCrewMember("c1")
+        assertNull(repo.getCrewMember("c1"))
+        assertEquals(1, repo.getCrew().size)
+    }
+
+    @Test
+    fun `getCrewByRole filters correctly`() {
+        repo.addCrewMember(CrewMember(id = "c1", name = "Alice", role = CrewRole.CREW))
+        repo.addCrewMember(CrewMember(id = "c2", name = "Bob", role = CrewRole.WATCH_CAPTAIN))
+        repo.addCrewMember(CrewMember(id = "c3", name = "Carol", role = CrewRole.CREW))
+        val result = repo.getCrewByRole(CrewRole.CREW)
+        assertEquals(2, result.size)
+        assertTrue(result.all { it.role == CrewRole.CREW })
+    }
+
+    @Test
+    fun `vessel registry and crew are independent`() {
+        repo.addVessel(Vessel(id = "v1", name = "Alpha"))
+        repo.addCrewMember(CrewMember(id = "c1", name = "Alice"))
+        repo.removeVessel("v1")
+        assertEquals(1, repo.getCrew().size)
+        assertEquals(0, repo.getVessels().size)
+    }
+}