Skip to content

add more tests#2291

Merged
tevko merged 73 commits into
edgefrom
te-delphi-tests-2
Nov 19, 2025
Merged

add more tests#2291
tevko merged 73 commits into
edgefrom
te-delphi-tests-2

Conversation

@tevko
Copy link
Copy Markdown
Collaborator

@tevko tevko commented Nov 14, 2025

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 14, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
4077 0 0% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 73832ba by action🐍

@jucor
Copy link
Copy Markdown
Collaborator

jucor commented Nov 18, 2025

Hi @tevko !

Flagging that we might have conflicts in the tests/ folder, especially around the clojure and the edge-case tests this PR adds, with a lot of the clarification I have in #2282 where I clarified the role of "legacy regression comparing to python" and "proper regression of python version against itself using a golden record".
At best there'll be redundancy, at worst there'll be conflicts.
(You'll see in that other PR: i moved a lot of tests around, removed duplicates, separated concerns, etc)

@github-actions
Copy link
Copy Markdown

Coverage Report

File Stmts Miss Cover
init.py 3 0 100%
main.py 55 55 0%
components/init.py 2 0 100%
components/config.py 165 133 19%
components/server.py 116 72 38%
conversation/init.py 2 0 100%
conversation/conversation.py 1035 403 61%
conversation/manager.py 131 42 68%
database/init.py 1 0 100%
database/dynamodb.py 387 234 40%
database/postgres.py 298 199 33%
pca_kmeans_rep/init.py 6 0 100%
pca_kmeans_rep/clusters.py 235 8 97%
pca_kmeans_rep/corr.py 101 17 83%
pca_kmeans_rep/named_matrix.py 257 25 90%
pca_kmeans_rep/pca.py 272 76 72%
pca_kmeans_rep/repness.py 274 50 82%
pca_kmeans_rep/stats.py 107 22 79%
poller.py 224 188 16%
run_math_pipeline.py 259 114 56%
system.py 85 55 35%
utils/general.py 62 40 35%
Total 4077 1733 57%

@tevko tevko marked this pull request as ready for review November 19, 2025 03:09
@github-actions
Copy link
Copy Markdown

Coverage Report

File Stmts Miss Cover
init.py 3 0 100%
main.py 55 55 0%
components/init.py 2 0 100%
components/config.py 165 133 19%
components/server.py 116 72 38%
conversation/init.py 2 0 100%
conversation/conversation.py 1035 403 61%
conversation/manager.py 131 42 68%
database/init.py 1 0 100%
database/dynamodb.py 387 234 40%
database/postgres.py 298 199 33%
pca_kmeans_rep/init.py 6 0 100%
pca_kmeans_rep/clusters.py 235 8 97%
pca_kmeans_rep/corr.py 101 17 83%
pca_kmeans_rep/named_matrix.py 257 25 90%
pca_kmeans_rep/pca.py 272 76 72%
pca_kmeans_rep/repness.py 274 50 82%
pca_kmeans_rep/stats.py 107 22 79%
poller.py 224 188 16%
run_math_pipeline.py 259 114 56%
system.py 85 55 35%
utils/general.py 62 40 35%
Total 4077 1733 57%

Comment thread .github/workflows/python-ci.yml Outdated
echo '--- Installing Pytest ---'; \
pip install pytest; \
echo '--- Installing Pytest & Coverage tools ---'; \
pip install pytest pytest-cov coverage; \
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fine for now, but #2299 will update the Dockerfile to include dev libs in the test build

@github-actions
Copy link
Copy Markdown

Coverage Report

File Stmts Miss Cover
init.py 3 0 100%
main.py 55 55 0%
components/init.py 2 0 100%
components/config.py 165 133 19%
components/server.py 116 72 38%
conversation/init.py 2 0 100%
conversation/conversation.py 1035 403 61%
conversation/manager.py 131 42 68%
database/init.py 1 0 100%
database/dynamodb.py 387 234 40%
database/postgres.py 298 199 33%
pca_kmeans_rep/init.py 6 0 100%
pca_kmeans_rep/clusters.py 235 8 97%
pca_kmeans_rep/corr.py 101 17 83%
pca_kmeans_rep/named_matrix.py 257 25 90%
pca_kmeans_rep/pca.py 272 76 72%
pca_kmeans_rep/repness.py 274 50 82%
pca_kmeans_rep/stats.py 107 22 79%
poller.py 224 188 16%
run_math_pipeline.py 259 114 56%
system.py 85 55 35%
utils/general.py 62 40 35%
Total 4077 1733 57%

@tevko tevko merged commit d95ad13 into edge Nov 19, 2025
4 checks passed
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.

3 participants