Skip to content

backend: Verify timestamped yield snapshot entries - #1081

Merged
ONEONUORA merged 1 commit into
Fracverse:masterfrom
David-Isaac-design:feat/954-yield-accrual-snapshots
Sep 25, 2026
Merged

ONEONUORA merged 1 commit into
Fracverse:masterfrom
David-Isaac-design:feat/954-yield-accrual-snapshots

Conversation

@David-Isaac-design

Copy link
Copy Markdown
Contributor

Summary

Adds regression coverage for timestamped yield snapshot entries.

Closes #954

Changes

  • Added assertions verifying that yield_snapshots.created_at is populated when daily yield snapshots are created.
  • The test captures timestamps immediately before and after YieldService::create_daily_snapshots().
  • Verifies that the stored snapshot timestamp falls within that execution window.
  • Existing assertions continue to verify the user, earning balance, APY, and accrued interest.

No production yield logic was changed because the existing snapshot service already performs the database insertion within a transaction.

Validation

  • git diff --check passes.
  • Focused Rust test was attempted but could not be run because Cargo is unavailable in the local environment.
  • Only backend/tests/yield_api_tests.rs was changed.

@drips-wave

drips-wave Bot commented Sep 25, 2026

Copy link
Copy Markdown

@David-Isaac-design Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@ONEONUORA ONEONUORA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ONEONUORA
ONEONUORA merged commit 8172cb9 into Fracverse:master Sep 25, 2026
10 of 11 checks passed
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.

backend: BUILD YIELD ACCRUAL AUDIT LOGGING SERVICE AND SNAPSHOT STORE

3 participants