Skip to content

Commit 3ad5707

Browse files
matiasedgeandnodematiasedgeandnode
committed
fix: Horizon run lint:sol
1 parent 3cccb4b commit 3ad5707

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+368
-224
lines changed

‎packages/horizon/test/GraphBase.t.sol‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,12 @@ abstract contract GraphBaseTest is IHorizonStakingTypes, Utils, Constants {
201201
subgraphDataServiceLegacyAddress
202202
);
203203

204-
graphTallyCollector = new GraphTallyCollector("GraphTallyCollector", "1", address(controller), revokeSignerThawingPeriod);
204+
graphTallyCollector = new GraphTallyCollector(
205+
"GraphTallyCollector",
206+
"1",
207+
address(controller),
208+
revokeSignerThawingPeriod
209+
);
205210

206211
resetPrank(users.governor);
207212
proxyAdmin.upgrade(stakingProxy, address(stakingBase));

‎packages/horizon/test/data-service/DataService.t.sol‎

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,10 @@ contract DataServiceTest is HorizonStakingSharedTest {
6565
assertEq(max, dataServiceOverride.PROVISION_TOKENS_MAX());
6666
}
6767

68-
function test_ProvisionTokens_WhenCheckingAValidProvision_WithThawing(uint256 tokens, uint256 tokensThaw) external useIndexer {
68+
function test_ProvisionTokens_WhenCheckingAValidProvision_WithThawing(
69+
uint256 tokens,
70+
uint256 tokensThaw
71+
) external useIndexer {
6972
dataService.setProvisionTokensRange(dataService.PROVISION_TOKENS_MIN(), dataService.PROVISION_TOKENS_MAX());
7073
tokens = bound(tokens, dataService.PROVISION_TOKENS_MIN(), dataService.PROVISION_TOKENS_MAX());
7174
tokensThaw = bound(tokensThaw, tokens - dataService.PROVISION_TOKENS_MIN() + 1, tokens);
@@ -313,12 +316,7 @@ contract DataServiceTest is HorizonStakingSharedTest {
313316
dataService.VERIFIER_CUT_MIN(),
314317
dataService.THAWING_PERIOD_MIN()
315318
);
316-
_setProvisionParameters(
317-
users.indexer,
318-
address(dataService),
319-
dataService.VERIFIER_CUT_MIN(),
320-
thawingPeriod
321-
);
319+
_setProvisionParameters(users.indexer, address(dataService), dataService.VERIFIER_CUT_MIN(), thawingPeriod);
322320

323321
// accept provision parameters
324322
vm.expectRevert(
@@ -351,12 +349,7 @@ contract DataServiceTest is HorizonStakingSharedTest {
351349
dataService.VERIFIER_CUT_MIN(),
352350
dataService.THAWING_PERIOD_MIN()
353351
);
354-
_setProvisionParameters(
355-
users.indexer,
356-
address(dataService),
357-
maxVerifierCut,
358-
dataService.THAWING_PERIOD_MIN()
359-
);
352+
_setProvisionParameters(users.indexer, address(dataService), maxVerifierCut, dataService.THAWING_PERIOD_MIN());
360353

361354
// accept provision parameters
362355
if (maxVerifierCut != dataService.VERIFIER_CUT_MIN()) {

‎packages/horizon/test/data-service/extensions/DataServiceFees.t.sol‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,12 @@ contract DataServiceFeesTest is HorizonStakingSharedTest {
146146

147147
// it should emit a an event
148148
vm.expectEmit();
149-
emit IDataServiceFees.StakeClaimLocked(serviceProvider, calcValues.predictedClaimId, calcValues.stakeToLock, calcValues.unlockTimestamp);
149+
emit IDataServiceFees.StakeClaimLocked(
150+
serviceProvider,
151+
calcValues.predictedClaimId,
152+
calcValues.stakeToLock,
153+
calcValues.unlockTimestamp
154+
);
150155
dataService.lockStake(serviceProvider, tokens);
151156

152157
// after state
@@ -195,7 +200,9 @@ contract DataServiceFeesTest is HorizonStakingSharedTest {
195200
tokensReleased: 0,
196201
head: beforeHead
197202
});
198-
while (calcValues.head != bytes32(0) && (calcValues.claimsCount < numClaimsToRelease || numClaimsToRelease == 0)) {
203+
while (
204+
calcValues.head != bytes32(0) && (calcValues.claimsCount < numClaimsToRelease || numClaimsToRelease == 0)
205+
) {
199206
(uint256 claimTokens, , uint256 releasableAt, bytes32 nextClaim) = dataService.claims(calcValues.head);
200207
if (releasableAt > block.timestamp) {
201208
break;

‎packages/horizon/test/data-service/implementations/DataServiceBase.sol‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,11 @@ contract DataServiceBase is DataService {
2525

2626
function stopService(address serviceProvider, bytes calldata data) external {}
2727

28-
function collect(address serviceProvider, IGraphPayments.PaymentTypes feeType, bytes calldata data) external returns (uint256) {}
28+
function collect(
29+
address serviceProvider,
30+
IGraphPayments.PaymentTypes feeType,
31+
bytes calldata data
32+
) external returns (uint256) {}
2933

3034
function slash(address serviceProvider, bytes calldata data) external {}
3135

‎packages/horizon/test/data-service/implementations/DataServiceBaseUpgradeable.sol‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,11 @@ contract DataServiceBaseUpgradeable is DataService {
2121

2222
function stopService(address serviceProvider, bytes calldata data) external {}
2323

24-
function collect(address serviceProvider, IGraphPayments.PaymentTypes feeType, bytes calldata data) external returns (uint256) {}
24+
function collect(
25+
address serviceProvider,
26+
IGraphPayments.PaymentTypes feeType,
27+
bytes calldata data
28+
) external returns (uint256) {}
2529

2630
function slash(address serviceProvider, bytes calldata data) external {}
2731

‎packages/horizon/test/data-service/implementations/DataServiceImpFees.sol‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,11 @@ contract DataServiceImpFees is DataServiceFees {
2121

2222
function stopService(address serviceProvider, bytes calldata data) external {}
2323

24-
function collect(address serviceProvider, IGraphPayments.PaymentTypes, bytes calldata data) external returns (uint256) {
24+
function collect(
25+
address serviceProvider,
26+
IGraphPayments.PaymentTypes,
27+
bytes calldata data
28+
) external returns (uint256) {
2529
uint256 amount = abi.decode(data, (uint256));
2630
_releaseStake(serviceProvider, 0);
2731
_lockStake(serviceProvider, amount * STAKE_TO_FEES_RATIO, block.timestamp + LOCK_DURATION);

‎packages/horizon/test/data-service/implementations/DataServiceImpPausable.sol‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,11 @@ contract DataServiceImpPausable is DataServicePausable {
2929

3030
function stopService(address serviceProvider, bytes calldata data) external {}
3131

32-
function collect(address serviceProvider, IGraphPayments.PaymentTypes feeType, bytes calldata data) external returns (uint256) {}
32+
function collect(
33+
address serviceProvider,
34+
IGraphPayments.PaymentTypes feeType,
35+
bytes calldata data
36+
) external returns (uint256) {}
3337

3438
function slash(address serviceProvider, bytes calldata data) external {}
3539

‎packages/horizon/test/data-service/implementations/DataServiceImpPausableUpgradeable.sol‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,11 @@ contract DataServiceImpPausableUpgradeable is DataServicePausableUpgradeable {
2323

2424
function stopService(address serviceProvider, bytes calldata data) external {}
2525

26-
function collect(address serviceProvider, IGraphPayments.PaymentTypes feeType, bytes calldata data) external returns (uint256) {}
26+
function collect(
27+
address serviceProvider,
28+
IGraphPayments.PaymentTypes feeType,
29+
bytes calldata data
30+
) external returns (uint256) {}
2731

2832
function slash(address serviceProvider, bytes calldata data) external {}
2933

‎packages/horizon/test/data-service/libraries/ProvisionTracker.t.sol‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,13 +86,19 @@ contract ProvisionTrackerTest is HorizonStakingSharedTest, ProvisionTrackerImple
8686
assertEq(provisionTracker[users.indexer], delta);
8787
}
8888

89-
function test_Release_RevertGiven_TheProvisionHasInsufficientLockedTokens(uint256 tokens) external useIndexer useProvisionDataService(address(this), tokens, 0, 0) {
89+
function test_Release_RevertGiven_TheProvisionHasInsufficientLockedTokens(
90+
uint256 tokens
91+
) external useIndexer useProvisionDataService(address(this), tokens, 0, 0) {
9092
// setup
9193
provisionTracker.lock(staking, users.indexer, tokens, uint32(0));
9294

9395
uint256 tokensToRelease = tokens + 1;
9496
vm.expectRevert(
95-
abi.encodeWithSelector(ProvisionTracker.ProvisionTrackerInsufficientTokens.selector, tokens, tokensToRelease)
97+
abi.encodeWithSelector(
98+
ProvisionTracker.ProvisionTrackerInsufficientTokens.selector,
99+
tokens,
100+
tokensToRelease
101+
)
96102
);
97103
provisionTracker.release(users.indexer, tokensToRelease);
98104
}

‎packages/horizon/test/data-service/libraries/ProvisionTrackerImplementation.sol‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,5 @@ pragma solidity 0.8.27;
44
import { ProvisionTracker } from "../../../contracts/data-service/libraries/ProvisionTracker.sol";
55

66
contract ProvisionTrackerImplementation {
7-
mapping(address => uint256) public provisionTracker;
7+
mapping(address => uint256) public provisionTracker;
88
}

0 commit comments

Comments
 (0)