nagayon-935 / mping

81%

main: 81%

LAST BUILD ON BRANCH fix/code-review-issues

branch: fix/code-review-issues

CHANGE BRANCH

x

Reset

• fix/code-review-issues
• add_copilot_reviewer
• add_function_port_check
• docs/readme-english
• feature/configurable-thresholds
• feature/json-export-and-yaml-watch
• feature/linux-support
• feature/mtr-monitor
• feature/phase1-tcp-udp-latency
• feature/phase2-geoip-asn
• feature/phase3-route-flap
• feature/phase4-http-healthcheck
• feature/phase5-yaml-groups
• feature/phase6-dynamic-hosts
• feature/phase7-paris-traceroute
• feature/pmtu-hop-detection
• feature/port-monitor-improvements
• feature/refactor-cleanup
• feature/traceroute-hops-initttl
• file_option
• fix-ui-layout-duplication
• fix/go-review-issues
• fix/stability-issues
• fix/ui-rendering-issues
• main
• refactor/code-cleanup
• refactor/go-review-fixes
• refactor/hardcoded-constants
• refactor/improvements

Build # 27549112569

Build Type

Pull #35

github

Committed by nagayon-935

Commit Message

test: restore cmd/main to 80%+ and cover new code from review fixes

- appendErrorLogRaw: direct test verifying slice append and writer write
- httpCodeColorTag: table-driven test covering all five branches (0/2xx/3xx/4xx/5xx)
- renderHTTPMonitorTable status-change path: exercises the log writer
  and lastStatuses update when a URL transitions from Down→Up
- setupHTTPChecker: nil-URL (returns nil) and non-nil path
  (starts and stops a real checker against a local TCP listener)
- setupLogger empty path, invalid path error
- writeJSONSnapshot write error path (non-existent parent directory)
- printExitSummary with recv stats: covers the rtt min/avg/max branch

cmd/main: 79.5% → 80.5%  internal/ui: 89.3% → 90.5%

Pull Request Pull Request #35: fix: address all code review issues (HIGH + MEDIUM)

Coverage Stats

46 of 81 new or added lines in 10 files covered. (56.79%)

2 existing lines in 1 file now uncovered.

4341 of 5344 relevant lines covered (81.23%)

0.95 hits per line