From 1a6428a93c75813613cb2321f976a15a3fff5a3b Mon Sep 17 00:00:00 2001 From: Pato Moschcovich Date: Thu, 12 Jun 2025 14:44:42 -0400 Subject: [PATCH] Updated the unit test for detecting cycles so it shows no cycles initially and later one cycle --- .../org/hjug/dsm/CircularReferenceCheckerTests.java | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/dsm/src/test/java/org/hjug/dsm/CircularReferenceCheckerTests.java b/dsm/src/test/java/org/hjug/dsm/CircularReferenceCheckerTests.java index fb2e8439..a550278f 100644 --- a/dsm/src/test/java/org/hjug/dsm/CircularReferenceCheckerTests.java +++ b/dsm/src/test/java/org/hjug/dsm/CircularReferenceCheckerTests.java @@ -23,9 +23,16 @@ void detectCyclesTest() { classReferencesGraph.addVertex("C"); classReferencesGraph.addEdge("A", "B"); classReferencesGraph.addEdge("B", "C"); - classReferencesGraph.addEdge("C", "A"); + Map> cyclesForEveryVertexMap = sutCircularReferenceChecker.getCycles(classReferencesGraph); - assertEquals(1, cyclesForEveryVertexMap.size()); + assertEquals(0, cyclesForEveryVertexMap.size(), "Not expecting any circular references at this point"); + + classReferencesGraph.addEdge("C", "A"); + + cyclesForEveryVertexMap = sutCircularReferenceChecker.getCycles(classReferencesGraph); + assertEquals(1, cyclesForEveryVertexMap.size(), "Now we expect one circular reference"); + assertEquals("([A, B, C], [(A,B), (B,C), (C,A)])", cyclesForEveryVertexMap.get("A").toString(), + "Expected a different circular reference"); } }