econia-labs / emojicoin-dot-fun

https://emojicoin-dot-fun.vercel.app
Other
6 stars 13 forks source link

[ECO-2420] Refactor claim link module for ease of DevOps #360

Closed alnoki closed 1 week ago

alnoki commented 1 week ago

Description

  1. Implement add_public_keys_and_fund_gas_escrows API with 100% coverage testing.
  2. Update vector-wise for_each to for_each_ref since the latter does not have to reverse the vector in place before iteration.
  3. Split public keys into claimed and eligible tables for ease of fetching eligible public keys.
  4. Update assorted APIs as needed for claimed and eligible table paradigm.

Testing

emojicoin_dot_fun_claim_link module tested to 100% coverage. From in repo root:

cd src/move/rewards
aptos move test --dev --move-2 --coverage

Note that per https://github.com/aptos-labs/aptos-core/issues/15284 the test_general_flow test fails non-deterministically. If it fails, running it again will probably result in a pass, eventually.

Checklist

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
emojicoin-dot-fun ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 17, 2024 4:35am
emojicoin-dot-fun-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 17, 2024 4:35am