Merge pull request #1527 from cdowding-sl/fix/1072

Improved JSON deserialisation exception messages
finos · Nov 8, 2019 · 402c219 · 402c219
2 parents 3f941ce + 151bca7
commit 402c219
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/profile/src/main/java/com/scottlogic/deg/profile/serialisation/ProfileDeserialiser.java b/profile/src/main/java/com/scottlogic/deg/profile/serialisation/ProfileDeserialiser.java
@@ -18,12 +18,10 @@
 
 import com.fasterxml.jackson.databind.DeserializationFeature;
 import com.fasterxml.jackson.databind.ObjectMapper;
+import com.scottlogic.deg.common.ValidationException;
 import com.scottlogic.deg.profile.dtos.ProfileDTO;
 import com.scottlogic.deg.profile.reader.FileReader;
 
-import java.io.IOException;
-import java.io.UncheckedIOException;
-
 public class ProfileDeserialiser
 {
 
@@ -35,8 +33,8 @@ public static ProfileDTO deserialise(String json, FileReader fileReader)
         mapper.enable(DeserializationFeature.USE_BIG_DECIMAL_FOR_FLOATS);
         try {
             return mapper.readerFor(ProfileDTO.class).readValue(json);
-        } catch (IOException e) {
-            throw new UncheckedIOException(e);
+        } catch (Exception e) {
+            throw new ValidationException("Profile json is not valid\n" + e.getMessage());
         }
     }
 }