Fix GitHub Actions CI compilation error

Fixed compilation error in UT_MainService_Contracts.cs that was preventing CI from passing:
- Updated test setup to use manual ContractState creation instead of internal TestUtils.GetContract method
- Fixed NeoSystem field injection from static to instance field
- Improved invokeabi command logic to find methods by name first, then validate argument count
- Updated test assertions to match new error message format
- All tests now pass (21/21)

Author: Jim8y

src/Neo.CLI/CLI/MainService.Contracts.cs

@@ -209,11 +209,11 @@ private void OnInvokeAbiCommand(UInt160 scriptHash, string operation, JArray? ar
                 return;
             }
 
-            // Find the method in the ABI
-            var method = contract.Manifest.Abi.GetMethod(operation, args?.Count ?? 0);
+            // Find the method in the ABI by name only
+            var method = contract.Manifest.Abi.GetMethod(operation, -1);
             if (method == null)
             {
-                ConsoleHelper.Error($"Method '{operation}' with {args?.Count ?? 0} parameters does not exist in this contract.");
+                ConsoleHelper.Error($"Method '{operation}' does not exist in this contract.");
                 return;
             }
 

tests/Neo.CLI.Tests/UT_MainService_Contracts.cs

@@ -59,8 +59,10 @@ public void TestSetup()
             _mainService = new MainService();
 
             // Set NeoSystem using reflection
-            var neoSystemField = typeof(MainService).GetField("NeoSystem", BindingFlags.NonPublic | BindingFlags.Static);
-            neoSystemField?.SetValue(null, _neoSystem);
+            var neoSystemField = typeof(MainService).GetField("_neoSystem", BindingFlags.NonPublic | BindingFlags.Instance);
+            if (neoSystemField == null)
+                Assert.Fail("_neoSystem field not found");
+            neoSystemField.SetValue(_mainService, _neoSystem);
 
             // Setup mock wallet
             _mockWallet = new Mock<Wallet>();
@@ -163,10 +165,24 @@ private void SetupTestContract()
             sb.Emit(OpCode.RET);
             var script = sb.ToArray();
 
-            // Create the contract using TestUtils
-            _contractState = TestUtils.GetContract(script, manifest);
-            _contractState.Id = 1;
-            _contractState.Hash = _contractHash;
+            // Create NefFile
+            var nef = new NefFile
+            {
+                Compiler = "",
+                Source = "",
+                Tokens = Array.Empty<MethodToken>(),
+                Script = script
+            };
+            nef.CheckSum = NefFile.ComputeChecksum(nef);
+
+            // Create the contract state manually
+            _contractState = new ContractState
+            {
+                Id = 1,
+                Hash = _contractHash,
+                Nef = nef,
+                Manifest = manifest
+            };
 
             // Properly add the contract to the test snapshot using the extension method
             var snapshot = _neoSystem.GetSnapshotCache();
@@ -390,7 +406,7 @@ public void TestInvokeAbiCommand_MethodNotFound()
 
             // Assert
             var output = _consoleOutput.ToString();
-            Assert.IsTrue(output.Contains("Method 'nonExistentMethod' with 0 parameters does not exist"));
+            Assert.IsTrue(output.Contains("Method 'nonExistentMethod' does not exist"));
         }
 
         [TestMethod]
@@ -406,7 +422,7 @@ public void TestInvokeAbiCommand_WrongParameterCount()
 
             // Assert
             var output = _consoleOutput.ToString();
-            Assert.IsTrue(output.Contains("Method 'testBoolean' with 2 parameters does not exist"));
+            Assert.IsTrue(output.Contains("Too many arguments. Method 'testBoolean' expects 1 parameters"));
         }
 
         [TestMethod]