From 4c6580e04be3878cb07816151066692f067deab5 Mon Sep 17 00:00:00 2001 From: Leonard Lyubich Date: Thu, 8 Jun 2023 09:31:48 +0400 Subject: [PATCH] tests/migration: Don't pass extra data on NNS contract deployment After recent changes, NNS contract may be deployed with pre-defined TLDs. For this reason, the contract now expects additional parameters in a strictly defined format. Previously, the contract migration testing framework explicitly passed an empty array of arguments as deployment parameters, which started to fail. Pass `nil` data parameter to `neotest.Executor.DeployContract` method in `migration.NewContract`. This heals migration tests. Signed-off-by: Leonard Lyubich --- tests/migration/storage.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/migration/storage.go b/tests/migration/storage.go index 97b9a457..bba63d2c 100644 --- a/tests/migration/storage.go +++ b/tests/migration/storage.go @@ -152,7 +152,7 @@ func NewContract(tb testing.TB, d *dump.Reader, name string, opts ContractOption const nnsSourceCodeDir = "../nns" exec.DeployContract(tb, neotest.CompileFile(tb, exec.CommitteeHash, nnsSourceCodeDir, filepath.Join(nnsSourceCodeDir, "config.yml")), - []interface{}{}, + nil, ) // compile new contract version