Skip to content

Conversation

@ocean
Copy link
Owner

@ocean ocean commented Jan 19, 2026

Summary by CodeRabbit

Documentation

  • Updated documentation to clarify current capabilities and limitations with SQLite.

Tests

  • Removed obsolete test modules from the test suite to streamline the test infrastructure.
  • Updated test configuration to adjust which tests are excluded from running in both local and CI environments.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 19, 2026

Walkthrough

This pull request removes two entire test files related to replication and savepoint replication features, updates test exclusion logic to skip Turso remote tests, and removes a documentation note about RETURNING limitations with SQLite. No functional code changes are introduced.

Changes

Cohort / File(s) Summary
Documentation
USAGE.md
Removed a bullet point documenting that update_all() with a select clause and RETURNING is not well-supported with SQLite
Test deletions
test/replication_integration_test.exs, test/savepoint_replication_test.exs
Removed entire ExUnit test modules for replication and savepoint replication integration tests, including all test blocks, setup/teardown logic, and environment-based conditional connection handling (~767 lines total)
Test configuration
test/test_helper.exs
Updated test exclusion logic to also skip Turso remote tests on both CI and local environments; minor wording adjustments in comments

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 Tests take flight, away they go,
Replication frames no longer flow,
Savepoints swept like morning dew,
RETURNING's note bids us adieu,
Cleaner paths for what's brand new! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately summarises the main change: removal of unsupported replication tests across multiple test files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch replication-test-fixes

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ocean ocean merged commit 638a7ee into main Jan 19, 2026
14 checks passed
@ocean ocean deleted the replication-test-fixes branch January 19, 2026 04:09
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