Files
gf/os/gcmd/gcmd_z_unit_test.go
hailaz ee24da4e72 refactor: interface{} to any and reflect.Ptr to reflect.Pointer (#4395)
This pull request standardizes the use of the Go 1.18+ `any` type alias
instead of `interface{}` throughout the codebase. The change improves
code readability and aligns with modern Go best practices. The update
touches many files, including core data structures, code generation
templates, logging utilities, and test data, ensuring consistency across
all usages.

**Type alias migration to `any`:**

* Replaced all instances of `interface{}` with `any` in core data
structures such as `garray` and in generated model structs (e.g.,
`TableUser`, `User1`, `User2`) to modernize type usage.
[[1]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L31-R31)
[[2]](diffhunk://#diff-6c19859cb32c7516ea95ddc8f8235460818eb2f24d2204308e0d9e1b19e7d90fL15-R19)
[[3]](diffhunk://#diff-a15ba2f5e830b4833c47b902515a4f9e5a4f83a3707698f3229b307ec3776b41L15-R18)
[[4]](diffhunk://#diff-52e0837e84d49221d1b810d88fdf78221f36cffcd664fb42f8aba49a79b974dcL15-R19)
[[5]](diffhunk://#diff-11c3457d1a23a4ca6ecd00d6b856289774936b6a708384cf03aff164044e7546L15-R19)
[[6]](diffhunk://#diff-2cff9cf8e6a0cc34087326d8c8149c3bbaf74c76fdbdf5a73daed13cc04249e1L15-R19)
* Updated function signatures, method parameters, and return types from
`interface{}` to `any` in various parts of the codebase, including code
generation, service logic, and logging utilities (e.g., `mlog`).
[[1]](diffhunk://#diff-175edfeea54490b8fe4e18ffcbea5835efaf8f0b8acf623359073987cae7eb76L48-R55)
[[2]](diffhunk://#diff-2b1953fb78cf3593d8c2c7d911e95b65fd0b847c30ed0b4d167d16fe6d781235L54-R74)
[[3]](diffhunk://#diff-e001b7a4b63603b9b14f00de78a4d570bb76c5f57d856a24643f071032e12356L66-R73)
[[4]](diffhunk://#diff-5582954e8a9983988dc8854ad82067fb2ac6269b988e07357ad8db1dfec5f1a0L39-R41)
[[5]](diffhunk://#diff-c5d51d56f487779a2b6207c7ad26c7a20bbadcc846ce094fe60ab4cabff58c51L107-R107)
[[6]](diffhunk://#diff-f96e6a9fdb416eb1804ceaba1fe0ac637bff22c43837f8bb849c2366ce72d4a1L116-R121)
[[7]](diffhunk://#diff-f94c83a1b08ae060d9346f4a6031fc4a7b9a0b894e02d9afaa09018b6598eac0L112-R112)
[[8]](diffhunk://#diff-748b11dbe8828dd4c040ec23cae0b8fe57ecf0a2d1b7694ea39102294e633c64L36-R36)
[[9]](diffhunk://#diff-748b11dbe8828dd4c040ec23cae0b8fe57ecf0a2d1b7694ea39102294e633c64L74-R74)
[[10]](diffhunk://#diff-748b11dbe8828dd4c040ec23cae0b8fe57ecf0a2d1b7694ea39102294e633c64L96-R96)

**Generated code and templates:**

* Adjusted generated files and code generation templates to output `any`
instead of `interface{}` for relevant struct fields and function
signatures, ensuring that new code generation aligns with the updated
convention.
[[1]](diffhunk://#diff-6c19859cb32c7516ea95ddc8f8235460818eb2f24d2204308e0d9e1b19e7d90fL15-R19)
[[2]](diffhunk://#diff-a15ba2f5e830b4833c47b902515a4f9e5a4f83a3707698f3229b307ec3776b41L15-R18)
[[3]](diffhunk://#diff-52e0837e84d49221d1b810d88fdf78221f36cffcd664fb42f8aba49a79b974dcL15-R19)
[[4]](diffhunk://#diff-11c3457d1a23a4ca6ecd00d6b856289774936b6a708384cf03aff164044e7546L15-R19)
[[5]](diffhunk://#diff-2cff9cf8e6a0cc34087326d8c8149c3bbaf74c76fdbdf5a73daed13cc04249e1L15-R19)
[[6]](diffhunk://#diff-175edfeea54490b8fe4e18ffcbea5835efaf8f0b8acf623359073987cae7eb76L48-R55)
[[7]](diffhunk://#diff-e001b7a4b63603b9b14f00de78a4d570bb76c5f57d856a24643f071032e12356L66-R73)
[[8]](diffhunk://#diff-5582954e8a9983988dc8854ad82067fb2ac6269b988e07357ad8db1dfec5f1a0L39-R41)

**Container and utility updates:**

* Refactored the `garray` container implementation and related
constructors/methods to use `[]any` instead of `[]interface{}`, along
with corresponding function signatures.
[[1]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L31-R31)
[[2]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L52-R52)
[[3]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L62-R62)
[[4]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L73-R86)
[[5]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L96-R97)
[[6]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L107-R114)
[[7]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L124-R124)
[[8]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L135-R143)
[[9]](diffhunk://#diff-3a1259e160a4dfa5fe49dfe739fbdb986c0d0a2220a709882ea48d3ae1b8f911L167-R167)

These changes collectively modernize the codebase and prepare it for
future Go developments by using the idiomatic `any` type.
2025-08-28 16:53:19 +08:00

273 lines
7.2 KiB
Go

// Copyright GoFrame Author(https://goframe.org). All Rights Reserved.
//
// This Source Code Form is subject to the terms of the MIT License.
// If a copy of the MIT was not distributed with this file,
// You can obtain one at https://github.com/gogf/gf.
// go test *.go -bench=".*" -benchmem
package gcmd_test
import (
"context"
"fmt"
"os"
"strings"
"testing"
"github.com/gogf/gf/v2/errors/gcode"
"github.com/gogf/gf/v2/errors/gerror"
"github.com/gogf/gf/v2/frame/g"
"github.com/gogf/gf/v2/os/gcmd"
"github.com/gogf/gf/v2/os/gctx"
"github.com/gogf/gf/v2/os/genv"
"github.com/gogf/gf/v2/test/gtest"
)
func Test_Default(t *testing.T) {
gtest.C(t, func(t *gtest.T) {
gcmd.Init([]string{"gf", "--force", "remove", "-fq", "-p=www", "path", "-n", "root"}...)
t.Assert(len(gcmd.GetArgAll()), 2)
t.Assert(gcmd.GetArg(1), "path")
t.Assert(gcmd.GetArg(100, "test"), "test")
t.Assert(gcmd.GetOpt("force"), "remove")
t.Assert(gcmd.GetOpt("n"), "root")
t.Assert(gcmd.GetOpt("fq").IsNil(), false)
t.Assert(gcmd.GetOpt("p").IsNil(), false)
t.Assert(gcmd.GetOpt("none").IsNil(), true)
t.Assert(gcmd.GetOpt("none", "value"), "value")
})
gtest.C(t, func(t *gtest.T) {
gcmd.Init([]string{"gf", "gen", "-h"}...)
t.Assert(len(gcmd.GetArgAll()), 2)
t.Assert(gcmd.GetOpt("h"), "")
t.Assert(gcmd.GetOpt("h").IsNil(), false)
})
}
func Test_BuildOptions(t *testing.T) {
gtest.C(t, func(t *gtest.T) {
s := gcmd.BuildOptions(g.MapStrStr{
"n": "john",
})
t.Assert(s, "-n=john")
})
gtest.C(t, func(t *gtest.T) {
s := gcmd.BuildOptions(g.MapStrStr{
"n": "john",
}, "-test")
t.Assert(s, "-testn=john")
})
gtest.C(t, func(t *gtest.T) {
s := gcmd.BuildOptions(g.MapStrStr{
"n1": "john",
"n2": "huang",
})
t.Assert(strings.Contains(s, "-n1=john"), true)
t.Assert(strings.Contains(s, "-n2=huang"), true)
})
}
func Test_GetWithEnv(t *testing.T) {
gtest.C(t, func(t *gtest.T) {
genv.Set("TEST", "1")
defer genv.Remove("TEST")
t.Assert(gcmd.GetOptWithEnv("test"), 1)
})
gtest.C(t, func(t *gtest.T) {
genv.Set("TEST", "1")
defer genv.Remove("TEST")
gcmd.Init("-test", "2")
t.Assert(gcmd.GetOptWithEnv("test"), 2)
})
}
func Test_Command(t *testing.T) {
gtest.C(t, func(t *gtest.T) {
var (
ctx = gctx.New()
err error
)
commandRoot := &gcmd.Command{
Name: "gf",
}
// env
commandEnv := &gcmd.Command{
Name: "env",
Func: func(ctx context.Context, parser *gcmd.Parser) error {
fmt.Println("env")
return nil
},
}
// test
commandTest := &gcmd.Command{
Name: "test",
Brief: "test brief",
Description: "test description current Golang environment variables",
Examples: `
gf get github.com/gogf/gf
gf get github.com/gogf/gf@latest
gf get github.com/gogf/gf@master
gf get golang.org/x/sys
`,
Arguments: []gcmd.Argument{
{
Name: "my-option",
Short: "o",
Brief: "It's my custom option",
Orphan: true,
},
{
Name: "another",
Short: "a",
Brief: "It's my another custom option",
Orphan: true,
},
},
Func: func(ctx context.Context, parser *gcmd.Parser) error {
fmt.Println("test")
return nil
},
}
err = commandRoot.AddCommand(
commandEnv,
)
if err != nil {
g.Log().Fatal(ctx, err)
}
err = commandRoot.AddObject(
commandTest,
)
if err != nil {
g.Log().Fatal(ctx, err)
}
if err = commandRoot.RunWithError(ctx); err != nil {
if gerror.Code(err) == gcode.CodeNotFound {
commandRoot.Print()
}
}
})
}
func Test_Command_Print(t *testing.T) {
gtest.C(t, func(t *gtest.T) {
var (
ctx = gctx.New()
err error
)
c := &gcmd.Command{
Name: "gf",
Description: `GoFrame Command Line Interface, which is your helpmate for building GoFrame application with convenience.`,
Additional: `
Use 'gf help COMMAND' or 'gf COMMAND -h' for detail about a command, which has '...' in the tail of their comments.`,
}
// env
commandEnv := &gcmd.Command{
Name: "env",
Brief: "show current Golang environment variables, long brief.long brief.long brief.long brief.long brief.long brief.long brief.long brief.",
Description: "show current Golang environment variables",
Func: func(ctx context.Context, parser *gcmd.Parser) error {
return nil
},
}
if err = c.AddCommand(commandEnv); err != nil {
g.Log().Fatal(ctx, err)
}
// get
commandGet := &gcmd.Command{
Name: "get",
Brief: "install or update GF to system in default...",
Description: "show current Golang environment variables",
Examples: `
gf get github.com/gogf/gf
gf get github.com/gogf/gf@latest
gf get github.com/gogf/gf@master
gf get golang.org/x/sys
`,
Func: func(ctx context.Context, parser *gcmd.Parser) error {
return nil
},
}
if err = c.AddCommand(commandGet); err != nil {
g.Log().Fatal(ctx, err)
}
// build
//-n, --name output binary name
//-v, --version output binary version
//-a, --arch output binary architecture, multiple arch separated with ','
//-s, --system output binary system, multiple os separated with ','
//-o, --output output binary path, used when building single binary file
//-p, --path output binary directory path, default is './bin'
//-e, --extra extra custom "go build" options
//-m, --mod like "-mod" option of "go build", use "-m none" to disable go module
//-c, --cgo enable or disable cgo feature, it's disabled in default
commandBuild := gcmd.Command{
Name: "build",
Usage: "gf build FILE [OPTION]",
Brief: "cross-building go project for lots of platforms...",
Description: `
The "build" command is most commonly used command, which is designed as a powerful wrapper for
"go build" command for convenience cross-compiling usage.
It provides much more features for building binary:
1. Cross-Compiling for many platforms and architectures.
2. Configuration file support for compiling.
3. Build-In Variables.
`,
Examples: `
gf build main.go
gf build main.go --swagger
gf build main.go --pack public,template
gf build main.go --cgo
gf build main.go -m none
gf build main.go -n my-app -a all -s all
gf build main.go -n my-app -a amd64,386 -s linux -p .
gf build main.go -n my-app -v 1.0 -a amd64,386 -s linux,windows,darwin -p ./docker/bin
`,
Func: func(ctx context.Context, parser *gcmd.Parser) error {
return nil
},
}
if err = c.AddCommand(&commandBuild); err != nil {
g.Log().Fatal(ctx, err)
}
_ = c.RunWithError(ctx)
})
}
func Test_Command_NotFound(t *testing.T) {
gtest.C(t, func(t *gtest.T) {
c0 := &gcmd.Command{
Name: "c0",
}
c1 := &gcmd.Command{
Name: "c1",
FuncWithValue: func(ctx context.Context, parser *gcmd.Parser) (any, error) {
return nil, nil
},
}
c21 := &gcmd.Command{
Name: "c21",
FuncWithValue: func(ctx context.Context, parser *gcmd.Parser) (any, error) {
return nil, nil
},
}
c22 := &gcmd.Command{
Name: "c22",
FuncWithValue: func(ctx context.Context, parser *gcmd.Parser) (any, error) {
return nil, nil
},
}
t.AssertNil(c0.AddCommand(c1))
t.AssertNil(c1.AddCommand(c21, c22))
os.Args = []string{"c0", "c1", "c23", `--test="abc"`}
err := c0.RunWithError(gctx.New())
t.Assert(err.Error(), `command "c1 c23" not found for command "c0", command line: c0 c1 c23 --test="abc"`)
})
}