veeso / ic-dbms / 21729219694

95%

main: 96%

push

github

refactor(canister): clean up dbms.rs with multiple improvements (#40)

* refactor(canister): clean up dbms.rs with multiple improvements

- Extract duplicated record collection pattern from update/delete into
  collect_matching_records helper method
- Fix with_transaction to use immutable borrow (get_transaction)
  instead of unnecessarily getting a mutable reference
- Extract values_to_schema_entity as a standalone module-level function
  instead of an inner function inside update
- Wrap insert oneshot path in atomic for consistency with update/delete
- Hoist PK lookup outside inner loop in delete_foreign_keys_cascade
- Use pks.len() for update transaction count instead of re-querying
- Select only PK column in existing_primary_keys_for_filter
- Re-export NextRecord from memory module for crate-wide use

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* refactor: Moved dbms.rs tests to a separate file to cleanup

47 of 48 new or added lines in 1 file covered. (97.92%)

8948 of 9392 relevant lines covered (95.27%)

799397.79 hits per line