From d6d128eeced62e867bd5c804eef72589b3af7bb4 Mon Sep 17 00:00:00 2001
From: simonh5 <simonh5@illinois.edu>
Date: Mon, 23 Oct 2023 15:47:52 -0500
Subject: [PATCH] fix flaky test

---
 .../check/BrokenCrossReferencesChecker.groovy                | 2 +-
 .../check/BrokenCrossReferencesCheckerTest.groovy            | 5 ++---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/main/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesChecker.groovy b/src/main/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesChecker.groovy
index 8fcdafe3..569dc717 100644
--- a/src/main/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesChecker.groovy
+++ b/src/main/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesChecker.groovy
@@ -38,7 +38,7 @@ class BrokenCrossReferencesChecker extends SuggestingChecker {
     protected SingleCheckResults check(final HtmlPage pageToCheck) {
         //get list of all a-tags "<a href=..." in html file
         hrefList = pageToCheck.getAllHrefStrings()
-        hrefSet = hrefList.toSet()
+        hrefSet = hrefList.toSet().sort()
 
         // get list of all id="XYZ"
         listOfIds = pageToCheck.getAllIdStrings()
diff --git a/src/test/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesCheckerTest.groovy b/src/test/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesCheckerTest.groovy
index f9bf09a2..d3eb9cf6 100644
--- a/src/test/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesCheckerTest.groovy
+++ b/src/test/groovy/org/aim42/htmlsanitycheck/check/BrokenCrossReferencesCheckerTest.groovy
@@ -101,14 +101,13 @@ class BrokenCrossReferencesCheckerTest extends GroovyTestCase {
 
         // first finding: aim42 link missing
         String actual = collector.findings.first()
-        String expected = "link target \"arc42\" missing"
+        String expected = "link target \"aim42\" missing"
         String message = "expected $expected"
-
         assertEquals(message, expected, actual)
 
         // second finding: arc42 link missing
         actual = collector.findings[1]
-        expected = "link target \"aim42\" missing"
+        expected = "link target \"arc42\" missing"
         assertEquals(message, expected, actual)
     }