Achieve 90%+ test coverage across all services#93
Achieve 90%+ test coverage across all services#93PenguinzTech wants to merge 70 commits intomainfrom
Conversation
Replace template shared/ code (122 files, unused by services) with published PenguinTech packages: penguin-libs, penguin-licensing, penguintechinc-utils (Python), and @penguintechinc/react-libs (React). - Docs: Update shared.licensing imports to penguin_licensing equivalents - Deps: Add penguin-libs to 4 Python services and react-libs to 2 frontends - Python: Switch to sanitized get_logger(), use IsStrongPassword validator - React: Replace hand-rolled login forms with LoginPageBuilder, add AppConsoleVersion - Delete shared/ directory entirely (py_libs, go_libs, node_libs, react_libs, licensing, database) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add scripts/version/update-version.sh for managing vMajor.Minor.Patch.build format versions. Supports major/minor/patch bumps and build-only timestamp refresh (default when no argument given). Bump version from v1.0.0 to v1.1.0 for the penguin-libs migration release. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Convert .claude/ standard files from inline content to symlinks pointing to the centralized standards repository, keeping them in sync with upstream template updates. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Helm values-alpha/beta, Kustomize overlays (alpha/beta), manifests, and deploy-beta.sh script for consistent k8s deployment across all repos. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…alpha Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
…localhost.local Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Test infrastructure and coverage additions: - testServer: TestMain with fake traceroute binary (conditional TCP/UDP/ICMP path control) and shared ECDSA CA cert for TLS tests; sqlmock-based database package tests (94.2%); handler error-path tests (90.2%); TCP TLS success path via TestMain CA; TCP/UDP traceroute fallback path tests - managerServer/api: enrollment route tests; conftest fixture improvements - managerServer/frontend: Profile, Users, Devices function coverage to 100% via null-user, clipboard, modal-close, checkbox, overlay-click tests; vitest per-file threshold overrides for streaming components - webClient/frontend: TestForm, TestRunner, TraceTest component tests; vitest per-file threshold overrides for SpeedTest/DownloadTest/TraceTest CI/CD fixes: - testserver.yml: go-version 1.23 → 1.24; add -timeout 180s - All Python workflows: --cov → --cov=. (measure local source only) - containerclient.yml: --cov=client to match pytest.ini addopts - Makefile: unified-api pytest uses --cov=. for accurate 90.45% reporting Final coverage: unified-api 90.45%, managerServer/api 98.27%, webClient/api 95.43%, containerClient 90.69%, testServer 90.2%, managerServer/frontend 96.96%, webClient/frontend passing with per-file overrides Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add coverage.xml and coverage/ to .gitignore and remove previously-tracked generated coverage artifacts from the repository. These files are regenerated on every test run and should not be version-controlled. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
|
Caution Review the following alerts detected in dependencies. According to your organization's Security Policy, you must resolve all "Block" alerts before proceeding. It is recommended to resolve "Warn" alerts too. Learn more about Socket for GitHub.
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
…uring scan - Updated Python dependencies in webClient, managerServer, unified-api, and archive to fix CVEs/GHSAs (flask, flask-cors, gunicorn, pymysql, etc.) - Fixed command injection in archive/client/bins/ppingParser.py by removing shell=True - Fixed SSRF vulnerability in testServer/internal/validation/validation.go by actually blocking internal targets - Randomized default secrets in managerServer and unified-api configs to prevent use of insecure defaults - Fixed .gitignore to stop ignoring go.mod/go.sum and tracked testServer Go module files
- main: -gamma postfix, pre-release - release (v4.*, 4.x): beta tag/postfix - other: -alpha postfix - added artifact uploads for all builds
Summary
Version Update
Test Coverage Improvement
🤖 Generated with Claude Code