Skip to content

Commit 1599ec5

Browse files
committed
fix: remove Validation of coin/subnet while building move stake transaction
Ticket: SC-3247
1 parent 04e70e4 commit 1599ec5

File tree

2 files changed

+8
-29
lines changed

2 files changed

+8
-29
lines changed

modules/sdk-coin-tao/src/lib/moveStakeTransaction.ts

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { Interface as SubstrateInterface, Transaction as SubstrateTransaction } from '@bitgo/abstract-substrate';
2-
import { InvalidTransactionError, TransactionRecipient } from '@bitgo/sdk-core';
2+
import { InvalidTransactionError } from '@bitgo/sdk-core';
33
import { decode } from '@substrate/txwrapper-polkadot';
44
import { MoveStakeTxData } from './iface';
55
import utils from './utils';
@@ -42,33 +42,24 @@ export class MoveStakeTransaction extends SubstrateTransaction {
4242
this._inputs.push({
4343
address: txMethod.originHotkey,
4444
value: txMethod.alphaAmount,
45-
coin: utils.getTaoTokenBySubnetId(txMethod.originNetuid).name,
4645
});
4746

4847
this._outputs.push({
4948
address: txMethod.destinationHotkey,
5049
value: txMethod.alphaAmount,
51-
coin: utils.getTaoTokenBySubnetId(txMethod.destinationNetuid).name,
5250
});
5351
}
5452

5553
/** @inheritdoc */
5654
explainTransaction(): SubstrateInterface.TransactionExplanation {
57-
const result = this.toJson();
58-
const outputs: TransactionRecipient[] = this._outputs.map((output) => {
59-
return {
60-
address: output.address,
61-
amount: output.value,
62-
tokenName: output.coin,
63-
};
64-
});
55+
const result = this.toJson() as MoveStakeTxData;
6556

6657
const explanationResult: SubstrateInterface.TransactionExplanation = {
6758
id: result.id,
68-
outputAmount: result.amount?.toString() || '0',
59+
outputAmount: result.alphaAmount?.toString() || '0',
6960
changeAmount: '0',
7061
changeOutputs: [],
71-
outputs,
62+
outputs: [],
7263
fee: {
7364
fee: result.tip?.toString() || '',
7465
type: 'tip',

modules/sdk-coin-tao/test/unit/transactionBuilder/moveStakeBuilder.ts

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -107,13 +107,7 @@ describe('Tao Move Stake Builder', function () {
107107

108108
// Verify transaction explanation
109109
const explanation = tx.explainTransaction();
110-
explanation.should.have.properties(['outputs', 'outputAmount', 'changeAmount', 'fee']);
111-
explanation.outputs.should.have.length(1);
112-
explanation.outputs[0].should.deepEqual({
113-
address: '5Ffp1wJCPu4hzVDTo7XaMLqZSvSadyUQmxWPDw74CBjECSoq',
114-
amount: '9007199254740995',
115-
tokenName: utils.getTaoTokenBySubnetId('1').name,
116-
});
110+
explanation.should.have.properties(['outputAmount', 'changeAmount', 'fee']);
117111
});
118112

119113
it('should validate required fields', function () {
@@ -154,9 +148,6 @@ describe('Tao Move Stake Builder', function () {
154148
txJson.originNetuid.should.equal('1');
155149
txJson.destinationNetuid.should.equal('2');
156150
txJson.alphaAmount.should.equal('1000000000000');
157-
158-
const explanation = tx.explainTransaction();
159-
explanation.outputs[0].tokenName.should.equal('ttao:onion');
160151
});
161152
});
162153

@@ -468,11 +459,8 @@ describe('Tao Move Stake Builder', function () {
468459
const tx = await builder.build();
469460
const explanation = tx.explainTransaction();
470461

471-
explanation.should.have.properties(['outputs', 'outputAmount', 'changeAmount', 'fee', 'type']);
472-
explanation.outputs.should.have.length(1);
473-
explanation.outputs[0].should.have.properties(['address', 'amount', 'tokenName']);
474-
explanation.outputs[0].address.should.equal('5Ffp1wJCPu4hzVDTo7XaMLqZSvSadyUQmxWPDw74CBjECSoq');
475-
explanation.outputs[0].amount.should.equal('1000000000000');
462+
explanation.should.have.properties(['outputAmount', 'changeAmount', 'fee', 'type']);
463+
explanation.outputAmount.should.equal('1000000000000');
476464
explanation.changeAmount.should.equal('0');
477465
explanation.fee.should.have.properties(['fee', 'type']);
478466
explanation.fee.type.should.equal('tip');
@@ -495,7 +483,7 @@ describe('Tao Move Stake Builder', function () {
495483
const explanation = tx.explainTransaction();
496484

497485
explanation.fee.fee.should.equal('0');
498-
explanation.outputAmount.should.equal('0');
486+
explanation.outputAmount.should.equal('500000000');
499487
});
500488
});
501489

0 commit comments

Comments
 (0)