Avoid SerializationException when Exception deserialize

planetarium · Jul 3, 2024 · 509cc5b · 509cc5b
1 parent adc7c30
commit 509cc5b
Show file tree

Hide file tree

Showing 10 changed files with 94 additions and 0 deletions.
diff --git a/.Lib9c.Tests/Action/ExceptionTest.cs b/.Lib9c.Tests/Action/ExceptionTest.cs
@@ -8,6 +8,8 @@ namespace Lib9c.Tests.Action
     using MessagePack;
     using MessagePack.Resolvers;
     using Nekoyume.Action;
+    using Nekoyume.Action.Exceptions;
+    using Nekoyume.Action.Exceptions.AdventureBoss;
     using Nekoyume.Exceptions;
     using Nekoyume.Model.State;
     using Nekoyume.TableData;
@@ -73,6 +75,15 @@ public ExceptionTest()
         [InlineData(typeof(ItemNotFoundException))]
         [InlineData(typeof(NotEnoughItemException))]
         [InlineData(typeof(StateNullException))]
+        [InlineData(typeof(AlreadyClaimedException))]
+        [InlineData(typeof(ClaimExpiredException))]
+        [InlineData(typeof(InsufficientStakingException))]
+        [InlineData(typeof(InvalidAdventureBossSeasonException))]
+        [InlineData(typeof(InvalidBountyException))]
+        [InlineData(typeof(MaxInvestmentCountExceededException))]
+        [InlineData(typeof(PreviousBountyException))]
+        [InlineData(typeof(SeasonInProgressException))]
+        [InlineData(typeof(EmptyRewardException))]
         public void Exception_Serializable(Type excType)
         {
             if (Activator.CreateInstance(excType, "for testing") is Exception exc)

diff --git a/Lib9c/Action/Exceptions/AdventureBoss/AlreadyClaimedException.cs b/Lib9c/Action/Exceptions/AdventureBoss/AlreadyClaimedException.cs
@@ -1,10 +1,19 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
     [Serializable]
     public class AlreadyClaimedException : Exception
     {
+        public AlreadyClaimedException()
+        {
+        }
+
+        protected AlreadyClaimedException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
+
         public AlreadyClaimedException(string msg) : base(msg)
         {
         }

diff --git a/Lib9c/Action/Exceptions/AdventureBoss/ClaimExpiredException.cs b/Lib9c/Action/Exceptions/AdventureBoss/ClaimExpiredException.cs
@@ -1,10 +1,19 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
     [Serializable]
     public class ClaimExpiredException : Exception
     {
+        public ClaimExpiredException()
+        {
+        }
+
+        protected ClaimExpiredException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
+
         public ClaimExpiredException(string msg) : base(msg)
         {
         }

diff --git a/Lib9c/Action/Exceptions/AdventureBoss/InsufficientStakingException.cs b/Lib9c/Action/Exceptions/AdventureBoss/InsufficientStakingException.cs
@@ -1,10 +1,19 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
     [Serializable]
     public class InsufficientStakingException : Exception
     {
+        public InsufficientStakingException()
+        {
+        }
+
+        protected InsufficientStakingException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
+
         public InsufficientStakingException(string msg) : base(msg)
         {
         }

diff --git a/Lib9c/Action/Exceptions/AdventureBoss/InvalidAdventureBossSeasonException.cs b/Lib9c/Action/Exceptions/AdventureBoss/InvalidAdventureBossSeasonException.cs
@@ -1,10 +1,18 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
     [Serializable]
     public class InvalidAdventureBossSeasonException : Exception
     {
+        public InvalidAdventureBossSeasonException()
+        {
+        }
+
+        protected InvalidAdventureBossSeasonException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
 
         public InvalidAdventureBossSeasonException(string message) : base(message)
         {

diff --git a/Lib9c/Action/Exceptions/AdventureBoss/InvalidBountyException.cs b/Lib9c/Action/Exceptions/AdventureBoss/InvalidBountyException.cs
@@ -1,10 +1,19 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
     [Serializable]
     public class InvalidBountyException : Exception
     {
+        public InvalidBountyException()
+        {
+        }
+
+        protected InvalidBountyException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
+
         public InvalidBountyException(string msg) : base(msg)
         {
         }

diff --git a/Lib9c/Action/Exceptions/AdventureBoss/MaxInvestmentCountExceededException.cs b/Lib9c/Action/Exceptions/AdventureBoss/MaxInvestmentCountExceededException.cs
@@ -1,10 +1,19 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
     [Serializable]
     public class MaxInvestmentCountExceededException : Exception
     {
+        public MaxInvestmentCountExceededException()
+        {
+        }
+
+        protected MaxInvestmentCountExceededException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
+
         public MaxInvestmentCountExceededException(string message) : base(message)
         {
         }

diff --git a/Lib9c/Action/Exceptions/AdventureBoss/PreviousBountyException.cs b/Lib9c/Action/Exceptions/AdventureBoss/PreviousBountyException.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
@@ -8,5 +9,13 @@ public class PreviousBountyException : Exception
         public PreviousBountyException(string msg) : base(msg)
         {
         }
+
+        public PreviousBountyException()
+        {
+        }
+
+        protected PreviousBountyException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
     }
 }
diff --git a/Lib9c/Action/Exceptions/AdventureBoss/SeasonInProgressException.cs b/Lib9c/Action/Exceptions/AdventureBoss/SeasonInProgressException.cs
@@ -1,10 +1,19 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions.AdventureBoss
 {
     [Serializable]
     public class SeasonInProgressException : Exception
     {
+        public SeasonInProgressException()
+        {
+        }
+
+        protected SeasonInProgressException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
+
         public SeasonInProgressException(string msg) : base(msg)
         {
         }

diff --git a/Lib9c/Action/Exceptions/EmptyRewardException.cs b/Lib9c/Action/Exceptions/EmptyRewardException.cs
@@ -1,12 +1,24 @@
 using System;
+using System.Runtime.Serialization;
 
 namespace Nekoyume.Action.Exceptions
 {
     [Serializable]
     public class EmptyRewardException : Exception
     {
+        public EmptyRewardException()
+        {
+        }
+
         public EmptyRewardException(string msg) : base(msg)
         {
         }
+
+        protected EmptyRewardException(
+            SerializationInfo info,
+            StreamingContext context)
+            : base(info, context)
+        {
+        }
     }
 }