Skip to content

[ENHANCEMENT] improves test coverage across codebase (helpers, execution lifecyle, SSE/exec error paths etc)#202

Merged
siddu-k merged 2 commits into
siddu-k:mainfrom
Ishita-190:tests
Jun 4, 2026
Merged

[ENHANCEMENT] improves test coverage across codebase (helpers, execution lifecyle, SSE/exec error paths etc)#202
siddu-k merged 2 commits into
siddu-k:mainfrom
Ishita-190:tests

Conversation

@Ishita-190
Copy link
Copy Markdown
Contributor

Fixes issue #117

Fixes this issue

Adds a set of unit tests covering JSONL helpers, execution lifecycle helpers, session parsing, command-history helpers, and some SSE/exec error paths.

Overall coverage increased from 54% to 70%.

Changes made

  • Added test_app_helpers.py : helpers, jsonl, reliability rebuild, session parsing, command history
  • Added test_execution_lifecycle.py : _cleanup_execution unit tests and exec timeout branch
  • Modified test_reliability.py , test_security.py , test_sse.py : improved coverage in these files as well

Testing and validation

python -m pytest --cov=. --cov-report=term-missing --cov-report=html
start htmlcov\index.html

Screenshots and logs

image

Checklist

  • No breaking changes
  • Tests added or updated
  • Documentation added or updated
  • Branch is up to date with main

@Ishita-190
Copy link
Copy Markdown
Contributor Author

@siddu-k please take a look at this!

@siddu-k siddu-k merged commit 7f87dad into siddu-k:main Jun 4, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants