Skip to content

CIP112: Fix custom-token coverage gate, add missing README entries and add missing tests - #57

Open
0xNeshi wants to merge 9 commits into
igingu-cip112-simplification-and-radfrom
0xneshi/cip112-fixes
Open

0xNeshi wants to merge 9 commits into
igingu-cip112-simplification-and-radfrom
0xneshi/cip112-fixes

Conversation

@0xNeshi

@0xNeshi 0xNeshi commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

Related #45

  • Address failing coverage job. Merging main added examples/ to scripts/check-coverage.sh, so the coverage job now checks the custom-token example. The job failed because the example's tests never exercised 8 production choices in MyApp.Token.
  • tokenCIP112-v1 README. The README now says when a transfer completes in one step and when the receiver has to accept a pending instruction.
  • add missing test cases

@0xNeshi 0xNeshi self-assigned this Oct 8, 2026
@0xNeshi 0xNeshi changed the title Pin deadline-free transfer withdraw with a test and document pending-instruction recovery CIP112: Pin deadline-free transfer withdraw with a test and document pending-instruction recovery Oct 8, 2026
@0xNeshi 0xNeshi changed the title CIP112: Pin deadline-free transfer withdraw with a test and document pending-instruction recovery CIP112: Fix custom-token coverage gate, add missing README entries and add missing tests Oct 8, 2026
-> AllocationV2.SettlementFactory_SettleBatch
-> Update AllocationV2.SettlementFactory_SettleBatchResult
settleBatchImpl pin admin self arg = do
checkActors arg.actors [arg.settlement.executors]

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This check is already performed in settlementFactoryV2_settleBatchDefaultImpl a couple of lines later in this function

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.

1 participant