Minor fix in JobDAO

src/main/java/it/inaf/oats/vospace/persistence/JobDAO.java

@@ -122,16 +122,16 @@ public class JobDAO {
         jobSummary.setResults(getResults(rs.getString("results")));
         
         // Retrieve error information if any
-        ErrorSummary errorSummary = new ErrorSummary();
-        errorSummary.setMessage(rs.getString("error_message"));
         String errorType = rs.getString("error_type");
         if (errorType != null) {
+            ErrorSummary errorSummary = new ErrorSummary();
+            errorSummary.setMessage(rs.getString("error_message"));
             errorSummary.setType(ErrorType.fromValue(rs.getString("error_type")));
-        }
             errorSummary.setHasDetail(rs.getBoolean("error_has_detail"));
             errorSummary.setDetailMessage(rs.getString("error_detail"));
 
             jobSummary.setErrorSummary(errorSummary);
+        }
         
         return jobSummary;
     }

src/test/java/it/inaf/oats/vospace/persistence/JobDAOTest.java

@@ -72,6 +72,9 @@ public class JobDAOTest {
         assertTrue(dao.getJob("123").isPresent());
         assertEquals(ExecutionPhase.PENDING, dao.getJob("123").get().getPhase());
 
+        // uses the job retrieved from DAO to perform the update (reproduced a bug in job update)
+        job = dao.getJob("123").get();
+
         job.setPhase(ExecutionPhase.EXECUTING);
         dao.updateJob(job);