Skip to content

fix: stabilize Windows build and tests, add HTML test report - #22

Merged
qdriven merged 1 commit into
mainfrom
qdriven/windows-build
Oct 20, 2026
Merged

qdriven merged 1 commit into
mainfrom
qdriven/windows-build

Conversation

@qdriven

@qdriven qdriven commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Track go.sum (fresh clones failed to build) + task build always reinstalls (fixes 构建稳定的Windows Build脚本 #21)
  • Root-cause Windows test failures: submodule paths normalized to forward slashes; nonexistent-PID test uses math.MaxInt32 (PID 0 is System Idle on Windows); accept ginkgo .test binary suffix
  • Add HTML test report: make test-report / task test-report -> reports/test-report.html (internal/testreport renderer + tools/testreport CLI, BDD tested)

Test plan

  • task test (go test ./...) all green
  • task test-report (16 suites, 44 specs) green, HTML report generated
  • go vet clean
  • task build + spark version works

Build:
- Track go.sum (was gitignored; fresh clones failed to build) (#21)
- task build: drop sources/generates fingerprinting so reinstall always runs

Windows test fixes (root cause, not deletion):
- submodule add: normalize git-facing paths with filepath.ToSlash
  (update-index --cacheinfo rejects backslashes; rm/ls-files silently
  mismatch them)
- ReadProcess nonexistent-PID test: use math.MaxInt32 (PID 0 is the
  readable System Idle Process on Windows)
- cmdline self test: also accept ginkgo CLI's .test binary suffix

HTML test report:
- internal/testreport: Ginkgo types.Report -> self-contained HTML
  renderer (summary cards, suite/spec tables, failure details, state
  filter + text search) with BDD tests
- tools/testreport: CLI converting ginkgo --json-report JSON to HTML
- make test-report / task test-report -> reports/test-report.html
- Work around ginkgo path.Dir bug on Windows via --output-dir
@qdriven
qdriven merged commit 3eaad1f into main Oct 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

构建稳定的Windows Build脚本

1 participant