From 0c558fb33bf5fa0904baecb750596757ea815a03 Mon Sep 17 00:00:00 2001 From: zarzet Date: Thu, 23 Jul 2026 09:56:22 +0700 Subject: [PATCH] test(backend): use a valid descriptor in fd coverage --- go_backend/misc_coverage_supplement_test.go | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/go_backend/misc_coverage_supplement_test.go b/go_backend/misc_coverage_supplement_test.go index b187f163..9d2d1268 100644 --- a/go_backend/misc_coverage_supplement_test.go +++ b/go_backend/misc_coverage_supplement_test.go @@ -61,15 +61,26 @@ func TestOutputFDFilePathBranches(t *testing.T) { if _, err := file.Write([]byte("data")); err != nil { t.Fatalf("write output: %v", err) } - file.Close() + if err := file.Close(); err != nil { + t.Fatalf("close output: %v", err) + } if !isFDOutput(1) || isFDOutput(0) { t.Fatal("isFDOutput mismatch") } closeOwnedOutputFD(0) - if err := prepareDupFDForWrite(11, 10); err != nil { + fdSource, err := os.OpenFile(outputPath, os.O_RDWR, 0600) + if err != nil { + t.Fatalf("open fd source: %v", err) + } + defer fdSource.Close() + dupFD, err := dupOutputFD(int(fdSource.Fd())) + if err != nil { + t.Fatalf("duplicate output fd: %v", err) + } + if err := prepareDupFDForWrite(dupFD, int(fdSource.Fd())); err != nil { t.Fatalf("prepareDupFDForWrite: %v", err) } - closeOwnedOutputFD(11) + closeOwnedOutputFD(dupFD) cleanupOutputOnError(outputPath, 0) if _, err := os.Stat(outputPath); !os.IsNotExist(err) { t.Fatalf("cleanup should remove output path, stat err=%v", err)