-
Notifications
You must be signed in to change notification settings - Fork 372
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: write tests for file backed db #1230
base: main
Are you sure you want to change the base?
Conversation
Tests already detected some problems with btree |
UPDATE also specifically overwrites the cell (usually), so it would be valuable to add some tests like:
|
Oh that is good to test. I will add this. I'm also adding the UV workspace stuff to facilitate dependency management for our Python code. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff here
c1574d8
to
c2c34d3
Compare
First attempt at closing #1212. Also with this PR, I added the option of using
with
syntax forTestLimboShell
. It automatically closes the shell on error, and facilitates error handling overall. If this is merged, I can update the other python tests to usewith
as well.