Skip to content

Commit facfa96

Browse files
authored
Merge pull request #7754 from BitGo/COIN-6943-support-forwarder-version-5
2 parents 0f4d0b9 + 36ffe11 commit facfa96

File tree

4 files changed

+14
-7
lines changed

4 files changed

+14
-7
lines changed

modules/bitgo/test/v2/unit/wallet.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -820,8 +820,8 @@ describe('V2 Wallet:', function () {
820820
// @ts-expect-error checking type mismatch
821821
await wallet.createAddress({ allowSkipVerifyAddress: null }).should.be.rejectedWith(message);
822822

823-
message = 'forwarderVersion has to be an integer 0, 1, 2, 3 or 4';
824-
await wallet.createAddress({ forwarderVersion: 5 }).should.be.rejectedWith(message);
823+
message = 'forwarderVersion has to be an integer 0, 1, 2, 3, 4 or 5';
824+
await wallet.createAddress({ forwarderVersion: 6 }).should.be.rejectedWith(message);
825825
await wallet.createAddress({ forwarderVersion: -1 }).should.be.rejectedWith(message);
826826
});
827827

modules/express/src/typedRoutes/schemas/address.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,13 @@
11
import * as t from 'io-ts';
22

3-
export const ForwarderVersion = t.union([t.literal(0), t.literal(1), t.literal(2), t.literal(3), t.literal(4)]);
3+
export const ForwarderVersion = t.union([
4+
t.literal(0),
5+
t.literal(1),
6+
t.literal(2),
7+
t.literal(3),
8+
t.literal(4),
9+
t.literal(5),
10+
]);
411

512
export const EIP1559 = t.type({
613
maxFeePerGas: t.number,

modules/express/test/unit/typedRoutes/createAddress.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ describe('CreateAddress codec tests', function () {
276276
});
277277

278278
it('should validate body with all forwarder version values', function () {
279-
const forwarderVersions = [0, 1, 2, 3, 4];
279+
const forwarderVersions = [0, 1, 2, 3, 4, 5];
280280

281281
forwarderVersions.forEach((version) => {
282282
const validBody = {
@@ -289,7 +289,7 @@ describe('CreateAddress codec tests', function () {
289289

290290
it('should reject body with invalid forwarder version', function () {
291291
const invalidBody = {
292-
forwarderVersion: 5, // not in union [0,1,2,3,4]
292+
forwarderVersion: 6, // not in union [0,1,2,3,4,5]
293293
};
294294

295295
assert.throws(() => {

modules/sdk-core/src/bitgo/wallet/wallet.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1290,8 +1290,8 @@ export class Wallet implements IWallet {
12901290
}
12911291

12921292
if (!_.isUndefined(forwarderVersion)) {
1293-
if (!_.isInteger(forwarderVersion) || forwarderVersion < 0 || forwarderVersion > 4) {
1294-
throw new Error('forwarderVersion has to be an integer 0, 1, 2, 3 or 4');
1293+
if (!_.isInteger(forwarderVersion) || forwarderVersion < 0 || forwarderVersion > 5) {
1294+
throw new Error('forwarderVersion has to be an integer 0, 1, 2, 3, 4 or 5');
12951295
}
12961296
addressParams.forwarderVersion = forwarderVersion;
12971297
}

0 commit comments

Comments
 (0)