Skip to content
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.

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

Test and fix debug mode #2481

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Test and fix debug mode #2481

wants to merge 4 commits into from

Conversation

ADBond
Copy link
Contributor

@ADBond ADBond commented Oct 22, 2024

Some tests of various bits of functionality with debug_mode switched on. Tests are structured so that we can check that issues specifically arise only when debug_mode is on - if the test fails when it is off, a different error will be flagged, to help isolate things. These can be expanded to help diagnose issues, but just as an initial starting point.

This will also include fixes for the failing tests, which will be opened as separate PRs into this branch.

Partial work towards #2429. This tackles 'does stuff work when debug_mode is switched on, but doesn't address 'is debug_mode doing what we think it is'.

@ADBond ADBond added bug Something isn't working testing debug_mode labels Oct 22, 2024
@ADBond
Copy link
Contributor Author

ADBond commented Oct 22, 2024

Oddly the tests that fail are passing in sqlite - makes sense that the tests are okay in sqlite - no salting, and sampling uses absolute number rather than proportion.

Really it's slightly odd that the postgres tests fail - but looks like that is a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working debug_mode testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant