Skip to content

Integration and regression tests#38

Open
stoutta6 wants to merge 4 commits intomainfrom
tas/add-tests
Open

Integration and regression tests#38
stoutta6 wants to merge 4 commits intomainfrom
tas/add-tests

Conversation

@stoutta6
Copy link
Collaborator

Adding tests of batbot preprocess CLI with and without fast mode processing. Includes regression test on the example wav files.

@stoutta6
Copy link
Collaborator Author

stoutta6 commented Feb 24, 2026

@bluemellophone The new tests are failing to process example3.wav and example4.wav but not 1 or 2. Any idea why that might be?

I just tried a fresh environment install on my computer and it passes the tests.

edit: Tests also pass within the docker container.

@bluemellophone
Copy link
Collaborator

Following up, see here

@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.6%. Comparing base (c398120) to head (30ccf91).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##            main     #38      +/-   ##
========================================
+ Coverage   57.4%   70.6%   +13.2%     
========================================
  Files          7       9       +2     
  Lines       1317    1354      +37     
========================================
+ Hits         756     956     +200     
+ Misses       561     398     -163     
Files with missing lines Coverage Δ
batbot/batbot_cli.py 59.0% <100.0%> (+59.0%) ⬆️
tests/test_preprocess.py 100.0% <100.0%> (ø)
tests/test_preprocess_parallel.py 100.0% <100.0%> (ø)

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c398120...30ccf91. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stoutta6
Copy link
Collaborator Author

@bluemellophone Should be ready for your review now

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.

2 participants