Skip to content

Commit

Permalink
tests/migration: Don't pass extra data on NNS contract deployment
Browse files Browse the repository at this point in the history
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 <leonard@morphbits.io>
  • Loading branch information
cthulhu-rider committed Jun 8, 2023
1 parent 850c3a6 commit 4c6580e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/migration/storage.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 4c6580e

Please sign in to comment.