Refactor tranche token #330

hieronx · 2024-06-23T18:36:29Z

No description provided.

…into clean-up-erc20

* Consistent mocks naming (#299) * rename mocks to consistent formatting * fix MockRestrictionManagerFactory --------- Co-authored-by: Jeroen <[email protected]> * update all relative imports to absolute imports (#297) Co-authored-by: Jeroen <[email protected]> * Remove trusted forwarder pattern (#294) * Remove trusted forwarder pattern * Update comment * Fix allowance * Change to callback * Change allowance * Fix tests * Update name * Set up 7575 share interface * Format * Remove unused transferFrom * Remove successCheck * Don't use address 0 for authTransferFrom * Fix comment * Readd transfer callback checks * Add operator logic (#298) * Add operator logic * Format * Cleanup * Add test * Update pragmas in interfaces * Update docs * Add docs * Add redeem test * Fix test * Clarify * Add modifier * Move owner check to internal method * ERC7575 updates (#302) * Replace delayed/pause admins with Guardian contract (#283) * Set up idea * plug in guardian everywhere, remove pauseAdmin, comment out broken tests * add safeMock.sol * add values_mapping_address_uint field to Mock.sol * fix some tests * fix tests * update scripts and fix more tests * update fork tests * update var name * fix tests * remove delayedAdmin * remove duplicate imports * Add IGuardian.sol and update Guardian.sol * fix tests * update Guardian.sol * Format --------- Co-authored-by: Jeroen Offerijns <[email protected]> Co-authored-by: Jeroen <[email protected]> * Remove `state.exists` and use pending Deposit and redeem requests instead (#296) * remove state.exists and use pending Deposit and redeem requests instead * forge fmt * Update requires and fix tests --------- Co-authored-by: Jeroen <[email protected]> * forge fmt * update MockRestrictionManagerFactory * uncomment name and symbol asserts --------- Co-authored-by: Adam Stox <[email protected]> Co-authored-by: Jeroen <[email protected]> Co-authored-by: Jeroen Offerijns <[email protected]> Co-authored-by: Adam Stox <[email protected]>

…into liquidity-pool-router

…xecution ( bridge + destination chain )

…Gateway.

… overall gas cost on all routers. Mark the contract address used as an entry point for the transaction.

…if there is provided value for gas payment.

…nsaction not triggered by CentrifugeRouter (#335) * fea:(Gateway) Allow partial payments from the gateway balance for transaction not triggered by CentrifugeRouter * chore: A forgotten file to be renamed

…ntrifuge/liquidity-pools into refactor-tt

src/token/Tranche.sol

src/CentrifugeRouter.sol

src/interfaces/token/IHook.sol

src/token/RestrictionManager.sol

test/unit/token/Tranche.t.sol

src/PoolManager.sol

…ntrifuge/liquidity-pools into refactor-tt

The base branch was changed.

…into refactor-tt

github-actions · 2024-07-02T10:24:45Z

Coverage after merging refactor-tt into main will be

87.68%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
src
Auth.sol	100%	100%	100%	100%
CentrifugeRouter.sol	60.16%	50%	70%	62.32%	101–102, 102, 102–103, 105, 107–109, 116, 131, 152, 152, 162, 166, 176, 181, 183–184, 184–185, 187–188, 188, 192–193, 193, 195, 195, 201, 206, 208, 225, 34–36, 38–39, 43, 50, 50, 73, 86
ERC7540Vault.sol	91.41%	72.73%	95.35%	93.88%	100, 128, 176–178, 203–205, 231–233, 245
Escrow.sol	75%	100%	66.67%	71.43%	29–30
InvestmentManager.sol	88.47%	83.70%	81.82%	92.43%	112, 230, 239, 256, 283, 312, 334, 366, 370–371, 38–39, 41–42, 490, 493, 524, 536–539, 548–551, 551, 551, 585–586
PoolManager.sol	90.41%	91.51%	69.23%	93.13%	186, 188, 188, 188–189, 191, 247, 346, 346, 346–347, 347, 347–348, 371, 52–54, 56–57
Root.sol	94.74%	88.89%	92.86%	97.73%	120, 125, 34
src/admin
Guardian.sol	66.67%	50%	77.78%	63.64%	21–23, 27, 27, 61
src/factories
ERC7540VaultFactory.sol	71.43%	100%	66.67%	72.73%	31, 33–34
TrancheFactory.sol	100%	100%	100%	100%
TransferProxyFactory.sol	100%	100%	100%	100%
src/gateway
GasService.sol	0%	0%	0%	0%	25–29, 31–32, 37, 37, 37–38, 38, 38–40, 45, 45, 45–47, 52, 57–58, 60, 60, 60–61, 63, 66, 71
Gateway.sol	89.11%	82.56%	82.35%	93.51%	133, 133, 133–134, 136, 169–170, 174, 234, 237–238, 241, 249, 285, 291, 308, 317, 62–65, 67–68, 83, 85
src/gateway/adapters/axelar
Adapter.sol	50%	80%	33.33%	37.50%	49–51, 53–54, 59, 59, 59–61, 91, 99
Forwarder.sol	80%	100%	75%	75%	38, 40–41
src/token
ERC20.sol	97.17%	94.12%	100%	98.33%	170, 69–70
RestrictionManager.sol	96.39%	96.43%	91.67%	97.67%	108–109
Tranche.sol	78.08%	54.55%	83.33%	90.91%	101–102, 117–118, 129, 135, 141, 141, 141–142, 147, 31, 91

ilinzweilin and others added 30 commits April 16, 2024 13:43

add liquidity pool router

d082fb3

merge main

820e025

apply comments from review

31c7dde

fix error

6e4b175

add cfg router deposit integration test

5f66b6c

add cfg router deposit integration test

1cf403c

Clean up ERC20

ff7f3d9

Merge branch 'main' of https://github.com/centrifuge/liquidity-pools …

4424ae5

…into clean-up-erc20

PoC

79dc102

WIP

192e8b5

WIP

3c81599

Merge branch 'main' of https://github.com/centrifuge/liquidity-pools …

0a46425

…into liquidity-pool-router

WIP

83b62fa

Fix conflicts

b3261e8

WIP

8f422ba

Compiling

f3308a0

Almost working

1868396

Merge branch 'main' of https://github.com/centrifuge/liquidity-pools …

81b4a1e

…into liquidity-pool-router

Move endorsements

8c3f436

Cleanup

1a60212

Reorder

587c27d

Rename

9166ea6

Typos

1fcad41

feat: Introduce cost payments for all components during transaction e…

73f754f

…xecution ( bridge + destination chain )

feat: Introduce cost payments for all components during transaction e…

47a2db5

…xecution ( bridge + destination chain )

feat: Add functio behind auth that allows to recover tokens from the …

b0d00fc

…Gateway.

chore: Make send in IGateway non-payable.

f366225

feat: Recover funds from Gateway. Provide a view function to estimate…

97a3260

… overall gas cost on all routers. Mark the contract address used as an entry point for the transaction.

fix: Borken test due to oncorrect conflict resolution. Estimate only …

651d1a5

…if there is provided value for gas payment.

hieronx and others added 6 commits June 30, 2024 12:32

Fix more encoding

795524c

Rename

6293c79

More interface simplifications

63c7166

Change requestRedeem sender

a1e1d92

fea:(Gateway) Allow partial payments from the gateway balance for tra…

9b35dc1

…nsaction not triggered by CentrifugeRouter (#335) * fea:(Gateway) Allow partial payments from the gateway balance for transaction not triggered by CentrifugeRouter * chore: A forgotten file to be renamed

Merge branch 'gg/feat/automate-gas-payments' of https://github.com/ce…

c87cf0c

…ntrifuge/liquidity-pools into refactor-tt