Changed a few small things

src/main/java/previewcode/backend/DTO/OrderingGroup.java

@@ -51,13 +51,18 @@ public OrderingGroup(String title, String description, List<HunkChecksum> hunks)
         this.defaultGroup = false;
     }
 
-    public OrderingGroup(String title, String description, List<HunkChecksum> hunks, boolean defaultGroup) {
+    public OrderingGroup(String title, String description, List<HunkChecksum> hunks, Boolean defaultGroup) {
         this.hunkChecksums = hunks;
         if(title == null){
             title = "";
         }
+
+        if(defaultGroup == null) {
+            this.defaultGroup = false;
+        } else {
+            this.defaultGroup = defaultGroup;
+        }
         this.info = new TitleDescription(title, description);
-        this.defaultGroup = defaultGroup;
 
     }
 

src/main/java/previewcode/backend/services/DatabaseService.java

@@ -21,12 +21,12 @@ public Action<Unit> updateOrdering(PullRequestIdentifier pull, List<OrderingGrou
         return insertPullIfNotExists(pull).then(pullID ->
                 fetchGroups(pullID).then(existingGroups ->
                         traverse(newGroups, createGroup(pullID))
-                        .then(deriveDefaultGroup(existingGroups, newGroups)
-                             .then(defaultGroup -> {
+                            .then(deriveDefaultGroup(existingGroups, newGroups)
+                            .then(defaultGroup -> traverse(existingGroups, g -> delete(g.id)).pure(defaultGroup))
+                            .then(defaultGroup -> {
                                  if (!defaultGroup.isEmpty()) return createGroup(pullID).apply(defaultGroup);
                                  else return pure(unit);
                              }))
-                        .then(traverse(existingGroups, g -> delete(g.id)))
                 )
         ).toUnit();
     }

src/main/java/previewcode/backend/services/actiondsl/Interpreter.java

@@ -6,6 +6,7 @@
 import io.vavr.CheckedFunction1;
 import io.vavr.collection.List;
 
+import javax.ws.rs.core.MediaType;
 import javax.ws.rs.core.Response;
 import java.util.HashMap;
 import java.util.Map;
@@ -170,7 +171,7 @@ private static <T extends Throwable, R> R sneakyThrow(Throwable t) throws T {
      * @return The response built from the action result.
      */
     public Response evaluateToResponse(Action<?> action) {
-        return Response.ok().entity(unsafeEvaluate(action)).build();
+        return Response.ok().type(MediaType.APPLICATION_JSON_TYPE).entity(unsafeEvaluate(action)).build();
     }
 
     protected <A, X> Try<A> run(Action<A> action) {

src/main/java/previewcode/backend/services/interpreters/DatabaseInterpreter.java

@@ -124,6 +124,7 @@ protected PullRequestID fetchPullRequest(FetchPull action) {
 
     protected GroupID insertNewGroup(NewGroup newGroup) {
         Boolean defaultGroup = newGroup.defaultGroup;
+        //Needed for the uniqueness constraint, as there should only be one default group
         if(!newGroup.defaultGroup) {
             defaultGroup = null;
         }

src/main/java/previewcode/backend/services/interpreters/GitHubAuthInterpreter.java

@@ -74,8 +74,8 @@ public GitHubAuthInterpreter(
     }
 
     public static ActionCache.Builder configure(ActionCache.Builder b) {
-        return b.expire(GetUser.class).afterWrite(15, TimeUnit.MINUTES)
-                .expire(AuthenticateInstallation.class).afterWrite(1, TimeUnit.HOURS);
+        return b.expire(GetUser.class).afterWrite(15, TimeUnit.MINUTES);
+                //.expire(AuthenticateInstallation.class).afterWrite(1, TimeUnit.HOURS);
     }
 
     protected Unit authInstallation(AuthenticateInstallation action) throws IOException {

src/test/java/previewcode/backend/api/v2/EndPointTest.java

@@ -9,6 +9,7 @@
 import previewcode.backend.APIModule;
 import previewcode.backend.DTO.*;
 import previewcode.backend.services.IGithubService;
+import previewcode.backend.services.actiondsl.ActionDSL;
 import previewcode.backend.services.actiondsl.Interpreter;
 import previewcode.backend.test.helpers.ApiEndPointTest;
 import previewcode.backend.database.PullRequestGroup;
@@ -131,6 +132,10 @@ public Action<Ordering> getOrdering(PullRequestIdentifier pull) {
         return new NoOp<>();
     }
 
+    @Override
+    public Action<Unit> mergeNewHunks(PullRequestIdentifier pull, List<HunkChecksum> newHunks) {
+        return new NoOp<>();
+    }
 
 
     @Override