Use eval.InvalidArgError() to dsl.Payload() (#3689)

Author: tchssk

dsl/payload.go

@@ -176,13 +176,13 @@ func methodDSL(m *expr.MethodExpr, suffix string, p any, args ...any) *expr.Attr
 	case expr.DataType:
 		att = &expr.AttributeExpr{Type: actual}
 	default:
-		eval.ReportError("invalid %s argument, must be a type or a function", suffix)
+		eval.InvalidArgError("type or function", p)
 		return nil
 	}
 	if len(args) >= 1 {
 		if f, ok := args[len(args)-1].(func()); ok {
 			if fn != nil {
-				eval.ReportError("invalid arguments in %s call, must be (type), (func), (type, func), (type, desc) or (type, desc, func)", suffix)
+				eval.InvalidArgError("(type), (func), (type, func), (type, desc) or (type, desc, func)", f)
 			}
 			fn = f
 		}

eval/eval_test.go

@@ -30,6 +30,8 @@ func TestInvalidArgError(t *testing.T) {
 		"OneOf (function)":     {func() { Type("name", func() { OneOf("name", "description", 1) }) }, "cannot use 1 (type int) as type function"},
 		"OneOf (string)":       {func() { Type("name", func() { OneOf("name", 1, func() {}) }) }, "cannot use 1 (type int) as type string"},
 		"Param":                {func() { API("name", func() { HTTP(func() { Params(1) }) }) }, "cannot use 1 (type int) as type function"},
+		"Payload":              {func() { Service("s", func() { Method("m", func() { Payload(1) }) }) }, "cannot use 1 (type int) as type type or function"},
+		"Payload (args)":       {func() { Service("s", func() { Method("m", func() { Payload(func() {}, func() {}) }) }) }, "(type func()) as type (type), (func), (type, func), (type, desc) or (type, desc, func)"},
 		"Response":             {func() { Service("s", func() { HTTP(func() { Response(1) }) }) }, "cannot use 1 (type int) as type name of error"},
 		"ResultType":           {func() { ResultType("identifier", 1) }, "cannot use 1 (type int) as type function or string"},
 		"Security":             {func() { Security(1) }, "cannot use 1 (type int) as type security scheme or security scheme name"},