@bananapus/core-v6 0.0.37 → 0.0.38

package/test/regression/WeightCacheBoundary.t.sol

@@ -1,291 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity >=0.8.6;
-
-import {TestBaseWorkflow} from "../helpers/TestBaseWorkflow.sol";
-import {IJBController} from "../../src/interfaces/IJBController.sol";
-import {IJBRulesets} from "../../src/interfaces/IJBRulesets.sol";
-import {IJBRulesetApprovalHook} from "../../src/interfaces/IJBRulesetApprovalHook.sol";
-import {JBConstants} from "../../src/libraries/JBConstants.sol";
-import {JBRulesetMetadata} from "../../src/structs/JBRulesetMetadata.sol";
-import {JBRulesetConfig} from "../../src/structs/JBRulesetConfig.sol";
-import {JBRuleset} from "../../src/structs/JBRuleset.sol";
-import {JBSplitGroup} from "../../src/structs/JBSplitGroup.sol";
-import {JBFundAccessLimitGroup} from "../../src/structs/JBFundAccessLimitGroup.sol";
-import {JBTerminalConfig} from "../../src/structs/JBTerminalConfig.sol";
-import {JBRulesets} from "../../src/JBRulesets.sol";
-
-import {mulDiv} from "@prb/math/src/Common.sol";
-
-/// @notice Deterministic (non-fuzz) tests for the JBRulesets weight cache at the 20,000 iteration boundary.
-/// The contract reverts with `JBRulesets_WeightCacheRequired` when the cycle gap exceeds the threshold
-/// and the cache has not been populated. These tests verify:
-///   1. Exactly 20,000 cycles work without cache.
-///   2. 20,001 cycles revert without cache.
-///   3. Progressive `updateRulesetWeightCache()` calls unblock the revert.
-///   4. The cached weight equals the iteratively computed weight.
-contract WeightCacheBoundary_Local is TestBaseWorkflow {
-    uint256 private constant _THRESHOLD = 20_000;
-
-    /// @dev 1-second duration makes each cycle = 1 second, so we can hit the boundary fast.
-    uint32 private constant _DURATION = 1;
-
-    /// @dev Tiny weight cut (0.1%) so the weight doesn't decay to zero over 20k cycles.
-    uint32 private constant _WEIGHT_CUT_PERCENT = 1_000_000; // 0.1% of MAX_WEIGHT_CUT_PERCENT (1e9)
-
-    uint112 private constant _INITIAL_WEIGHT = 1000e18;
-
-    IJBController private _controller;
-    IJBRulesets private _rulesets;
-    address private _projectOwner;
-
-    JBRulesetMetadata private _metadata;
-
-    function setUp() public override {
-        super.setUp();
-
-        _projectOwner = multisig();
-        _rulesets = jbRulesets();
-        _controller = jbController();
-
-        _metadata = JBRulesetMetadata({
-            reservedPercent: 0,
-            cashOutTaxRate: 0,
-            baseCurrency: uint32(uint160(JBConstants.NATIVE_TOKEN)),
-            pausePay: false,
-            pauseCreditTransfers: false,
-            allowOwnerMinting: false,
-            allowSetCustomToken: false,
-            allowTerminalMigration: false,
-            allowSetTerminals: false,
-            ownerMustSendPayouts: false,
-            allowSetController: false,
-            allowAddAccountingContext: true,
-            allowAddPriceFeed: false,
-            holdFees: false,
-            useTotalSurplusForCashOuts: true,
-            useDataHookForPay: false,
-            useDataHookForCashOut: false,
-            dataHook: address(0),
-            metadata: 0
-        });
-    }
-
-    /// @dev Helper: launch a project with a 1-second duration and the given weight cut percent.
-    function _launchProject() internal returns (uint256 projectId) {
-        JBRulesetConfig[] memory configs = new JBRulesetConfig[](1);
-        configs[0].mustStartAtOrAfter = 0;
-        configs[0].duration = _DURATION;
-        configs[0].weight = _INITIAL_WEIGHT;
-        configs[0].weightCutPercent = _WEIGHT_CUT_PERCENT;
-        configs[0].approvalHook = IJBRulesetApprovalHook(address(0));
-        configs[0].metadata = _metadata;
-        configs[0].splitGroups = new JBSplitGroup[](0);
-        configs[0].fundAccessLimitGroups = new JBFundAccessLimitGroup[](0);
-
-        projectId = _controller.launchProjectFor({
-            owner: _projectOwner,
-            projectUri: "weightCacheBoundary",
-            rulesetConfigurations: configs,
-            terminalConfigurations: new JBTerminalConfig[](0),
-            memo: ""
-        });
-    }
-
-    /// @dev Compute expected weight by iterating the cut percent `n` times on the initial weight.
-    function _expectedWeight(uint256 n) internal pure returns (uint256 weight) {
-        weight = _INITIAL_WEIGHT;
-        for (uint256 i; i < n; i++) {
-            weight = mulDiv(
-                weight, JBConstants.MAX_WEIGHT_CUT_PERCENT - _WEIGHT_CUT_PERCENT, JBConstants.MAX_WEIGHT_CUT_PERCENT
-            );
-            if (weight == 0) break;
-        }
-    }
-
-    // ═══════════════════════════════════════════════════════════════════
-    //  Test 1: Exactly 20,000 cycles works without cache
-    // ═══════════════════════════════════════════════════════════════════
-
-    function test_exactlyAtThreshold_noCache_succeeds() public {
-        uint256 projectId = _launchProject();
-
-        // Warp forward exactly 20,000 durations.
-        vm.warp(block.timestamp + _DURATION * _THRESHOLD);
-
-        // currentOf should succeed — still within the iteration limit.
-        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
-
-        // Verify weight matches the iterative calculation.
-        uint256 expected = _expectedWeight(_THRESHOLD);
-        assertEq(ruleset.weight, expected, "Weight at exactly 20,000 cycles should match iterative calculation");
-        assertTrue(ruleset.weight > 0, "Weight should not have decayed to zero at 20k cycles with 0.1% cut");
-    }
-
-    // ═══════════════════════════════════════════════════════════════════
-    //  Test 2: 20,001 cycles reverts without cache
-    // ═══════════════════════════════════════════════════════════════════
-
-    function test_aboveThreshold_noCache_reverts() public {
-        uint256 projectId = _launchProject();
-
-        // Warp forward 20,001 durations.
-        vm.warp(block.timestamp + _DURATION * (_THRESHOLD + 1));
-
-        // currentOf should revert because the iteration count exceeds the threshold.
-        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
-        _rulesets.currentOf(projectId);
-    }
-
-    // ═══════════════════════════════════════════════════════════════════
-    //  Test 3: Single cache call unblocks 20,001 cycles
-    // ═══════════════════════════════════════════════════════════════════
-
-    function test_aboveThreshold_singleCacheCall_succeeds() public {
-        uint256 projectId = _launchProject();
-        uint256 rulesetId = _rulesets.latestRulesetIdOf(projectId);
-
-        // Warp forward 20,001 durations.
-        vm.warp(block.timestamp + _DURATION * (_THRESHOLD + 1));
-
-        // Without cache, this reverts.
-        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
-        _rulesets.currentOf(projectId);
-
-        // Populate the cache.
-        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
-
-        // Now currentOf should succeed.
-        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
-
-        // Verify weight matches iterative calculation.
-        uint256 expected = _expectedWeight(_THRESHOLD + 1);
-        assertEq(ruleset.weight, expected, "Weight at 20,001 cycles should match after single cache call");
-    }
-
-    // ═══════════════════════════════════════════════════════════════════
-    //  Test 4: Progressive caching for large gaps (40,001 cycles)
-    // ═══════════════════════════════════════════════════════════════════
-
-    function test_largeGap_progressiveCaching_succeeds() public {
-        uint256 projectId = _launchProject();
-        uint256 rulesetId = _rulesets.latestRulesetIdOf(projectId);
-
-        // Warp forward 40,001 durations (needs 2 cache calls).
-        vm.warp(block.timestamp + _DURATION * (_THRESHOLD * 2 + 1));
-
-        // Without cache, reverts.
-        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
-        _rulesets.currentOf(projectId);
-
-        // First cache call: advances cache by up to 20,000 iterations.
-        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
-
-        // Still reverts — we need another cache call to cover the remaining gap.
-        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
-        _rulesets.currentOf(projectId);
-
-        // Second cache call: covers the rest.
-        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
-
-        // Now it should succeed.
-        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
-
-        // Verify weight matches iterative calculation.
-        uint256 expected = _expectedWeight(_THRESHOLD * 2 + 1);
-        assertEq(ruleset.weight, expected, "Weight at 40,001 cycles should match after two cache calls");
-    }
-
-    // ═══════════════════════════════════════════════════════════════════
-    //  Test 5: Cache produces same weight as direct iteration
-    // ═══════════════════════════════════════════════════════════════════
-
-    function test_cachedWeight_matchesDirect_atBoundary() public {
-        // Launch two identical projects.
-        uint256 projectA = _launchProject();
-        uint256 projectB = _launchProject();
-
-        uint256 rulesetIdB = _rulesets.latestRulesetIdOf(projectB);
-
-        // Warp to exactly 20,000 cycles (within threshold for both).
-        vm.warp(block.timestamp + _DURATION * _THRESHOLD);
-
-        // Project A: no cache, direct iteration.
-        JBRuleset memory rulesetA = _rulesets.currentOf(projectA);
-
-        // Project B: populate cache first, then query.
-        _rulesets.updateRulesetWeightCache(projectB, rulesetIdB);
-        JBRuleset memory rulesetB = _rulesets.currentOf(projectB);
-
-        // Both should produce identical weights.
-        assertEq(rulesetA.weight, rulesetB.weight, "Cached and uncached weights should be identical at threshold");
-    }
-
-    // ═══════════════════════════════════════════════════════════════════
-    //  Test 6: Zero weightCutPercent — cache is no-op
-    // ═══════════════════════════════════════════════════════════════════
-
-    function test_zeroWeightCut_cacheNoOp() public {
-        // Launch a project with zero weight cut.
-        JBRulesetConfig[] memory configs = new JBRulesetConfig[](1);
-        configs[0].mustStartAtOrAfter = 0;
-        configs[0].duration = _DURATION;
-        configs[0].weight = _INITIAL_WEIGHT;
-        configs[0].weightCutPercent = 0; // No decay
-        configs[0].approvalHook = IJBRulesetApprovalHook(address(0));
-        configs[0].metadata = _metadata;
-        configs[0].splitGroups = new JBSplitGroup[](0);
-        configs[0].fundAccessLimitGroups = new JBFundAccessLimitGroup[](0);
-
-        uint256 projectId = _controller.launchProjectFor({
-            owner: _projectOwner,
-            projectUri: "zeroCut",
-            rulesetConfigurations: configs,
-            terminalConfigurations: new JBTerminalConfig[](0),
-            memo: ""
-        });
-
-        // Warp far beyond the threshold — since weightCutPercent = 0, no iterations needed.
-        vm.warp(block.timestamp + _DURATION * (_THRESHOLD * 5));
-
-        // Should succeed without any cache because the weight cut loop is skipped.
-        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
-        assertEq(ruleset.weight, _INITIAL_WEIGHT, "Weight should remain unchanged with zero cut percent");
-    }
-
-    // ═══════════════════════════════════════════════════════════════════
-    //  Test 7: Cache with zero duration — updateRulesetWeightCache is no-op
-    // ═══════════════════════════════════════════════════════════════════
-
-    function test_zeroDuration_cacheNoOp() public {
-        // Launch a project with zero duration.
-        JBRulesetConfig[] memory configs = new JBRulesetConfig[](1);
-        configs[0].mustStartAtOrAfter = 0;
-        configs[0].duration = 0; // Never expires
-        configs[0].weight = _INITIAL_WEIGHT;
-        configs[0].weightCutPercent = _WEIGHT_CUT_PERCENT;
-        configs[0].approvalHook = IJBRulesetApprovalHook(address(0));
-        configs[0].metadata = _metadata;
-        configs[0].splitGroups = new JBSplitGroup[](0);
-        configs[0].fundAccessLimitGroups = new JBFundAccessLimitGroup[](0);
-
-        uint256 projectId = _controller.launchProjectFor({
-            owner: _projectOwner,
-            projectUri: "zeroDuration",
-            rulesetConfigurations: configs,
-            terminalConfigurations: new JBTerminalConfig[](0),
-            memo: ""
-        });
-
-        uint256 rulesetId = _rulesets.latestRulesetIdOf(projectId);
-
-        // Warp far ahead.
-        vm.warp(block.timestamp + 1_000_000);
-
-        // updateRulesetWeightCache should be a no-op (returns without caching).
-        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
-
-        // currentOf should succeed — zero duration means the same ruleset is always current.
-        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
-        assertEq(ruleset.weight, _INITIAL_WEIGHT, "Weight should remain unchanged with zero duration");
-    }
-}

package/test/trees/JBController/burnTokensOf.tree

@@ -1,9 +0,0 @@
-TestBurnTokensOf_Local
-├── when caller is not permissioned or terminal
-│   └── it should revert UNAUTHORIZED()
-├── when caller is terminal and tokenCount gt zero
-│   └── it should burn and emit BurnTokens
-├── when caller has permission and tokenCount gt zero
-│   └── it should burn and emit BurnTokens
-└── when tokenCount eq zero
-    └── it should revert NO_BURNABLE_TOKENS

package/test/trees/JBController/claimTokensFor.tree

@@ -1,5 +0,0 @@
-TestClaimTokensFor_Local
-├── when caller is permissioned
-│   └── it should call JBTokens to claim on behalf of holder
-└── when caller is not permissioned
-    └── it should revert UNAUTHORIZED

package/test/trees/JBController/deployERC20For.tree

@@ -1,5 +0,0 @@
-TestDeployERC20For_Local
-├── when caller is permissioned
-│   └── it will deploy ERC20 and return IJBToken
-└── when caller is not permissioned
-    └── it will revert UNAUTHORIZED

package/test/trees/JBController/getRulesetOf.tree

@@ -1,5 +0,0 @@
-TestGetRulesetOf
-├── when rulesetId is zero
-│   └── it should return empty ruleset and metadata
-└── when rulesetID is non-zero
-    └── it should return populated ruleset and metadata

package/test/trees/JBController/launchProjectFor.tree

@@ -1,12 +0,0 @@
-TestLaunchProjectFor_Local
-└── when called
-    ├── given metadata is provided
-    │   └── it will set metadata
-    ├── given rulesets are provided
-    │   └── it will set rulesets
-    ├── given terminals are provided
-    │   └── it will set terminals
-    ├── given memo is provided
-    │   └── it will be included in the emit
-    └── given Directory exists
-        └── it will setControllerOf to itself

package/test/trees/JBController/launchRulesetsFor.tree

@@ -1,8 +0,0 @@
-TestLaunchRulesetsFor
-├── when caller does not have permission
-│   └── it should revert
-└── when caller has permission
-    ├── given the project already has rulesets
-    │   └── it should revert
-    └── given the project does not yet have rulesets
-        └── it should set the controller, queue the rulesets, configure terminals, and emit LaunchRulesets

package/test/trees/JBController/migrateController.tree

@@ -1,12 +0,0 @@
-TestMigrateController
-├── when caller does not have MIGRATE_CONTROLLER permission
-│   └── it should revert
-└── when caller has MIGRATE_CONTROLLER permission
-    ├── given migration is not allowed by ruleset
-    │   └── it should revert
-    └── given migration is allowed by ruleset
-        ├── given reserved token balance is pending
-        │   └── it should send reserved tokens to splits
-        └── given no reserved token balance is pending
-            ├── it should prepare new controller for migration
-            └── it should emit MigrateController event

package/test/trees/JBController/mintTokensOf.tree

@@ -1,12 +0,0 @@
-TestMintTokensOf_Local
-├── when no ruleset is active
-│   └── it should be able to mint
-├── when minting is disabled in the funding cycle ruleset
-│   └── it should revert
-└── when minting is enabled in the funding cycle ruleset
-    ├── given that there are tokens available to mint
-    │   └── it should be possible to mint
-    ├── given that a data source is configured
-    │   └── it should be able to mint
-    └── given that a data source has permissioned another contract to mint
-        └── it should be able to mint

package/test/trees/JBController/payReservedTokenToTerminal.tree

@@ -1,8 +0,0 @@
-TestExecutePayReservedTokenToTerminal_Local
-├── when caller is not itself
-│   └── it will revert
-└── when caller is itself
-    ├── given allowance EQ zero after pay
-    │   └── it will not revert
-    └── given allowance DNEQ zero after pay
-        └── it will revert

package/test/trees/JBController/receiveMigrationFrom.tree

@@ -1,4 +0,0 @@
-TestReceiveMigrationFrom
-└── when caller supports the correct interface
-    └── given that the caller is also controllerOf projectId
-        └── it should set metadata

package/test/trees/JBController/sendReservedTokensToSplitsOf.tree

@@ -1,12 +0,0 @@
-TestSendReservedTokensToSplitsOf_Local
-├── when the project has no reserved token splits
-│   └── it will mint to the project owner
-└── when the project has reserved token splits
-    ├── given a hook is configured
-    │   └── it will mint to hook and call its processSplitWith function
-    ├── given a beneficiary is configured
-    │   └── it will mint for the beneficiary
-    ├── given the projectId of split is nonzero and a beneficiary and hook are not configured
-    │   └── it will mint to the owner of the project
-    └── given projectId is zero and nothing is configured
-        └── it will mint to whoever called sendReservedTokens

package/test/trees/JBController/setMetadataOf.tree

@@ -1,5 +0,0 @@
-TestSetMetadataOf_Local
-├── when caller is owner or has permission
-│   └── it should set metadata and emit SetMetadata
-└── when caller is not owner or has permission
-    └── it should revert

package/test/trees/JBController/setSplitGroupsOf.tree

@@ -1,5 +0,0 @@
-TestSetSplitGroupsOf_Local
-├── when caller is permissioned
-│   └── it will set splits
-└── when caller is not permissioned
-    └── it will revert UNAUTHORIZED

package/test/trees/JBController/setTokenFor.tree

@@ -1,5 +0,0 @@
-TestSetTokenFor_Local
-├── when caller is permissioned
-│   └── it will set token
-└── when caller is not permissioned
-    └── it will revert UNAUTHORIZED

package/test/trees/JBController/transferCreditsFrom.tree

@@ -1,8 +0,0 @@
-TestTransferCreditsFrom_Local
-├── when caller is permissioned
-│   ├── given ruleset allows credit transfers
-│   │   └── it will call JBTokens to transfer the credits
-│   └── given ruleset does not allow credit transfers
-│       └── it will revert CREDIT_TRANSFERS_PAUSED
-└── when caller is not permissioned
-    └── it will revert UNAUTHORIZED

package/test/trees/JBDirectory/primaryTerminalOf.tree

@@ -1,8 +0,0 @@
-TestPrimaryTerminalOf_Local
-├── when there are no terminals supporting the input token
-│   └── it will return the zero address
-└── when there is at least one supporting terminal
-    ├── given there are multiple terminals with proper accountingContextForTokenOf
-    │   └── it will check each terminals accountingContextForTokenOf and return the first
-    └── given there are multiple terminals but none with the proper accountingContextForTokenOf
-        └── it will return the zero address

package/test/trees/JBDirectory/setControllerOf.tree

@@ -1,11 +0,0 @@
-TestSetControllerOf_Local
-├── when caller does not have any permission
-│   └── it should revert
-└── when caller isAllowedToSetFirstController or has permission
-    ├── given that a project doesn't exist
-    │   └── it should revert
-    └── given that the project exists
-        ├── given that the current controller is not setControllerAllowed
-        │   └── it should revert
-        └── given that the current controller is setControllerAllowed
-            └── it should set controllerOf and emit SetController

package/test/trees/JBDirectory/setPrimaryTerminalOf.tree

@@ -1,15 +0,0 @@
-TestSetPrimaryTerminalOf_Local
-├── when caller has no permission
-│   └── it should revert with UNAUTHORIZED()
-└── when caller has permission
-    ├── given that there is no accountingContextForTokenOf
-    │   └── it should revert with TOKEN_NOT_ACCEPTED
-    └── given that there is an accountingContextForTokenOf
-        ├── given that the terminal has already been added
-        │   └── it should not add the terminal
-        └── given that the terminal has not been added
-            ├── given that the project is not allowed to set terminals
-            │   └── it should revert with SET_TERMINALS_NOT_ALLOWED
-            └── given that the project is allowed to set terminals
-                ├── it should set the terminal and emit AddTerminal
-                └── it should set the terminal as primary and emit SetPrimaryTerminal

package/test/trees/JBDirectory/setTerminalsOf.tree

@@ -1,11 +0,0 @@
-TestSetTerminalsOf_Local
-├── when caller has no permission
-│   └── it should revert with UNAUTHORIZED()
-└── when caller has permission
-    ├── given that not setTerminalsAllowed
-    │   └── it should revert with revert SET_TERMINALS_NOT_ALLOWED()
-    └── given that setTerminalsAllowed
-        ├── given that duplicate terminals were added
-        │   └── it should revert with DUPLICATE_TERMINALS()
-        └── given that duplicate terminals were not added
-            └── it should set terminals and emit SetTerminals

package/test/trees/JBERC20/initialize.tree

@@ -1,7 +0,0 @@
-TestInitialize_Local
-├── when a name is already set
-│   └── it will revert
-├── when name_ EQ nothing
-│   └── it will revert
-└── when name is valid and not already set
-    └── it will set the name and symbol and transfer ownership

package/test/trees/JBERC20/name.tree

@@ -1,5 +0,0 @@
-TestName_Local
-├── When a name is set
-│   └── it will return the name
-└── When a name is not set
-    └── it will return an empty string

package/test/trees/JBERC20/nonces.tree

@@ -1,5 +0,0 @@
-TestNonces_Local
-├── When a user has not called permit
-│   └── it will return zero
-└── When a user has called permit
-    └── it will return a nonce GT zero

package/test/trees/JBERC20/symbol.tree

@@ -1,5 +0,0 @@
-TestSymbol_Local
-├── When a symbol is set
-│   └── it will return a non-empty string
-└── When a symbol is not set
-    └── it will return an empty string

package/test/trees/JBFeelessAddresses/setFeelessAddress.tree

@@ -1,5 +0,0 @@
-TestSetFeelessAddress_Local
-├── when caller is owner
-│   └── it should set isFeeless and emit SetFeelessAddress
-└── when caller is not owner
-    └── it should revert

package/test/trees/JBFeelessAddresses/supportsInterface.tree

@@ -1,5 +0,0 @@
-TestSupportsInterface_Local
-├── when it supports either IJBFeelessAddresses or IERC165
-│   └── it should return true
-└── when it doesnt support either IJBFeelessAddresses or IERC165
-    └── it should return false

package/test/trees/JBFundAccessLimits/payoutLimitOf.tree

@@ -1,5 +0,0 @@
-TestPayoutLimitOf_Local
-├── when the project has the specific payout limit
-│   └── it will return the uint256 payoutLimit
-└── when the project doesnt have the specific payout limit
-    └── it will return 0

package/test/trees/JBFundAccessLimits/payoutLimitsOf.tree

@@ -1,8 +0,0 @@
-TestPayoutLimitsOf_Local
-└── when a project has payout limits
-    ├── given they are configured for a specific token
-    │   └── it will return them
-    ├── given they are configured for a specific terminal
-    │   └── it will return them
-    └── given they are configured for a specific rulesetId
-        └── it will return them

package/test/trees/JBFundAccessLimits/setFundAccessLimitsFor.tree

@@ -1,18 +0,0 @@
-TestSetFundAccessLimitsFor_Local
-├── when caller is not controller
-│   └── it will revert
-└── when caller is controller of project
-    ├── given payoutLimit amount is gt uint224
-    │   └── it will revert INVALID_PAYOUT_LIMIT
-    ├── given payoutLimit currency is gt uint32
-    │   └── it will revert INVALID_PAYOUT_LIMIT_CURRENCY
-    ├── given payoutLimit currency is not given in ascending order
-    │   └── it will revert INVALID_PAYOUT_LIMIT_CURRENCY_ORDERING
-    ├── given surplusAllowance amount gt uint224
-    │   └── it will revert INVALID_SURPLUS_ALLOWANCE
-    ├── given surplusAllowance currency gt uint32
-    │   └── it will revert INVALID_PAYOUT_LIMIT_CURRENCY
-    ├── given surplus allowance currencies are not ascending order
-    │   └── it will revert INVALID_SURPLUS_ALLOWANCE_CURRENCY_ORDERING
-    └── given valid config
-        └── it will set packed properties and emit SetFundAccessLimits

package/test/trees/JBFundAccessLimits/surplusAllowanceOf.tree

@@ -1,5 +0,0 @@
-TestSurplusAllowanceOf_Local
-├── when a project has the specific surplus configured
-│   └── it will return uin256 surplusAllowance
-└── when it doesnt have the specific surplus configured
-    └── it will return 0

package/test/trees/JBFundAccessLimits/surplusAllowancesOf.tree

@@ -1,8 +0,0 @@
-TestSurplusAllowancesOf_Local
-└── when a project has specified surplus allowances
-    ├── given they are specified for a specific ruleset
-    │   └── it will return them
-    ├── given they are specified for a specific terminal
-    │   └── it will return them
-    └── given they are specified for a specific token
-        └── it will return them

package/test/trees/JBMetadataResolver/getDataFor.tree

@@ -1,8 +0,0 @@
-TestGetDataFor_Local
-├── when metadata length LTEQ MIN_METADATA_LENGTH
-│   └── it will return false and empty bytes
-└── when metadata length GT MIN_METADATA_LENGTH
-    ├── given id is found
-    │   └── it will return found EQ true and targetData
-    └── given id is not found
-        └── it will return found EQ false and empty bytes targetData

package/test/trees/JBMultiTerminal/accountingContextsOf.tree

@@ -1,5 +0,0 @@
-TestAccountingContextsFor_Local
-├── when accounting contexts are set
-│   └── it will return contexts
-└── when accounting contexts are not set
-    └── it will return an empty array

package/test/trees/JBMultiTerminal/addAccountingContextsFor.tree

@@ -1,10 +0,0 @@
-TestAddAccountingContextsFor_Local
-├── when caller is not permissioned
-│   └── it will revert UNAUTHORIZED
-├── when caller is permissioned
-│   ├── given the context is already set
-│   │   └── it will revert ACCOUNTING_CONTEXT_ALREADY_SET
-│   └── given happypath
-│       └── it will set the context and emit SetAccountingContext
-└── when caller is controller
-    └── it will alsoGrantAccess

package/test/trees/JBMultiTerminal/addToBalanceOf.tree

@@ -1,23 +0,0 @@
-TestAddToBalanceOf_Local
-├── when shouldReturnHeldFees eq true
-│   ├── given return amount is zero
-│   │   └── it will set heldFeesOf project to zero
-│   ├── given return amount is non-zero and leftoverAmount GTEQ amountFromFee
-│   │   └── it will return feeAmountResultingIn
-│   └── given return amount is non-zero and leftoverAmount LT amountFromFee
-│       └── it will set heldFeesOf return feeAmountFrom and set leftoverAmount to zero
-├── when shouldReturnHeldFees eq false
-│   └── it will call terminalstore recordAddedBalanceFor and emit AddToBalance
-├── when the project DNH accounting context for the token
-│   └── it will revert TOKEN_NOT_ACCEPTED
-├── when the terminals token eq native token
-│   └── it will use msg.value
-├── when the terminals token eq native token and msgvalue eq zero
-│   └── it will revert NO_MSG_VALUE_ALLOWED
-├── when the terminal is calling itself
-│   └── it will not transfer
-└── when pay metadata contains permit data
-    ├── given the permit allowance lt amount
-    │   └── it will revert PERMIT_ALLOWANCE_NOT_ENOUGH
-    └── given permit allowance is good
-        └── it will set permit allowance to spend tokens for user via permit2