Skip to content

feat: add pending airdrop record unit test - #1760

Merged
Dosik13 merged 1 commit into
hiero-ledger:mainfrom
Adityarya11:fix/par-unit-test#1749
Jul 6, 2026
Merged

feat: add pending airdrop record unit test#1760
Dosik13 merged 1 commit into
hiero-ledger:mainfrom
Adityarya11:fix/par-unit-test#1749

Conversation

@Adityarya11

Copy link
Copy Markdown
Contributor

Description:

Add unit test coverage for PendingAirdropRecord, which previously had no dedicated test file.

  • Add sdk/pending_airdrop_record_unit_test.go
  • Add proto round-trip test covering the fungible token case
  • Add String() output assertion

Related issue(s):

Fixes #1749

Notes for reviewer:

$ go test ./sdk -tags="unit" -v -run TestUnitPendingAirdropRecord -timeout 9999s
PASS

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Aditya Arya <arya050411@gmail.com>
@Adityarya11
Adityarya11 requested review from a team as code owners June 30, 2026 13:09
@Adityarya11
Adityarya11 requested review from Dosik13 and gsstoykov June 30, 2026 13:09
@lfdt-bot

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Dosik13 Dosik13 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.

lgtm! thank you for the contribution!

@Dosik13
Dosik13 merged commit 6f6c6df into hiero-ledger:main Jul 6, 2026
13 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.

[Good First Issue]: Add unit tests for PendingAirdropRecord proto round-trip

3 participants