Skip to content

more tests for session.py #146

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jun 25, 2025
Merged

more tests for session.py #146

merged 1 commit into from
Jun 25, 2025

Conversation

yangm2
Copy link
Contributor

@yangm2 yangm2 commented Jun 24, 2025 •
edited
Loading

  • fix missing/broken test
  • add another test to hit missing line

uv run pytest --cov tenantfirstaid ...

Name                          Stmts   Miss  Cover
-------------------------------------------------
tenantfirstaid/__init__.py        0      0   100%
tenantfirstaid/app.py            29      5    83%
tenantfirstaid/chat.py           55     39    29%
tenantfirstaid/citations.py      14      0   100%
tenantfirstaid/session.py        41      0   100%
tenantfirstaid/shared.py         11      0   100%
-------------------------------------------------
TOTAL                           150     44    71%

uv run pytest --cov tenantfirstaid --cov-branch

Name                          Stmts   Miss Branch BrPart  Cover
---------------------------------------------------------------
tenantfirstaid/__init__.py        0      0      0      0   100%
tenantfirstaid/app.py            29      5      4      2    79%
tenantfirstaid/chat.py           55     39      8      0    25%
tenantfirstaid/citations.py      14      0      4      0   100%
tenantfirstaid/session.py        41      0      6      1    98%
tenantfirstaid/shared.py         11      0      2      1    92%
---------------------------------------------------------------
TOTAL                           150     44     24      4    68%

@yangm2 yangm2 self-assigned this Jun 24, 2025
@yangm2 yangm2 added the backend Bot implementation and other backend concerns label Jun 24, 2025
@yangm2 yangm2 linked an issue Jun 24, 2025 that may be closed by this pull request
Backend unit tests #68
Closed
@yangm2 yangm2 merged commit 9268462 into codeforpdx:main Jun 25, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KentShikama KentShikama KentShikama approved these changes

@wittejm wittejm Awaiting requested review from wittejm

@apkostka apkostka Awaiting requested review from apkostka

Assignees

@yangm2 yangm2

Labels
backend Bot implementation and other backend concerns
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Backend unit tests
2 participants
@yangm2 @KentShikama