Merge pull request #1562 from SoftUni-Internal/1664-improve-exceptiontype-in-submission-execution

Fixed a bug related to the exception type.

Author: gogo4ds

Servers/Worker/OJS.Servers.Worker/Models/ExecutionResult/FullExecutionResultResponseModel.cs

@@ -5,7 +5,7 @@
 
 public class FullExecutionResultResponseModel
 {
-    public ExceptionModel? Exception { get; set; }
+    public LegacyExceptionModel? Exception { get; set; }
 
     public ExecutionResultResponseModel? ExecutionResult { get; set; }
 
@@ -21,7 +21,7 @@ public void SetExecutionResult(ExecutionResultResponseModel executionResult)
 
     public void SetException(Exception exception, bool includeStackTrace, ExceptionType? exceptionType = null)
     {
-        this.Exception = new ExceptionModel(exception, includeStackTrace, exceptionType);
+        this.Exception = new LegacyExceptionModel(exception, includeStackTrace, exceptionType);
         this.ExecutionResult = null;
     }
 

Services/Common/OJS.Workers/OJS.Workers.Common/Models/LegacyExceptionModel.cs

@@ -0,0 +1,31 @@
+namespace OJS.Workers.Common.Models;
+
+using OJS.Workers.Common.Extensions;
+
+public class LegacyExceptionModel
+{
+    // Used to deserialize from json
+    public LegacyExceptionModel()
+    {
+    }
+
+    public LegacyExceptionModel(
+        Exception exception,
+        bool includeStackTrace = false,
+        ExceptionType? exceptionType = null)
+    {
+        this.Message = exception.GetAllMessages();
+        this.ExceptionType = exceptionType.ToString();
+
+        if (includeStackTrace)
+        {
+            this.StackTrace = exception.GetBaseException().StackTrace;
+        }
+    }
+
+    public string? Message { get; set; }
+
+    public string? StackTrace { get; set; }
+
+    public string? ExceptionType { get; set; }
+}