split up common reimplementation check #415

Feuermagier · Apr 4, 2024 · 9efb5ea · 9efb5ea
1 parent f20a866
commit 9efb5ea
Show file tree

Hide file tree

Showing 13 changed files with 1,036 additions and 818 deletions.
diff --git a/autograder-core/src/main/java/de/firemage/autograder/core/check/api/CollectionAddAll.java b/autograder-core/src/main/java/de/firemage/autograder/core/check/api/CollectionAddAll.java
@@ -7,7 +7,7 @@
 import de.firemage.autograder.core.integrated.IntegratedCheck;
 import de.firemage.autograder.core.integrated.SpoonUtil;
 import de.firemage.autograder.core.integrated.StaticAnalysis;
-import de.firemage.autograder.core.check.api.CommonReimplementation.CtEnumFieldRead;
+import de.firemage.autograder.core.check.api.UseEnumValues.CtEnumFieldRead;
 import spoon.processing.AbstractProcessor;
 import spoon.reflect.code.CtBlock;
 import spoon.reflect.code.CtExpression;
@@ -94,7 +94,7 @@ private void reportProblem(CtVariable<?> ctVariable, List<CtExpression<?>> added
             fieldReads.add(fieldRead.ctEnumValue());
         }
 
-        if (ctEnum != null && CommonReimplementation.checkEnumValues(ctEnum, isOrderedCollection(ctVariable.getType()), fieldReads)) {
+        if (ctEnum != null && UseEnumValues.checkEnumValues(ctEnum, isOrderedCollection(ctVariable.getType()), fieldReads)) {
             addLocalProblem(
                 ctVariable,
                 new LocalizedMessage(