fix: result.Ok to keep debugpy from converting Ok to Err

This is a strange bug in debugpy that randomly changes
an Ok variant to an Err variant. What happens is in the
background the python debugger evaluates all class properties.
But evaluating result.Err for an Ok(value) converts it to
an Err(e) and returns itself (since it resulted in an error).

This leads to random changing of Ok(value) variables by
the debugger, that would then work perfectly fine under
normal execution. This fix for changes both the Ok and
Err properties to return a new instance of an Err(e)
if they result in an error and DOES NOT alter self.

Author: ScottBoyce

ResultContainer/__init__.py

@@ -1087,10 +1087,14 @@ def Ok(self):
         Returns:
             value stored in Ok(value) or raise ResultErr
         """
-        if self._success:
-            return self._val
-        self.add_Err_msg("Result.Ok attribute for Err variant", 15, add_traceback=True)  # 15: "not_Ok",
-        raise self._val
+        if not self._success:
+            # Result.Ok raises error for Err variant
+            # Have to operate on new instance for debugpy, otherwise the Locals inspection will convert self to Err.
+            # old method: self.add_Err_msg("Result.Ok attribute for Err variant", 15)
+            err = ResultErr(self._val)
+            err.append("Result.Ok attribute for Err variant", 15, add_traceback=True)  # 15: "not_Ok",
+            raise err
+        return self._val
 
     @property
     def Err(self):