[Priyanshubhartistm]

Description

Adds unit specs for the three repositories that had none: notification-outbox, notification-delivery-log and nip05-verification. No production code changes.

They run the repositories' real knex queries on the postgres dialect through a small recording client (test/unit/repositories/recording-knex.ts) that never connects: each statement and its bindings are recorded and answered with scripted rows or counts, so the specs assert on the SQL that would run, not on stubbed repository methods.

• Outbox: enqueue (pending, due now, unattempted), claimBatch (due pending plus processing rows whose 5 minute lease ran out, oldest first, for update skip locked, claimed rows marked processing), markDelivered, markFailed (back to pending with exponential backoff capped at a minute, dead once attempts are used up, error truncated to 2000 characters), deleteTerminalOlderThan (delivered and dead only).
• Delivery log: append for success, failure and test notifications, findSuccessfulTargetIds (successes only, so a retry skips delivered targets), findRecent (default limit and each filter), deleteOlderThan.
• NIP-05: findByPubkey, upsert (what is updated on conflict), findPendingVerifications, deleteByPubkey, and listForAdmin (each filter, stale and fresh, the filters combined, count and page filtered alike, pagination).

Related Issue

Closes #905

Motivation and Context

The notification outbox service spec stubs its repository, so the SQL of the repository was never run by a test, and listForAdmin already serves the admin API untested.

claimBatch is asserted on its query (locking and skipping locked rows); that two workers never receive the same message is guaranteed by PostgreSQL, so it is not something a unit test can show.

How Has This Been Tested?

• The new specs pass (50 tests). Removing skipLocked, changing the backoff cap or the dead threshold, dropping the success filter, the null check of the stale filter or the clone of the count query each make a spec fail.
• pnpm lint, pnpm check:deps, pnpm run build:check, pnpm run test:unit.

Screenshots (if appropriate):

Types of changes

• Non-functional change (docs, style, minor refactor)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my code changes.
• I added a changeset, or this is docs-only and I added an empty changeset.
• All new and existing tests passed.

[changeset-bot]

🦋 Changeset detected

Latest commit: cc23343

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[greptile-apps]

Confidence Score: 5/5

[Low impact] The PR appears safe to merge; no blocking issues were found.

Summary

Adds SQL-focused unit tests for the notification outbox, delivery log, and NIP-05 verification repositories without changing production code.

• Outbox specs check how messages move through delivery.
• Delivery-log specs check how notification attempts are recorded and found.
• NIP-05 specs check verification lookups, updates, and admin lists.

_{Reviews (4) · Last reviewed commit: "Merge branch 'main' into test/repository..." · Reviewed by Greptile}

[coveralls]

Coverage Report for CI Build 40101217063

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Warning

No base build found for commit 062ee00 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 76.379%

Details

• Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →

---

Coverage Stats

Relevant Lines:	11578
Covered Lines:	9250
Line Coverage:	79.89%
Relevant Branches:	5983
Covered Branches:	4163
Branch Coverage:	69.58%
Branches in Coverage %:	Yes
Coverage Strength:	786.33 hits per line

---

💛 - Coveralls