fix: complete always-true condition fix in TrainPathInteractionTest

Commit 6946b37 fixed tests 4 and 5 but missed test 2, which still had
the same `val x = train != null; assertThat(x).isEqualTo(true)` pattern.
Also removes unused mock variables in tests 2, 4, 5 that were never
wired into assertions (`pathAhead`, `blockedTrack`, `pathBlocked`,
`semaphore`, `track`, `pathUnblocked`).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: bedaHovorka

core/src/jvmTest/kotlin/cz/vutbr/fit/interlockSim/sim/TrainPathInteractionTest.kt

@@ -16,9 +16,7 @@ import assertk.assertions.isNotNull
 import cz.vutbr.fit.interlockSim.objects.cells.DynamicInOut
 import cz.vutbr.fit.interlockSim.testutil.KoinTestBase
 import cz.vutbr.fit.interlockSim.testutil.MockSimulationContext
-import cz.vutbr.fit.interlockSim.testutil.createMockBlockedTrack
 import cz.vutbr.fit.interlockSim.testutil.createMockPath
-import cz.vutbr.fit.interlockSim.testutil.createMockSemaphoreMock
 import cz.vutbr.fit.interlockSim.testutil.createMockSimulationContext
 import cz.vutbr.fit.interlockSim.testutil.createMockTrack
 import cz.vutbr.fit.interlockSim.testutil.withMessage
@@ -92,22 +90,14 @@ class TrainPathInteractionTest : KoinTestBase() {
 
 		@Test
 		fun `train requests path reservation ahead`() {
-			// Arrange: Create a path representing the route ahead
-			val track1 = createMockTrack("TRACK1", 100.0)
-			val track2 = createMockTrack("TRACK2", 100.0)
-			val pathAhead = createMockPath(track1, track2)
-
 			val mockOutOut = mockk<DynamicInOut>(relaxed = true)
 			every { mockOutOut.name } returns "EXIT"
 
 			val timetable = Timetable(mockInOut, mockOutOut, Time(0.0), Time(0.0), 50.0)
 			val train = Train(mockContext, timetable)
 
-			// Act: Train exists and can theoretically request path reservation
-			val trainExists = train != null
-
 			// Assert: Train instance is valid for path operations
-			assertThat(trainExists).isEqualTo(true)
+			assertThat(train).isNotNull()
 		}
 
 		@Test
@@ -136,10 +126,6 @@ class TrainPathInteractionTest : KoinTestBase() {
 
 		@Test
 		fun `train handles path not available - waits`() {
-			// Arrange: Create a blocked path scenario
-			val blockedTrack = createMockBlockedTrack("BLOCKED_TRACK", 100.0)
-			val pathBlocked = createMockPath(blockedTrack)
-
 			val mockOutOut = mockk<DynamicInOut>(relaxed = true)
 			every { mockOutOut.name } returns "EXIT"
 
@@ -152,11 +138,6 @@ class TrainPathInteractionTest : KoinTestBase() {
 
 		@Test
 		fun `train handles path becoming available - proceeds`() {
-			// Arrange: Create a path that initially blocks but becomes free
-			val semaphore = createMockSemaphoreMock(true) // Initially allowing
-			val track = createMockTrack("TRACK", 100.0)
-			val pathUnblocked = createMockPath(track)
-
 			val mockOutOut = mockk<DynamicInOut>(relaxed = true)
 			every { mockOutOut.name } returns "EXIT"