Compare commits

...
2 Commits
Author SHA1 Message Date
queena f7d63d6e8f Merge pull request '測試改斷言具體結束碼數值,對照 README 規格(#10)' (#19) from fix/issue10-exitcode-tests into main 2026-09-10 10:20:23 +08:00
chenyunda218 f481377b9a 測試改斷言具體結束碼數值,對照 README 規格(#10)
cli_test、dailyops_commands_test、login_test、exitcode_test 原以
int(ExitUsage)/int(ExitAPI)/int(ExitInternal) 常數互比,常數若再
與 README 脫鉤測試仍會綠。改為直接斷言 0/2/3(內部錯誤 1)。

常數本體已隨 #5(6951817)改為明確數值,未知命令實測回 2。
2026-09-10 10:12:35 +08:00
4 changed files with 34 additions and 23 deletions
+4 -2
View File
@@ -51,8 +51,10 @@ func TestRunVersion(t *testing.T) {
func TestRunUnknownCommand(t *testing.T) {
_, stderr, code := run("no-such-command")
if code != int(ExitUsage) {
t.Errorf("未知命令應回 ExitUsage(%d),得到 %d", int(ExitUsage), code)
// 直接對照 README「輸出與結束碼」的數值(用法錯誤=2),
// 不以常數互比,避免常數與規格再次脫鉤(#10)。
if code != 2 {
t.Errorf("未知命令應回 2(README 用法錯誤),得到 %d", code)
}
if !strings.Contains(stderr, "unknown command") {
t.Errorf("未知命令應在 stderr 說明,得到:%q", stderr)
+8 -5
View File
@@ -53,8 +53,9 @@ func TestIssuesListRequiresRepo(t *testing.T) {
t.Errorf("不應發出任何請求")
})
_, stderr, code := run("issues", "list")
if code != int(ExitUsage) {
t.Errorf("缺 --repo 應回 ExitUsage,得到 %d", code)
// 對照 README「輸出與結束碼」:用法錯誤=2(#10)。
if code != 2 {
t.Errorf("缺 --repo 應回 2(README 用法錯誤),得到 %d", code)
}
if !strings.Contains(stderr, "--repo") {
t.Errorf("錯誤訊息應提及 --repo,得到:%s", stderr)
@@ -110,8 +111,9 @@ func TestWriteCommandsRequireYes(t *testing.T) {
t.Errorf("%s:未帶 --yes 不得發出請求(%s %s)", tc.name, r.Method, r.URL.Path)
})
_, stderr, code := run(tc.args...)
if code != int(ExitUsage) {
t.Errorf("%s 未帶 --yes 應回 ExitUsage(2),得到 %d", tc.name, code)
// 對照 README「輸出與結束碼」:用法錯誤=2(#10)。
if code != 2 {
t.Errorf("%s 未帶 --yes 應回 2(README 用法錯誤),得到 %d", tc.name, code)
}
if !strings.Contains(stderr, "--yes") {
t.Errorf("%s 錯誤訊息應提及 --yes,得到:%s", tc.name, stderr)
@@ -235,7 +237,8 @@ func TestAPIDirectPassthrough(t *testing.T) {
}
_, stderr, code := run("api", "--method", "POST", "/version")
if code != int(ExitUsage) || !strings.Contains(stderr, "--yes") {
// 對照 README「輸出與結束碼」:用法錯誤=2(#10)。
if code != 2 || !strings.Contains(stderr, "--yes") {
t.Errorf("POST 未帶 --yes 應回 2 並提及 --yes,得到 %d:%s", code, stderr)
}
}
+10 -8
View File
@@ -10,20 +10,22 @@ import (
)
func TestExitCodeMapping(t *testing.T) {
// want 直接寫 README「輸出與結束碼」的數值(0/2/3;內部錯誤 1),
// 不以常數互比,避免常數與規格再次脫鉤(#10)。
cases := []struct {
name string
err error
want ExitCode
want int
}{
{"usage error", &ErrUsage{Msg: "bad flag"}, ExitUsage},
{"api error 401", &gitea.ErrAPI{StatusCode: 401, Op: "GET /api/v1/user"}, ExitAPI},
{"api error transport", &gitea.ErrAPI{Op: "GET /api/v1/user", Err: errString("dial tcp")}, ExitAPI},
{"wrapped api error", wrapped(&gitea.ErrAPI{StatusCode: 500}), ExitAPI},
{"other error", errString("boom"), ExitInternal},
{"nil maps to OK", nil, ExitOK},
{"usage error", &ErrUsage{Msg: "bad flag"}, 2},
{"api error 401", &gitea.ErrAPI{StatusCode: 401, Op: "GET /api/v1/user"}, 3},
{"api error transport", &gitea.ErrAPI{Op: "GET /api/v1/user", Err: errString("dial tcp")}, 3},
{"wrapped api error", wrapped(&gitea.ErrAPI{StatusCode: 500}), 3},
{"other error", errString("boom"), 1},
{"nil maps to OK", nil, 0},
}
for _, tc := range cases {
if got := exitCodeFor(tc.err); got != tc.want {
if got := int(exitCodeFor(tc.err)); got != tc.want {
t.Errorf("%s: exitCodeFor = %d, want %d", tc.name, got, tc.want)
}
}
+12 -8
View File
@@ -145,8 +145,9 @@ func TestLoginAddRejected401NoWrite(t *testing.T) {
`)
_, stderr, code := run(append(prefix,
"--url", srv.URL, "login", "add", "--token", "bad-tok")...)
if code != int(ExitAPI) {
t.Fatalf("401 should exit 3, got %d (stderr %s)", code, stderr)
// 401 是 API 錯誤;對照 README「輸出與結束碼」=3(#10)。
if code != 3 {
t.Fatalf("401 should exit 3 (README api error), got %d (stderr %s)", code, stderr)
}
if !strings.Contains(stderr, "401") {
t.Fatalf("stderr should mention 401: %s", stderr)
@@ -250,8 +251,9 @@ func TestLoginRemoveAsksConfirmation(t *testing.T) {
func TestLoginUnknownSubcommandUsage(t *testing.T) {
prefix, _ := loginTestEnv(t, "")
_, stderr, code := run(append(prefix, "login", "bogus")...)
if code != int(ExitUsage) {
t.Fatalf("want exit 2, got %d", code)
// 對照 README「輸出與結束碼」:用法錯誤=2(#10)。
if code != 2 {
t.Fatalf("want exit 2 (README usage), got %d", code)
}
if !strings.Contains(stderr, "unknown login subcommand") {
t.Fatalf("stderr: %s", stderr)
@@ -261,8 +263,9 @@ func TestLoginUnknownSubcommandUsage(t *testing.T) {
func TestLoginNoSubcommandUsage(t *testing.T) {
prefix, _ := loginTestEnv(t, "")
_, stderr, code := run(append(prefix, "login")...)
if code != int(ExitUsage) {
t.Fatalf("want exit 2, got %d", code)
// 對照 README「輸出與結束碼」:用法錯誤=2(#10)。
if code != 2 {
t.Fatalf("want exit 2 (README usage), got %d", code)
}
if !strings.Contains(stderr, "subcommand") {
t.Fatalf("stderr: %s", stderr)
@@ -272,8 +275,9 @@ func TestLoginNoSubcommandUsage(t *testing.T) {
func TestLoginRemoveNotFound(t *testing.T) {
prefix, _ := loginTestEnv(t, "logins: []\n")
_, stderr, code := run(append(prefix, "login", "remove", "--yes", "ghost")...)
if code != int(ExitInternal) {
t.Fatalf("want exit 1, got %d", code)
// ExitInternal 不在 README 保證範圍;此處固定為 1(內部錯誤,#10)。
if code != 1 {
t.Fatalf("want exit 1 (internal), got %d", code)
}
if !strings.Contains(stderr, `login "ghost" not found`) {
t.Fatalf("stderr: %s", stderr)