Skip to content

Commit

Permalink
feat(contracts): Removed need of feesink template var.
Browse files Browse the repository at this point in the history
The feeSinkAddr template variable was removed as it was redundant. Refactored the logic to directly retrieve the feeSink address from the blockchain blocks. All relevant method invocations were updated accordingly.
Updated test to reference fee sink address that is default in devnet/localnet (A7NMWS3NT3IUDMLVO26ULGXGIIOUQ3ND2TXSER6EBGRZNOBOUIQXHIBGDE) which should match what the contracts use in saturated excess sends.
  • Loading branch information
pbennett committed Sep 14, 2024
1 parent 29167b6 commit f147a33
Show file tree
Hide file tree
Showing 18 changed files with 15,286 additions and 15,411 deletions.
3 changes: 1 addition & 2 deletions contracts/__test__/contracts.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ import {
ValidatorPoolKey,
} from '../helpers/helpers'

const FEE_SINK_ADDR = 'Y76M3MSY6DKBRHBL7C3NNDXGS5IIMQVQVUAB6MP4XEMMGVF2QWNPL226CA'
const FEE_SINK_ADDR = 'A7NMWS3NT3IUDMLVO26ULGXGIIOUQ3ND2TXSER6EBGRZNOBOUIQXHIBGDE'

const MaxPoolsPerNode = 3
// Periodically set this to max amount allowed in protocol (200 atm) but when testing more frequently this should be lowered to something like 20 stakers
Expand Down Expand Up @@ -77,7 +77,6 @@ beforeAll(async () => {
const { approvalCompiled } = await poolClient.appClient.compile({
deployTimeParams: {
nfdRegistryAppId: 0,
feeSinkAddr: decodeAddress(FEE_SINK_ADDR).publicKey,
},
})
validatorMasterClient = new ValidatorRegistryClient(
Expand Down
1 change: 0 additions & 1 deletion contracts/bootstrap/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,6 @@ async function main() {
id: 0,
deployTimeParams: {
nfdRegistryAppId: Number(registryAppID),
feeSinkAddr: decodeAddress(feeSink).publicKey,
},
},
algorand.client.algod,
Expand Down
Loading

0 comments on commit f147a33

Please sign in to comment.