diff --git a/bots/mlbridge/src/main/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterWorkItem.java b/bots/mlbridge/src/main/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterWorkItem.java index 6a0f9f31c..83730679f 100644 --- a/bots/mlbridge/src/main/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterWorkItem.java +++ b/bots/mlbridge/src/main/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterWorkItem.java @@ -46,7 +46,7 @@ public Collection run(Path scratchPath) { var configuredLabels = bot.lists().stream() .flatMap(configuration -> configuration.labels().stream() - .map(labelName -> new Label(labelName, configuration.list().toString()))) + .map(labelName -> new Label(labelName, configuration.list().address()))) .toList(); for (Label configuredLabel : configuredLabels) { @@ -61,7 +61,7 @@ public Collection run(Path scratchPath) { } } - log.fine("Done updating labels for: " + bot.codeRepo()); + log.fine("Done updating labels for: " + bot.codeRepo().name()); bot.setLabelsUpdated(true); return List.of(); } diff --git a/bots/mlbridge/src/test/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterTests.java b/bots/mlbridge/src/test/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterTests.java index 4e23d7e83..dd5af8ab3 100644 --- a/bots/mlbridge/src/test/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterTests.java +++ b/bots/mlbridge/src/test/java/org/openjdk/skara/bots/mlbridge/LabelsUpdaterTests.java @@ -32,10 +32,10 @@ void simple(TestInfo testInfo) throws IOException { assertEquals(2, targetRepo.labels().size(), "Wrong number of labels"); assertTrue(targetRepo.labels().stream() - .anyMatch(l -> l.name().equals("foo") && l.description().orElseThrow().equals(listAddress.toString())), + .anyMatch(l -> l.name().equals("foo") && l.description().orElseThrow().equals(listAddress.address())), "No label 'foo' found"); assertTrue(targetRepo.labels().stream() - .anyMatch(l -> l.name().equals("bar") && l.description().orElseThrow().equals(listAddress.toString())), + .anyMatch(l -> l.name().equals("bar") && l.description().orElseThrow().equals(listAddress.address())), "No label 'bar' found"); // Run again and expect no change @@ -65,7 +65,7 @@ void update(TestInfo testInfo) throws IOException { assertEquals(1, targetRepo.labels().size(), "Wrong number of labels"); assertTrue(targetRepo.labels().stream() - .anyMatch(l -> l.name().equals("foo") && l.description().orElseThrow().equals(listAddress.toString())), + .anyMatch(l -> l.name().equals("foo") && l.description().orElseThrow().equals(listAddress.address())), "No label 'foo' found"); var mlBot2 = MailingListBridgeBot.newBuilder() @@ -78,7 +78,7 @@ void update(TestInfo testInfo) throws IOException { assertEquals(1, targetRepo.labels().size(), "Wrong number of labels"); assertTrue(targetRepo.labels().stream() - .anyMatch(l -> l.name().equals("foo") && l.description().orElseThrow().equals(listAddress2.toString())), + .anyMatch(l -> l.name().equals("foo") && l.description().orElseThrow().equals(listAddress2.address())), "No label 'foo' found"); } }