@bananapus/core-v6 0.0.37 → 0.0.38

package/test/TestForwardedTokenConsumption.sol

@@ -1,425 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity >=0.8.6;
-
-import {ERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
-import {IERC165} from "@openzeppelin/contracts/utils/introspection/IERC165.sol";
-
-import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
-import {JBMultiTerminal} from "../src/JBMultiTerminal.sol";
-import {IJBController} from "../src/interfaces/IJBController.sol";
-import {IJBCashOutHook} from "../src/interfaces/IJBCashOutHook.sol";
-import {IJBMultiTerminal} from "../src/interfaces/IJBMultiTerminal.sol";
-import {IJBPayHook} from "../src/interfaces/IJBPayHook.sol";
-import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
-import {IJBRulesetDataHook} from "../src/interfaces/IJBRulesetDataHook.sol";
-import {IJBSplitHook} from "../src/interfaces/IJBSplitHook.sol";
-import {IJBTokens} from "../src/interfaces/IJBTokens.sol";
-import {JBConstants} from "../src/libraries/JBConstants.sol";
-import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
-import {JBAfterCashOutRecordedContext} from "../src/structs/JBAfterCashOutRecordedContext.sol";
-import {JBAfterPayRecordedContext} from "../src/structs/JBAfterPayRecordedContext.sol";
-import {JBCashOutHookSpecification} from "../src/structs/JBCashOutHookSpecification.sol";
-import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
-import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
-import {JBPayHookSpecification} from "../src/structs/JBPayHookSpecification.sol";
-import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
-import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
-import {JBSplit} from "../src/structs/JBSplit.sol";
-import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
-import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
-
-contract TestForwardedTokenConsumption_Local is TestBaseWorkflow {
-    uint112 private constant _WEIGHT = 1000 * 10 ** 18;
-    uint256 private constant _PAY_AMOUNT = 10 * 10 ** 6;
-    uint256 private constant _HOOK_FORWARD_AMOUNT = 1 * 10 ** 6;
-    uint256 private constant _PAYOUT_AMOUNT = 4 * 10 ** 6;
-    address private constant _DATA_HOOK = address(bytes20(keccak256("datahook")));
-
-    IJBController private _controller;
-    IJBMultiTerminal private _terminal;
-    IJBMultiTerminal private _terminal2;
-    IJBTokens private _tokens;
-    address private _projectOwner;
-
-    uint64 private _projectId;
-
-    function setUp() public override {
-        super.setUp();
-
-        _controller = jbController();
-        _terminal = jbMultiTerminal();
-        _terminal2 = jbMultiTerminal2();
-        _tokens = jbTokens();
-        _projectOwner = multisig();
-
-        // Launch the fee beneficiary project first so project ID 1 has a terminal and accounting context.
-        _launchProject({
-            terminal: _terminal,
-            projectUri: "fee-project",
-            metadata: _metadataWithHooks({useDataHookForPay: true, useDataHookForCashOut: true, dataHook: _DATA_HOOK}),
-            splitGroups: new JBSplitGroup[](0),
-            fundAccessLimitGroups: new JBFundAccessLimitGroup[](0)
-        });
-
-        _projectId = uint64(
-            _launchProject({
-                terminal: _terminal,
-                projectUri: "hook-project",
-                metadata: _metadataWithHooks({
-                    useDataHookForPay: true, useDataHookForCashOut: true, dataHook: _DATA_HOOK
-                }),
-                splitGroups: new JBSplitGroup[](0),
-                fundAccessLimitGroups: new JBFundAccessLimitGroup[](0)
-            })
-        );
-    }
-
-    function test_RevertIfERC20PayHookDoesNotConsumeForwardedTokens() external {
-        NonConsumingPayHook hook = new NonConsumingPayHook();
-        JBPayHookSpecification[] memory specifications = new JBPayHookSpecification[](1);
-        specifications[0] =
-            JBPayHookSpecification({hook: hook, noop: false, amount: _HOOK_FORWARD_AMOUNT, metadata: ""});
-
-        vm.mockCall(
-            _DATA_HOOK,
-            abi.encodeWithSelector(IJBRulesetDataHook.beforePayRecordedWith.selector),
-            abi.encode(_WEIGHT, specifications)
-        );
-
-        vm.prank(multisig());
-        jbFeelessAddresses().setFeelessAddress(address(hook), true);
-
-        address payer = makeAddr("payer");
-        usdcToken().mint(payer, _PAY_AMOUNT);
-        vm.prank(payer);
-        usdcToken().approve(address(_terminal), _PAY_AMOUNT);
-
-        vm.expectRevert(
-            abi.encodeWithSelector(
-                JBMultiTerminal.JBMultiTerminal_TemporaryAllowanceNotConsumed.selector,
-                address(usdcToken()),
-                address(hook),
-                _HOOK_FORWARD_AMOUNT
-            )
-        );
-        vm.prank(payer);
-        _terminal.pay({
-            projectId: _projectId,
-            token: address(usdcToken()),
-            amount: _PAY_AMOUNT,
-            beneficiary: payer,
-            minReturnedTokens: 0,
-            memo: "",
-            metadata: ""
-        });
-    }
-
-    function test_RevertIfERC20CashOutHookDoesNotConsumeForwardedTokens() external {
-        JBPayHookSpecification[] memory emptyPaySpecifications = new JBPayHookSpecification[](0);
-        vm.mockCall(
-            _DATA_HOOK,
-            abi.encodeWithSelector(IJBRulesetDataHook.beforePayRecordedWith.selector),
-            abi.encode(_WEIGHT, emptyPaySpecifications)
-        );
-
-        usdcToken().mint(address(this), _PAY_AMOUNT);
-        usdcToken().approve(address(_terminal), _PAY_AMOUNT);
-
-        _terminal.pay({
-            projectId: _projectId,
-            token: address(usdcToken()),
-            amount: _PAY_AMOUNT,
-            beneficiary: address(this),
-            minReturnedTokens: 0,
-            memo: "",
-            metadata: ""
-        });
-
-        uint256 cashOutCount = _tokens.totalBalanceOf(address(this), _projectId) / 2;
-
-        NonConsumingCashOutHook hook = new NonConsumingCashOutHook();
-        JBCashOutHookSpecification[] memory specifications = new JBCashOutHookSpecification[](1);
-        specifications[0] =
-            JBCashOutHookSpecification({hook: hook, noop: false, amount: _HOOK_FORWARD_AMOUNT, metadata: ""});
-
-        vm.mockCall(
-            _DATA_HOOK,
-            abi.encodeWithSelector(IJBRulesetDataHook.beforeCashOutRecordedWith.selector),
-            abi.encode(
-                0,
-                cashOutCount,
-                _controller.totalTokenSupplyWithReservedTokensOf(_projectId),
-                _PAY_AMOUNT,
-                specifications
-            )
-        );
-
-        vm.prank(multisig());
-        jbFeelessAddresses().setFeelessAddress(address(hook), true);
-
-        vm.expectRevert(
-            abi.encodeWithSelector(
-                JBMultiTerminal.JBMultiTerminal_TemporaryAllowanceNotConsumed.selector,
-                address(usdcToken()),
-                address(hook),
-                _HOOK_FORWARD_AMOUNT
-            )
-        );
-        _terminal.cashOutTokensOf({
-            holder: address(this),
-            projectId: _projectId,
-            cashOutCount: cashOutCount,
-            tokenToReclaim: address(usdcToken()),
-            minTokensReclaimed: 0,
-            beneficiary: payable(address(this)),
-            metadata: ""
-        });
-    }
-
-    function test_SendPayoutsToMultiTerminalAddToBalanceConsumesForwardedAllowance() external {
-        uint256 recipientProjectId = _launchProject({
-            terminal: _terminal2,
-            projectUri: "recipient-add-to-balance",
-            metadata: _metadataWithHooks({
-                useDataHookForPay: false, useDataHookForCashOut: false, dataHook: address(0)
-            }),
-            splitGroups: new JBSplitGroup[](0),
-            fundAccessLimitGroups: new JBFundAccessLimitGroup[](0)
-        });
-
-        uint256 sourceProjectId = _launchProject({
-            terminal: _terminal,
-            projectUri: "source-add-to-balance",
-            metadata: _metadataWithHooks({
-                useDataHookForPay: false, useDataHookForCashOut: false, dataHook: address(0)
-            }),
-            splitGroups: _splitGroupsToProject({targetProjectId: recipientProjectId, preferAddToBalance: true}),
-            fundAccessLimitGroups: _usdcPayoutLimitGroups({terminal: _terminal, payoutAmount: _PAYOUT_AMOUNT})
-        });
-
-        usdcToken().mint(address(this), _PAYOUT_AMOUNT);
-        usdcToken().approve(address(_terminal), _PAYOUT_AMOUNT);
-
-        _terminal.pay({
-            projectId: sourceProjectId,
-            token: address(usdcToken()),
-            amount: _PAYOUT_AMOUNT,
-            beneficiary: address(this),
-            minReturnedTokens: 0,
-            memo: "",
-            metadata: ""
-        });
-
-        uint256 fee = _PAYOUT_AMOUNT * _terminal.FEE() / JBConstants.MAX_FEE;
-        uint256 expectedNetPayout = _PAYOUT_AMOUNT - fee;
-
-        _terminal.sendPayoutsOf({
-            projectId: sourceProjectId,
-            amount: _PAYOUT_AMOUNT,
-            currency: uint32(uint160(address(usdcToken()))),
-            token: address(usdcToken()),
-            minTokensPaidOut: 0
-        });
-
-        assertEq(usdcToken().allowance(address(_terminal), address(_terminal2)), 0);
-        assertEq(jbTerminalStore().balanceOf(address(_terminal), sourceProjectId, address(usdcToken())), fee);
-        assertEq(
-            jbTerminalStore().balanceOf(address(_terminal2), recipientProjectId, address(usdcToken())),
-            expectedNetPayout
-        );
-        assertEq(usdcToken().balanceOf(address(_terminal2)), expectedNetPayout);
-    }
-
-    function test_SendPayoutsToMultiTerminalPayConsumesForwardedAllowance() external {
-        uint256 recipientProjectId = _launchProject({
-            terminal: _terminal2,
-            projectUri: "recipient-pay",
-            metadata: _metadataWithHooks({
-                useDataHookForPay: false, useDataHookForCashOut: false, dataHook: address(0)
-            }),
-            splitGroups: new JBSplitGroup[](0),
-            fundAccessLimitGroups: new JBFundAccessLimitGroup[](0)
-        });
-
-        uint256 sourceProjectId = _launchProject({
-            terminal: _terminal,
-            projectUri: "source-pay",
-            metadata: _metadataWithHooks({
-                useDataHookForPay: false, useDataHookForCashOut: false, dataHook: address(0)
-            }),
-            splitGroups: _splitGroupsToProject({targetProjectId: recipientProjectId, preferAddToBalance: false}),
-            fundAccessLimitGroups: _usdcPayoutLimitGroups({terminal: _terminal, payoutAmount: _PAYOUT_AMOUNT})
-        });
-
-        usdcToken().mint(address(this), _PAYOUT_AMOUNT);
-        usdcToken().approve(address(_terminal), _PAYOUT_AMOUNT);
-
-        _terminal.pay({
-            projectId: sourceProjectId,
-            token: address(usdcToken()),
-            amount: _PAYOUT_AMOUNT,
-            beneficiary: address(this),
-            minReturnedTokens: 0,
-            memo: "",
-            metadata: ""
-        });
-
-        uint256 fee = _PAYOUT_AMOUNT * _terminal.FEE() / JBConstants.MAX_FEE;
-        uint256 expectedNetPayout = _PAYOUT_AMOUNT - fee;
-
-        _terminal.sendPayoutsOf({
-            projectId: sourceProjectId,
-            amount: _PAYOUT_AMOUNT,
-            currency: uint32(uint160(address(usdcToken()))),
-            token: address(usdcToken()),
-            minTokensPaidOut: 0
-        });
-
-        assertEq(usdcToken().allowance(address(_terminal), address(_terminal2)), 0);
-        assertEq(jbTerminalStore().balanceOf(address(_terminal), sourceProjectId, address(usdcToken())), fee);
-        assertEq(
-            jbTerminalStore().balanceOf(address(_terminal2), recipientProjectId, address(usdcToken())),
-            expectedNetPayout
-        );
-        assertEq(usdcToken().balanceOf(address(_terminal2)), expectedNetPayout);
-    }
-
-    function _launchProject(
-        IJBMultiTerminal terminal,
-        string memory projectUri,
-        JBRulesetMetadata memory metadata,
-        JBSplitGroup[] memory splitGroups,
-        JBFundAccessLimitGroup[] memory fundAccessLimitGroups
-    )
-        internal
-        returns (uint256)
-    {
-        JBRulesetConfig[] memory rulesetConfigurations = new JBRulesetConfig[](1);
-        rulesetConfigurations[0].mustStartAtOrAfter = 0;
-        rulesetConfigurations[0].duration = 0;
-        rulesetConfigurations[0].weight = _WEIGHT;
-        rulesetConfigurations[0].weightCutPercent = 0;
-        rulesetConfigurations[0].approvalHook = IJBRulesetApprovalHook(address(0));
-        rulesetConfigurations[0].metadata = metadata;
-        rulesetConfigurations[0].splitGroups = splitGroups;
-        rulesetConfigurations[0].fundAccessLimitGroups = fundAccessLimitGroups;
-
-        JBTerminalConfig[] memory terminalConfigurations = new JBTerminalConfig[](1);
-        terminalConfigurations[0] =
-            JBTerminalConfig({terminal: terminal, accountingContextsToAccept: _usdcAccountingContexts()});
-
-        return _controller.launchProjectFor({
-            owner: _projectOwner,
-            projectUri: projectUri,
-            rulesetConfigurations: rulesetConfigurations,
-            terminalConfigurations: terminalConfigurations,
-            memo: ""
-        });
-    }
-
-    function _metadataWithHooks(
-        bool useDataHookForPay,
-        bool useDataHookForCashOut,
-        address dataHook
-    )
-        internal
-        view
-        returns (JBRulesetMetadata memory)
-    {
-        return JBRulesetMetadata({
-            reservedPercent: 0,
-            cashOutTaxRate: 0,
-            baseCurrency: uint32(uint160(address(usdcToken()))),
-            pausePay: false,
-            pauseCreditTransfers: false,
-            allowOwnerMinting: false,
-            allowSetCustomToken: false,
-            allowTerminalMigration: false,
-            allowSetTerminals: false,
-            ownerMustSendPayouts: false,
-            allowSetController: false,
-            allowAddAccountingContext: true,
-            allowAddPriceFeed: false,
-            holdFees: false,
-            useTotalSurplusForCashOuts: false,
-            useDataHookForPay: useDataHookForPay,
-            useDataHookForCashOut: useDataHookForCashOut,
-            dataHook: dataHook,
-            metadata: 0
-        });
-    }
-
-    function _splitGroupsToProject(
-        uint256 targetProjectId,
-        bool preferAddToBalance
-    )
-        internal
-        view
-        returns (JBSplitGroup[] memory)
-    {
-        JBSplit[] memory splits = new JBSplit[](1);
-        splits[0].percent = uint32(JBConstants.SPLITS_TOTAL_PERCENT);
-        // forge-lint: disable-next-line(unsafe-typecast)
-        splits[0].projectId = uint64(targetProjectId);
-        splits[0].beneficiary = payable(address(0));
-        splits[0].preferAddToBalance = preferAddToBalance;
-        splits[0].lockedUntil = 0;
-        splits[0].hook = IJBSplitHook(address(0));
-
-        JBSplitGroup[] memory splitGroups = new JBSplitGroup[](1);
-        splitGroups[0] = JBSplitGroup({groupId: uint256(uint160(address(usdcToken()))), splits: splits});
-
-        return splitGroups;
-    }
-
-    function _usdcAccountingContexts() internal view returns (JBAccountingContext[] memory) {
-        JBAccountingContext[] memory accountingContexts = new JBAccountingContext[](1);
-        accountingContexts[0] = JBAccountingContext({
-            token: address(usdcToken()),
-            decimals: usdcToken().decimals(),
-            currency: uint32(uint160(address(usdcToken())))
-        });
-
-        return accountingContexts;
-    }
-
-    function _usdcPayoutLimitGroups(
-        IJBMultiTerminal terminal,
-        uint256 payoutAmount
-    )
-        internal
-        view
-        returns (JBFundAccessLimitGroup[] memory)
-    {
-        JBCurrencyAmount[] memory payoutLimits = new JBCurrencyAmount[](1);
-        // forge-lint: disable-next-line(unsafe-typecast)
-        payoutLimits[0] =
-            JBCurrencyAmount({amount: uint224(payoutAmount), currency: uint32(uint160(address(usdcToken())))});
-
-        JBFundAccessLimitGroup[] memory fundAccessLimitGroups = new JBFundAccessLimitGroup[](1);
-        fundAccessLimitGroups[0] = JBFundAccessLimitGroup({
-            terminal: address(terminal),
-            token: address(usdcToken()),
-            payoutLimits: payoutLimits,
-            surplusAllowances: new JBCurrencyAmount[](0)
-        });
-
-        return fundAccessLimitGroups;
-    }
-}
-
-contract NonConsumingPayHook is ERC165, IJBPayHook {
-    function afterPayRecordedWith(JBAfterPayRecordedContext calldata) external payable override {}
-
-    function supportsInterface(bytes4 interfaceId) public view override(ERC165, IERC165) returns (bool) {
-        return interfaceId == type(IJBPayHook).interfaceId || super.supportsInterface(interfaceId);
-    }
-}
-
-contract NonConsumingCashOutHook is ERC165, IJBCashOutHook {
-    function afterCashOutRecordedWith(JBAfterCashOutRecordedContext calldata) external payable override {}
-
-    function supportsInterface(bytes4 interfaceId) public view override(ERC165, IERC165) returns (bool) {
-        return interfaceId == type(IJBCashOutHook).interfaceId || super.supportsInterface(interfaceId);
-    }
-}

package/test/TestInterfaceSupport.sol

@@ -1,81 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity ^0.8.6;
-
-import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
-import {JBController} from "../src/JBController.sol";
-import {JBDeadline} from "../src/JBDeadline.sol";
-import {JBMultiTerminal} from "../src/JBMultiTerminal.sol";
-import {JBProjects} from "../src/JBProjects.sol";
-import {IJBCashOutTerminal} from "../src/interfaces/IJBCashOutTerminal.sol";
-import {IJBController} from "../src/interfaces/IJBController.sol";
-import {IJBFeeTerminal} from "../src/interfaces/IJBFeeTerminal.sol";
-import {IJBMigratable} from "../src/interfaces/IJBMigratable.sol";
-import {IJBMultiTerminal} from "../src/interfaces/IJBMultiTerminal.sol";
-import {IJBPayoutTerminal} from "../src/interfaces/IJBPayoutTerminal.sol";
-import {IJBPermissioned} from "../src/interfaces/IJBPermissioned.sol";
-import {IJBPermitTerminal} from "../src/interfaces/IJBPermitTerminal.sol";
-import {IJBProjects} from "../src/interfaces/IJBProjects.sol";
-import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
-import {IJBTerminal} from "../src/interfaces/IJBTerminal.sol";
-import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
-import {IERC721Metadata} from "@openzeppelin/contracts/token/ERC721/extensions/IERC721Metadata.sol";
-import {IERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
-
-/// Contracts are introspective about the interfaces they adhere to.
-contract TestEIP165_Local is TestBaseWorkflow {
-    // forge-lint: disable-next-line(screaming-snake-case-const)
-    bytes4 private constant _notSupportedInterface = 0xffffffff;
-
-    function testJBController() public view {
-        JBController _controller = jbController();
-
-        // Should support these interfaces.
-        assertTrue(_controller.supportsInterface(type(IERC165).interfaceId));
-        assertTrue(_controller.supportsInterface(type(IJBMigratable).interfaceId));
-        assertTrue(_controller.supportsInterface(type(IJBPermissioned).interfaceId));
-        assertTrue(_controller.supportsInterface(type(IJBController).interfaceId));
-
-        // Make sure it doesn't always return true.
-        assertTrue(!_controller.supportsInterface(_notSupportedInterface));
-    }
-
-    function testJBMultiTerminal() public view {
-        JBMultiTerminal _terminal = jbMultiTerminal();
-
-        // Should support these interfaces.
-        assertTrue(_terminal.supportsInterface(type(IJBMultiTerminal).interfaceId));
-        assertTrue(_terminal.supportsInterface(type(IJBTerminal).interfaceId));
-        assertTrue(_terminal.supportsInterface(type(IJBCashOutTerminal).interfaceId));
-        assertTrue(_terminal.supportsInterface(type(IJBPayoutTerminal).interfaceId));
-        assertTrue(_terminal.supportsInterface(type(IJBPermitTerminal).interfaceId));
-        assertTrue(_terminal.supportsInterface(type(IJBFeeTerminal).interfaceId));
-        assertTrue(_terminal.supportsInterface(type(IERC165).interfaceId));
-
-        // Make sure it doesn't always return true.
-        assertTrue(!_terminal.supportsInterface(_notSupportedInterface));
-    }
-
-    function testJBProjects() public view {
-        JBProjects _projects = jbProjects();
-
-        // Should support these interfaces.
-        assertTrue(_projects.supportsInterface(type(IERC165).interfaceId));
-        assertTrue(_projects.supportsInterface(type(IERC721).interfaceId));
-        assertTrue(_projects.supportsInterface(type(IERC721Metadata).interfaceId));
-        assertTrue(_projects.supportsInterface(type(IJBProjects).interfaceId));
-
-        // Make sure it doesn't always return true.
-        assertTrue(!_projects.supportsInterface(_notSupportedInterface));
-    }
-
-    function testJBDeadline() public {
-        JBDeadline _approvalHook = new JBDeadline(3000);
-
-        // Should support these interfaces.
-        assertTrue(_approvalHook.supportsInterface(type(IERC165).interfaceId));
-        assertTrue(_approvalHook.supportsInterface(type(IJBRulesetApprovalHook).interfaceId));
-
-        // Make sure it doesn't always return true.
-        assertTrue(!_approvalHook.supportsInterface(_notSupportedInterface));
-    }
-}

package/test/TestJBERC20Inheritance.sol

@@ -1,103 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity 0.8.28;
-
-import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
-import {JBERC20} from "../src/JBERC20.sol";
-import {IJBPermissions} from "../src/interfaces/IJBPermissions.sol";
-import {IJBProjects} from "../src/interfaces/IJBProjects.sol";
-import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
-import {IJBToken} from "../src/interfaces/IJBToken.sol";
-import {JBConstants} from "../src/libraries/JBConstants.sol";
-import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
-import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
-import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
-import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
-import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
-import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
-
-import {ERC20Votes} from "../src/JBERC20.sol";
-
-contract JBERC20Inheritance_Local is JBERC20(IJBPermissions(address(1)), IJBProjects(address(2))), TestBaseWorkflow {
-    /// This test is to verify that the inheritance order of JBERC20 is correct and that it calls the
-    /// `ERC20Votes._update()`
-    /// forge-config: default.allow_internal_expect_revert = true
-    function test_votesUpdate() public {
-        uint256 _max = _maxSupply();
-        vm.expectRevert(abi.encodeWithSelector(ERC20Votes.ERC20ExceededSafeSupply.selector, _max + 1, _max));
-
-        _update(address(0), address(100), _max + 1);
-    }
-
-    // This test checks that voting power gets accounted for.
-    function test_votesUpdateOnTransfer() public {
-        JBRulesetMetadata memory _metadata = JBRulesetMetadata({
-            reservedPercent: 0,
-            cashOutTaxRate: 0,
-            baseCurrency: uint32(uint160(JBConstants.NATIVE_TOKEN)),
-            pausePay: false,
-            pauseCreditTransfers: false,
-            allowOwnerMinting: true,
-            allowSetCustomToken: false,
-            allowTerminalMigration: false,
-            allowSetTerminals: false,
-            ownerMustSendPayouts: false,
-            allowSetController: false,
-            allowAddAccountingContext: true,
-            allowAddPriceFeed: false,
-            holdFees: false,
-            useTotalSurplusForCashOuts: false,
-            useDataHookForPay: true,
-            useDataHookForCashOut: true,
-            dataHook: address(0),
-            metadata: 0
-        });
-        address _projectOwner = multisig();
-        address _recipient = address(200);
-        uint256 _amount = 10 ether;
-
-        // Package up ruleset configuration.
-        JBRulesetConfig[] memory _rulesetConfig = new JBRulesetConfig[](1);
-        _rulesetConfig[0].mustStartAtOrAfter = 0;
-        _rulesetConfig[0].duration = 0;
-        _rulesetConfig[0].weight = 0;
-        _rulesetConfig[0].weightCutPercent = 0;
-        _rulesetConfig[0].approvalHook = IJBRulesetApprovalHook(address(0));
-        _rulesetConfig[0].metadata = _metadata;
-        _rulesetConfig[0].splitGroups = new JBSplitGroup[](0);
-        _rulesetConfig[0].fundAccessLimitGroups = new JBFundAccessLimitGroup[](0);
-
-        // Package up terminal configuration.
-        JBTerminalConfig[] memory _terminalConfigurations = new JBTerminalConfig[](1);
-        JBAccountingContext[] memory _tokensToAccept = new JBAccountingContext[](1);
-        _tokensToAccept[0] = JBAccountingContext({
-            token: JBConstants.NATIVE_TOKEN, decimals: 18, currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
-        });
-        _terminalConfigurations[0] =
-            JBTerminalConfig({terminal: jbMultiTerminal(), accountingContextsToAccept: _tokensToAccept});
-
-        uint256 projectId = jbController()
-            .launchProjectFor({
-                owner: _projectOwner,
-                projectUri: "myIPFSHash",
-                rulesetConfigurations: _rulesetConfig,
-                terminalConfigurations: _terminalConfigurations,
-                memo: ""
-            });
-
-        // Configure a token.
-        vm.prank(_projectOwner);
-        IJBToken _token = jbController().deployERC20For(projectId, "TestToken", "TEST", bytes32(0));
-
-        // Have the user delegate to themselves.
-        vm.prank(_recipient);
-        ERC20Votes(address(_token)).delegate(_recipient);
-
-        // Mint tokens to the user, check that the balance is correct.
-        vm.startPrank(_projectOwner);
-        jbController().mintTokensOf(projectId, _amount, _recipient, "", false);
-        assertEq(_token.balanceOf(_recipient), _amount);
-
-        // Assert that the user received the voting power as expected.
-        assertEq(ERC20Votes(address(_token)).getVotes(_recipient), _amount);
-    }
-}