Skip to content

Comments

Report Generation: Adding unit tests#64

Open
lotif wants to merge 4 commits intomainfrom
marcelo/tests
Open

Report Generation: Adding unit tests#64
lotif wants to merge 4 commits intomainfrom
marcelo/tests

Conversation

@lotif
Copy link
Collaborator

@lotif lotif commented Feb 20, 2026

Summary

Adding unit tests for the report generation modules.

Also fixing some minor bugs that were found while writing the tests.

Clickup Ticket(s): NA

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test improvements
  • 🔒 Security fix

Changes Made

Adding unit tests for the report generation modules.
Fixing some minor bugs that were found while writing the tests.

Testing

  • Tests pass locally (uv run pytest tests/)
  • Type checking passes (uv run mypy <src_dir>)
  • Linting passes (uv run ruff check src_dir/)
  • Manual testing performed (describe below)

Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Documentation updated (if applicable)
  • No sensitive information (API keys, credentials) exposed

@lotif lotif requested review from amrit110 and fcogidi February 20, 2026 18:47
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.

1 participant