cerc-io/lotus
10
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 27 Wiki Activity

Check decoding params for new methods

Browse Source
This commit is contained in:
Geoff Stuart 2023-02-01 16:44:44 -05:00
parent 0c33eab20a
commit f05437d253
3 changed files with 101 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
.circleci/config.yml
View File

@ -603,6 +603,11 @@ workflows:
suite: itest-deals suite: itest-deals
target: "./itests/deals_test.go" target: "./itests/deals_test.go"
- test:
name: test-itest-decode_params
suite: itest-decode_params
target: "./itests/decode_params_test.go"
- test: - test:
name: test-itest-dup_mpool_messages name: test-itest-dup_mpool_messages
suite: itest-dup_mpool_messages suite: itest-dup_mpool_messages

8
cli/state.go
View File

@ -1354,7 +1354,7 @@ func ComputeStateHTMLTempl(w io.Writer, ts *types.TipSet, o *api.ComputeStateOut
"GetMethod": getMethod, "GetMethod": getMethod,
"ToFil": toFil, "ToFil": toFil,
"JsonParams": JsonParams, "JsonParams": JsonParams,
"JsonReturn": jsonReturn, "JsonReturn": JsonReturn,
"IsSlow": isSlow, "IsSlow": isSlow,
"IsVerySlow": isVerySlow, "IsVerySlow": isVerySlow,
"IntExit": func(i exitcode.ExitCode) int64 { return int64(i) }, "IntExit": func(i exitcode.ExitCode) int64 { return int64(i) },
@ -1440,7 +1440,7 @@ func JsonParams(code cid.Cid, method abi.MethodNum, params []byte) (string, erro
return string(b), err return string(b), err
} }
func jsonReturn(code cid.Cid, method abi.MethodNum, ret []byte) (string, error) { func JsonReturn(code cid.Cid, method abi.MethodNum, ret []byte) (string, error) {
methodMeta, found := filcns.NewActorRegistry().Methods[code][method] // TODO: use remote methodMeta, found := filcns.NewActorRegistry().Methods[code][method] // TODO: use remote
if !found { if !found {
return "", fmt.Errorf("method %d not found on actor %s", method, code) return "", fmt.Errorf("method %d not found on actor %s", method, code)
@ -1549,7 +1549,7 @@ func printReceiptReturn(ctx context.Context, api v0api.FullNode, m *types.Messag
return err return err
} }
jret, err := jsonReturn(act.Code, m.Method, r.Return) jret, err := JsonReturn(act.Code, m.Method, r.Return)
if err != nil { if err != nil {
return err return err
} }
@ -1689,7 +1689,7 @@ var StateCallCmd = &cli.Command{
return xerrors.Errorf("getting actor: %w", err) return xerrors.Errorf("getting actor: %w", err)
} }
retStr, err := jsonReturn(act.Code, abi.MethodNum(method), ret.MsgRct.Return) retStr, err := JsonReturn(act.Code, abi.MethodNum(method), ret.MsgRct.Return)
if err != nil { if err != nil {
return xerrors.Errorf("decoding return: %w", err) return xerrors.Errorf("decoding return: %w", err)
} }

92
itests/decode_params_test.go Normal file
View File

File diff suppressed because one or more lines are too long