From f8f8d5effe9dd627ce18d39a50fdbdf6a563b185 Mon Sep 17 00:00:00 2001
From: Anna Shaleva <shaleva.ann@nspcc.ru>
Date: Thu, 17 Nov 2022 16:23:33 +0300
Subject: [PATCH] cli: split TestLoad into multiple parts

Problem: failing part of TestLoad:
```
=== RUN   TestLoad/loadgo,_check_signers
    cli_test.go:160:
        	Error Trace:	/home/circleci/go/src/github.com/nspcc-dev/neo-go/cli/vm/cli_test.go:160
        	            				/home/circleci/go/src/github.com/nspcc-dev/neo-go/cli/vm/cli_test.go:147
        	            				/home/circleci/go/src/github.com/nspcc-dev/neo-go/cli/vm/cli_test.go:444
        	Error:      	command took too long time
        	Test:       	TestLoad/loadgo,_check_signers
```
Solution: split the test into multiple parts to reduce test execution time.
---
 cli/vm/cli_test.go | 54 +++++++++++++++++++++++++++-------------------
 1 file changed, 32 insertions(+), 22 deletions(-)

diff --git a/cli/vm/cli_test.go b/cli/vm/cli_test.go
index 724e2d118..2ee983055 100644
--- a/cli/vm/cli_test.go
+++ b/cli/vm/cli_test.go
@@ -440,28 +440,38 @@ go 1.17`)
 		}
 `
 		filename := prepareLoadgoSrc(t, srcCheckWitness)
-		e := newTestVMCLI(t)
-		e.runProg(t,
-			"loadgo "+filename+" "+ownerAddress, // owner:DefaultScope => true
-			"run main",
-			"loadgo "+filename+" "+ownerAddress+":None", // owner:None => false
-			"run main",
-			"loadgo "+filename+" "+ownerAcc.StringLE(), // ownerLE:DefaultScope => true
-			"run main",
-			"loadgo "+filename+" 0x"+ownerAcc.StringLE(), // owner0xLE:DefaultScope => true
-			"run main",
-			"loadgo "+filename+" "+sideAcc.StringLE(), // sideLE:DefaultScope => false
-			"run main")
-		e.checkNextLine(t, "READY: loaded \\d+ instructions")
-		e.checkStack(t, true)
-		e.checkNextLine(t, "READY: loaded \\d+ instructions")
-		e.checkStack(t, false)
-		e.checkNextLine(t, "READY: loaded \\d+ instructions")
-		e.checkStack(t, true)
-		e.checkNextLine(t, "READY: loaded \\d+ instructions")
-		e.checkStack(t, true)
-		e.checkNextLine(t, "READY: loaded \\d+ instructions")
-		e.checkStack(t, false)
+		t.Run("address", func(t *testing.T) {
+			e := newTestVMCLI(t)
+			e.runProg(t,
+				"loadgo "+filename+" "+ownerAddress, // owner:DefaultScope => true
+				"run main",
+				"loadgo "+filename+" "+ownerAddress+":None", // owner:None => false
+				"run main")
+			e.checkNextLine(t, "READY: loaded \\d+ instructions")
+			e.checkStack(t, true)
+			e.checkNextLine(t, "READY: loaded \\d+ instructions")
+			e.checkStack(t, false)
+		})
+		t.Run("string LE", func(t *testing.T) {
+			e := newTestVMCLI(t)
+			e.runProg(t,
+				"loadgo "+filename+" "+ownerAcc.StringLE(), // ownerLE:DefaultScope => true
+				"run main",
+				"loadgo "+filename+" 0x"+ownerAcc.StringLE(), // owner0xLE:DefaultScope => true
+				"run main")
+			e.checkNextLine(t, "READY: loaded \\d+ instructions")
+			e.checkStack(t, true)
+			e.checkNextLine(t, "READY: loaded \\d+ instructions")
+			e.checkStack(t, true)
+		})
+		t.Run("nonwitnessed signer", func(t *testing.T) {
+			e := newTestVMCLI(t)
+			e.runProg(t,
+				"loadgo "+filename+" "+sideAcc.StringLE(), // sideLE:DefaultScope => false
+				"run main")
+			e.checkNextLine(t, "READY: loaded \\d+ instructions")
+			e.checkStack(t, false)
+		})
 	})
 	t.Run("loadnef", func(t *testing.T) {
 		config.Version = "0.92.0-test"