Let plugin fail on error (#6)

.github/workflows/build.yml

@@ -23,7 +23,7 @@ jobs:
         uses: actions/setup-java@v3
         with:
           distribution: 'temurin'
-          java-version: '8'
+          java-version: '17'
           cache: 'gradle'
       - name: Production Build
         run: ./gradlew clean jar

src/main/java/de/tum/cit/ase/artemis_notification_plugin/NotificationPlugin.java

@@ -3,6 +3,7 @@
 import com.google.gson.Gson;
 import de.tum.cit.ase.artemis_notification_plugin.configuration.Context;
 import de.tum.cit.ase.artemis_notification_plugin.configuration.ContextFactory;
+import de.tum.cit.ase.artemis_notification_plugin.exception.PostResultException;
 import de.tum.cit.ase.artemis_notification_plugin.exception.TestParsingException;
 import de.tum.cit.ase.artemis_notification_plugin.model.Commit;
 import de.tum.cit.ase.artemis_notification_plugin.model.TestResults;
@@ -91,6 +92,7 @@ public void postResult(TestResults results, Context context) {
         }
         catch (HttpException | IOException e) {
             LOGGER.error(e.getMessage(), e);
+            throw new PostResultException(e);
         }
     }
 

src/main/java/de/tum/cit/ase/artemis_notification_plugin/exception/PostResultException.java

@@ -0,0 +1,8 @@
+package de.tum.cit.ase.artemis_notification_plugin.exception;
+
+public class PostResultException extends RuntimeException {
+
+    public PostResultException(Throwable cause) {
+        super(cause);
+    }
+}

src/test/java/de/tum/cit/ase/artemis_notification_plugin/model/TestsuiteTest.java

@@ -1,5 +1,8 @@
 package de.tum.cit.ase.artemis_notification_plugin.model;
 
+import de.tum.cit.ase.artemis_notification_plugin.CLIPlugin;
+import de.tum.cit.ase.artemis_notification_plugin.configuration.Context;
+import de.tum.cit.ase.artemis_notification_plugin.exception.PostResultException;
 import org.junit.jupiter.api.Test;
 
 import javax.xml.bind.JAXBContext;
@@ -62,6 +65,12 @@ void testFlattenNestedSuiteWithFailures() throws Exception {
         assertEquals(1, flattened.getErrors());
     }
 
+    @Test
+    void testPostResultFailure() {
+        CLIPlugin cliPlugin = new CLIPlugin();
+        assertThrows(PostResultException.class, () -> cliPlugin.postResult(new TestResults(), new Context(null, null, null, null, null, null, null, null, null, null, null, "http://invalid", "secret")));
+    }
+
     private Testsuite loadTestSuite(final Path reportXml) throws JAXBException {
         Path resourcePath = new File("testsuite_examples").toPath().resolve(reportXml);
         URL resource = getClass().getClassLoader().getResource(resourcePath.toString());