diff --git a/core/src/test/java/ai/timefold/solver/core/impl/heuristic/selector/move/generic/RuinRecreateMoveSelectorTest.java b/core/src/test/java/ai/timefold/solver/core/impl/heuristic/selector/move/generic/RuinRecreateMoveSelectorTest.java index 5665e102af..2755e8bd98 100644 --- a/core/src/test/java/ai/timefold/solver/core/impl/heuristic/selector/move/generic/RuinRecreateMoveSelectorTest.java +++ b/core/src/test/java/ai/timefold/solver/core/impl/heuristic/selector/move/generic/RuinRecreateMoveSelectorTest.java @@ -24,7 +24,6 @@ import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; -import org.junit.jupiter.api.RepeatedTest; import org.junit.jupiter.api.Test; import io.micrometer.core.instrument.MeterRegistry; @@ -59,7 +58,7 @@ void testRuining() { assertDoesNotThrow(() -> solver.solve(problem)); } - @RepeatedTest(2) + @Test void testRuiningWithMetric() { var meterRegistry = new TestMeterRegistry(); Metrics.addRegistry(meterRegistry); diff --git a/core/src/test/java/ai/timefold/solver/core/impl/solver/DefaultSolverTest.java b/core/src/test/java/ai/timefold/solver/core/impl/solver/DefaultSolverTest.java index b9d8eefdd1..68e81cfb17 100644 --- a/core/src/test/java/ai/timefold/solver/core/impl/solver/DefaultSolverTest.java +++ b/core/src/test/java/ai/timefold/solver/core/impl/solver/DefaultSolverTest.java @@ -11,6 +11,7 @@ import java.util.Collections; import java.util.List; import java.util.Map; +import java.util.UUID; import java.util.concurrent.CountDownLatch; import java.util.concurrent.ExecutionException; import java.util.concurrent.Executors; @@ -426,7 +427,7 @@ void solveMetrics() { SolverFactory solverFactory = SolverFactory.create(solverConfig); var solver = solverFactory.buildSolver(); - ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", "solveMetrics")); + ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", UUID.randomUUID().toString())); meterRegistry.publish(); var solution = new TestdataSolution("s1"); @@ -566,7 +567,7 @@ void solveBestScoreMetrics() { SolverFactory solverFactory = SolverFactory.create(solverConfig); var solver = solverFactory.buildSolver(); - ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", "solveMetrics")); + ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", UUID.randomUUID().toString())); meterRegistry.publish(); var solution = new TestdataHardSoftScoreSolution("s1"); solution.setValueList(Arrays.asList(new TestdataValue("none"), new TestdataValue("reward"))); @@ -670,7 +671,7 @@ void solveStepScoreMetrics() { SolverFactory solverFactory = SolverFactory.create(solverConfig); var solver = solverFactory.buildSolver(); - ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", "solveMetrics")); + ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", UUID.randomUUID().toString())); var step = new AtomicInteger(-1); ((DefaultSolver) solver) @@ -750,7 +751,7 @@ void solveMetricsError() { SolverFactory solverFactory = SolverFactory.create(solverConfig); var solver = solverFactory.buildSolver(); - ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", "solveMetricsError")); + ((DefaultSolver) solver).setMonitorTagMap(Map.of("solver.id", UUID.randomUUID().toString())); meterRegistry.publish(); var solution = new TestdataSolution("s1"); @@ -759,9 +760,8 @@ void solveMetricsError() { meterRegistry.publish(); - assertThatCode(() -> { - solver.solve(solution); - }).hasStackTraceContaining("Thrown exception in constraint provider"); + assertThatCode(() -> solver.solve(solution)) + .hasStackTraceContaining("Thrown exception in constraint provider"); meterRegistry.getClock().addSeconds(1); meterRegistry.publish();