Skip to content

Commit 029ee5f

Browse files
committed
Minor fixes:
- fix incorrect type of periodDuration in native-token-periodic-permission was Hex, now number - removed unnecessary type assertions - add necessary type assertion to test
1 parent f64d325 commit 029ee5f

File tree

2 files changed

+9
-11
lines changed

2 files changed

+9
-11
lines changed

packages/gator-permissions-controller/src/decodePermission/decodePermission.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ describe('decodePermission', () => {
274274
const contractsWithoutTimestampEnforcer = {
275275
...contracts,
276276
TimestampEnforcer: undefined,
277-
} as const;
277+
} as unknown as DeployedContractsByName;
278278

279279
expect(() =>
280280
identifyPermissionByEnforcers({
@@ -509,7 +509,7 @@ describe('decodePermission', () => {
509509

510510
expect(expiry).toBe(timestampBeforeThreshold);
511511
expect(hexToBigInt(data.periodAmount)).toBe(periodAmount);
512-
expect(hexToNumber(data.periodDuration)).toBe(periodDuration);
512+
expect(data.periodDuration).toBe(periodDuration);
513513
expect(data.startTime).toBe(startDate);
514514
});
515515

packages/gator-permissions-controller/src/decodePermission/decodePermission.ts

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ export const getPermissionDataAndExpiry = ({
152152
maxAmount,
153153
amountPerSecond,
154154
startTime: hexToNumber(startTimeRaw),
155-
} as unknown as DecodedPermission['permission']['data'];
155+
};
156156
break;
157157
}
158158
case 'erc20-token-periodic': {
@@ -164,14 +164,12 @@ export const getPermissionDataAndExpiry = ({
164164
const [tokenAddress, periodAmount, periodDurationRaw, startTimeRaw] =
165165
splitHex(erc20PeriodicTerms, [20, 32, 32, 32]);
166166

167-
const periodDuration = hexToNumber(periodDurationRaw);
168-
169167
data = {
170168
tokenAddress,
171169
periodAmount,
172-
periodDuration,
170+
periodDuration: hexToNumber(periodDurationRaw),
173171
startTime: hexToNumber(startTimeRaw),
174-
} as unknown as DecodedPermission['permission']['data'];
172+
};
175173
break;
176174
}
177175

@@ -189,7 +187,7 @@ export const getPermissionDataAndExpiry = ({
189187
maxAmount,
190188
amountPerSecond,
191189
startTime: hexToNumber(startTimeRaw),
192-
} as unknown as DecodedPermission['permission']['data'];
190+
};
193191
break;
194192
}
195193
case 'native-token-periodic': {
@@ -198,16 +196,16 @@ export const getPermissionDataAndExpiry = ({
198196
nativeTokenPeriodicEnforcer,
199197
);
200198

201-
const [periodAmount, periodDuration, startTimeRaw] = splitHex(
199+
const [periodAmount, periodDurationRaw, startTimeRaw] = splitHex(
202200
nativeTokenPeriodicTerms,
203201
[32, 32, 32],
204202
);
205203

206204
data = {
207205
periodAmount,
208-
periodDuration,
206+
periodDuration: hexToNumber(periodDurationRaw),
209207
startTime: hexToNumber(startTimeRaw),
210-
} as unknown as DecodedPermission['permission']['data'];
208+
};
211209
break;
212210
}
213211
default:

0 commit comments

Comments
 (0)