Skip to content

Test repeated finite CouchDB connection ownership - #114

Merged
vrtmrz merged 2 commits into
mainfrom
fix/finite-remote-connection-ownership
Aug 14, 2026
Merged

Test repeated finite CouchDB connection ownership#114
vrtmrz merged 2 commits into
mainfrom
fix/finite-remote-connection-ownership

Conversation

@vrtmrz

@vrtmrz vrtmrz commented Aug 14, 2026

Copy link
Copy Markdown
Owner

This follow-up protects finite CouchDB connection ownership by formatting the merged implementation and covering repeated one-shot Security Seed refreshes.

Summary

  • Apply the repository's formatting to LiveSyncReplicator without changing its behaviour.
  • Exercise two repeated one-shot cycles, each of which creates a Security Seed connection and a replication connection.
  • Verify that every temporary remote database handle is closed exactly once.

Rationale

PR #112 fixed the production ownership issue. This follow-up keeps that implementation unchanged while protecting the repeated refresh boundary against regression.

Verification

  • vitest run src/replication/couchdb/LiveSyncReplicator.unit.spec.ts — 33 tests passed.
  • npm run check:types
  • Prettier check for the changed files

@vrtmrz
vrtmrz marked this pull request as ready for review August 14, 2026 04:33
@vrtmrz
vrtmrz merged commit 7e8282b into main Aug 14, 2026
3 checks passed
@vrtmrz vrtmrz mentioned this pull request Aug 14, 2026
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.

1 participant