Skip to content

soroban-expiry.service.spec.ts tests an implementation that no longer exists #185

Description

@martinzhames

Category: testing

Problem

src/blockchain-wallet/soroban-expiry.service.spec.ts exercises setCurrentLedger, createPayment, confirm, getPayment, expirePayment, isExpired, advanceLedger, getExpiredEventLog, and static LEDGERS_PER_MINUTE/DEFAULT_EXPIRY_LEDGERS on SorobanService — none of which exist on the current class in src/blockchain-wallet/soroban.service.ts (which only has pause/unpause/deposit/release/refund/registerUser/getBalance/getStakeBalance).

Impact

This entire spec file (the ledger-based-expiry contract-mirroring test suite, ~17 tests) is orphaned and cannot pass — it appears to target an implementation that was reverted or overwritten, and pnpm test presumably fails or the file is silently excluded, either way masking a real regression.

Suggested fix

Reconcile with the SorobanService duplicate-export issue above; once one true implementation is settled on, update or delete this spec to match it.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Stellar WaveIssues in the Stellar wave program

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions