only highlight first line for switches #486

autograder-core/src/main/java/de/firemage/autograder/core/CodePosition.java

@@ -2,6 +2,7 @@
 
 import de.firemage.autograder.core.file.SourceInfo;
 import de.firemage.autograder.core.file.SourcePath;
+import spoon.reflect.code.CtAbstractSwitch;
 import spoon.reflect.code.CtLoop;
 import spoon.reflect.cu.SourcePosition;
 import spoon.reflect.declaration.CtElement;
@@ -33,7 +34,7 @@ public static CodePosition fromSourcePosition(
         // Instead of highlighting all lines of a class or method, only highlight the first line.
         //
         // Someone might explicitly specify a source position, in which case it will differ from the element's position.
-        if ((ctElement instanceof CtType<?> || ctElement instanceof CtMethod<?> || ctElement instanceof CtLoop) && ctElement.getPosition().equals(sourcePosition)) {
+        if ((ctElement instanceof CtType<?> || ctElement instanceof CtMethod<?> || ctElement instanceof CtLoop || ctElement instanceof CtAbstractSwitch<?>) && ctElement.getPosition().equals(sourcePosition)) {
             return new CodePosition(
                 sourceInfo,
                 relativePath,

autograder-core/src/test/resources/de/firemage/autograder/core/check_tests/EmptyBlockCheck/config.txt

@@ -8,5 +8,5 @@ Test.java:10
 Test.java:47
 Test.java:50-51
 Test.java:51-52
-Test.java:54-55
+Test.java:54
 Test.java:65

autograder-core/src/test/resources/de/firemage/autograder/core/check_tests/ShouldBeEnumAttribute/config.txt

@@ -1,6 +1,6 @@
 oop.ShouldBeEnumAttribute
 Switch value should be enum attribute.
-Test.java:15-19
-Test.java:23-28
-Test.java:33-41
-Test.java:46-62
+Test.java:15
+Test.java:23
+Test.java:33
+Test.java:46