don't use NewString

onflow · May 2, 2024 · a1765cd · a1765cd
1 parent d42df85
commit a1765cd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/flowkit_test.go b/flowkit_test.go
@@ -1372,9 +1372,7 @@ func TestScripts(t *testing.T) {
 		gw.ExecuteScript.Run(func(args mock.Arguments) {
 			assert.Len(t, string(args.Get(1).([]byte)), 86)
 			assert.Equal(t, "\"Foo\"", args.Get(2).([]cadence.Value)[0].String())
-			retVal, err := cadence.NewString("")
-			require.NoError(t, err)
-			gw.ExecuteScript.Return(retVal, nil)
+			gw.ExecuteScript.Return(cadence.String(""), nil)
 		})
 
 		args := []cadence.Value{cadence.String("Foo")}

diff --git a/mocks/services_mock.go b/mocks/services_mock.go
@@ -207,8 +207,7 @@ func DefaultMockServices() *MockServices {
 	})
 
 	t.ExecuteScript.Run(func(args mock.Arguments) {
-		retVal, err := cadence.NewString("")
-		t.ExecuteScript.Return(retVal, err)
+		t.ExecuteScript.Return(cadence.String(""), nil)
 	})
 
 	t.GetTransactionByID.Return(tests.NewTransaction(), nil)