Skip to content

Add unit tests for heartbeat module#21

Merged
NickStr11 merged 1 commit intomainfrom
add-heartbeat-tests-17813869763445344861
Feb 24, 2026
Merged

Add unit tests for heartbeat module#21
NickStr11 merged 1 commit intomainfrom
add-heartbeat-tests-17813869763445344861

Conversation

@NickStr11
Copy link
Copy Markdown
Owner

Added comprehensive unit tests for the tools/heartbeat/ module. The test suite covers all major components including data fetching from HN, GitHub, and Reddit (with mocks), formatting, project context reading, and CLI argument parsing. Achieved 97% code coverage, significantly exceeding the 80% target. All new test files follow the project's strict typing and beartype requirements.

Fixes #20


PR created automatically by Jules for task 17813869763445344861 started by @NickStr11

- Created tests/ directory with __init__.py
- Added pytest and pytest-cov to dev-dependencies
- Added tests for config.py, sources.py, formatter.py, analyzer.py, and main.py
- Mocked HTTP requests and external APIs
- Achieved 97% code coverage
- Followed project coding standards (beartype, future annotations)

Co-authored-by: NickStr11 <37849063+NickStr11@users.noreply.github.com>
@google-labs-jules
Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@NickStr11 NickStr11 merged commit 8fd55a1 into main Feb 24, 2026
1 check passed
NickStr11 pushed a commit that referenced this pull request Feb 25, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@NickStr11 NickStr11 deleted the add-heartbeat-tests-17813869763445344861 branch March 5, 2026 04:04
NickStr11 pushed a commit that referenced this pull request Mar 16, 2026
- docs/incidents.md — structured incident log with format template + 4 retroactive entries
- .claude/skills/eval/SKILL.md — /eval post-session self-evaluation (4 axes: Reasoning/Rules/Code/Efficiency)
- .claude/hooks/incident-detector.py — PostToolUse hook: 2+ consecutive Bash failures → RCA reminder
- .claude/settings.json — registered incident-detector hook

Inspired by Founder OS #21 (Aleksandr Vasiliev): eval + incidents + design injection feedback loop

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

test(heartbeat): add tests for heartbeat module

1 participant