fix: `LlamaAccount.execute` low-level call should use value from a parameter instead of `msg.value`

[0xrajath]

Motivation:

The LlamaAccount.execute allows tx.origin to include some eth in their call currently, which will then be included in the call from LlamaAccount.execute

Because of this, the account cannot for example, use call to deposit eth into weth, trade eth on uniswap, etc.

We should change this such that it uses the ETH in the Llama account.

Modifications:

• LlamaAccount.execute now takes an additional parameter uint256 value and uses that value in the low-level call instead of msg.value
• LlamaAccount.execute is now not payable.
• DelegatecallWithValueNotAllowed error is thrown when value > 0 and it is a delegatecall.
• Appropriate tests.

Result:

Closes #460

[github-actions[bot]]

Coverage after merging rajath/account-msg-value into main will be

84.03%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
src
LlamaCore.sol	99.26%	97.30%	100%	100%	292, 393
LlamaExecutor.sol	80%	50%	100%	100%	33
LlamaFactory.sol	100%	100%	100%	100%
LlamaLens.sol	72.09%	20%	100%	82.61%	175–176, 179, 179, 179–180, 180, 180–181, 181, 181, 189
LlamaPolicy.sol	91.49%	88.89%	94.59%	91.30%	324, 361, 361, 361–363, 363, 363, 365–368, 370, 383
LlamaPolicyMetadata.sol	100%	100%	100%	100%
src/accounts
LlamaAccount.sol	100%	100%	100%	100%
src/lib
ERC721NonTransferableMinimalProxy.sol	70.42%	72.73%	72.73%	68.42%	102, 104, 106, 118–120, 190, 88, 88, 88, 90, 90, 90, 92, 92, 92, 97, 99
LlamaUtils.sol	100%	100%	100%	100%
PolicyholderCheckpoints.sol	55.88%	50%	81.82%	53.62%	125, 179–181, 181, 181–182, 184, 187, 217, 224, 224–226, 228, 228–230, 232, 232–234, 236, 236–238, 257, 260–266, 273, 46, 46–48, 48, 48–49, 51
SupplyCheckpoints.sol	57.14%	50%	83.33%	54.93%	131, 183–185, 185, 185–186, 188, 191, 235, 242, 242–244, 246, 246–248, 250, 250–252, 254, 254–256, 275, 278–284, 291, 50, 50–52, 52, 52–53, 55
src/llama-scripts
LlamaGovernanceScript.sol	50%	33.33%	43.75%	53.45%	106–108, 115–117, 125–128, 135–137, 146–150, 157–158, 166–168, 67, 74, 74, 76, 89–90, 97–98
src/strategies/absolute
LlamaAbsolutePeerReview.sol	100%	100%	100%	100%
LlamaAbsoluteQuorum.sol	100%	100%	100%	100%
LlamaAbsoluteStrategyBase.sol	94.74%	87.50%	90.91%	97.96%	238, 241, 289
src/strategies/relative
LlamaRelativeHolderQuorum.sol	91.67%	75%	100%	100%	46, 53
LlamaRelativeQuantityQuorum.sol	0%	0%	0%	0%	26, 26, 26–28, 38, 38, 38–40, 45–46, 46, 46–47, 52–53, 53, 53–54
LlamaRelativeStrategyBase.sol	97.59%	90.91%	100%	100%	203, 303
LlamaRelativeUniqueHolderQuorum.sol	93.33%	83.33%	100%	100%	48, 55