Skip to content
This repository has been archived by the owner on Feb 13, 2024. It is now read-only.

fix: removed feePurse from test #55

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tgrecojs
Copy link

@tgrecojs tgrecojs commented Feb 4, 2023

agoric init currently scaffolds out a project with broken tests. this is a result of #5997 during which support for makeFeePurse and bindDefaultFeePurse has been dropped.

test-contract.js - current (broken)

  const feePurse = E(zoeService).makeFeePurse();
  const zoe = E(zoeService).bindDefaultFeePurse(feePurse);

test-contract.js - proposed in this PR

  const { zoeService: zoe } = makeZoeKit(makeFakeVatAdmin().admin);

if this is approved, then the same change is needed in a handful of other example repositories

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant