Merge pull request #954 from HorizenOfficial/st/SDK_1603

Added test on OpenForgerListCmd signature
HorizenOfficial · Nov 9, 2023 · e72528a · e72528a
2 parents 2199d4a + e045bb4
commit e72528a
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/sdk/src/test/scala/io/horizen/account/state/ForgerStakeMsgProcessorTest.scala b/sdk/src/test/scala/io/horizen/account/state/ForgerStakeMsgProcessorTest.scala
@@ -117,6 +117,9 @@ class ForgerStakeMsgProcessorTest
     assertEquals("Wrong MethodId for GetListOfForgersCmd", "f6ad3c23", ForgerStakeMsgProcessor.GetListOfForgersCmd)
     assertEquals("Wrong MethodId for AddNewStakeCmd", "5ca748ff", ForgerStakeMsgProcessor.AddNewStakeCmd)
     assertEquals("Wrong MethodId for RemoveStakeCmd", "f7419d79", ForgerStakeMsgProcessor.RemoveStakeCmd)
+    //TODO OpenStakeForgerListCmd signature is wrong, it misses a closing parenthesis. Fixing it requires an hard fork so
+    // for the moment we stick with the wrong one.
+    assertEquals("Wrong MethodId for OpenStakeForgerListCmd", "b05bf06c", ForgerStakeMsgProcessor.OpenStakeForgerListCmd)
   }
 
   @Test