Skip to content

Commit 4ac774c

Browse files
committed
Fixed transactionToJson
Fixed unit tests
1 parent 46274f5 commit 4ac774c

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

src/infrastructure/transaction/SerializeTransactionToJSON.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ export const SerializeTransactionToJSON = (transaction: Transaction): any => {
191191
scopedMetadataKey: (transaction as AccountMetadataTransaction).scopedMetadataKey.toString(),
192192
valueSizeDelta: (transaction as AccountMetadataTransaction).valueSizeDelta,
193193
valueSize: (transaction as AccountMetadataTransaction).value.length,
194-
value: Convert.uint8ToHex((transaction as AccountMetadataTransaction).value),
194+
value: (transaction as AccountMetadataTransaction).value,
195195

196196
};
197197
case TransactionType.MOSAIC_METADATA_TRANSACTION:
@@ -201,7 +201,7 @@ export const SerializeTransactionToJSON = (transaction: Transaction): any => {
201201
valueSizeDelta: (transaction as MosaicMetadataTransaction).valueSizeDelta,
202202
targetMosaicId: (transaction as MosaicMetadataTransaction).targetMosaicId.id.toHex(),
203203
valueSize: (transaction as MosaicMetadataTransaction).value.length,
204-
value: Convert.uint8ToHex((transaction as MosaicMetadataTransaction).value),
204+
value: (transaction as MosaicMetadataTransaction).value,
205205

206206
};
207207
case TransactionType.NAMESPACE_METADATA_TRANSACTION:
@@ -211,7 +211,7 @@ export const SerializeTransactionToJSON = (transaction: Transaction): any => {
211211
valueSizeDelta: (transaction as NamespaceMetadataTransaction).valueSizeDelta,
212212
targetNamespaceId: (transaction as NamespaceMetadataTransaction).targetNamespaceId.id.toHex(),
213213
valueSize: (transaction as NamespaceMetadataTransaction).value.length,
214-
value: Convert.uint8ToHex((transaction as NamespaceMetadataTransaction).value),
214+
value: (transaction as NamespaceMetadataTransaction).value,
215215

216216
};
217217
default:

test/core/utils/TransactionMapping.spec.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1067,7 +1067,7 @@ describe('TransactionMapping - createFromDTO (Transaction.toJSON() feed)', () =>
10671067
account.publicKey,
10681068
UInt64.fromUint(1000),
10691069
1,
1070-
Convert.uint8ToUtf8(new Uint8Array(10)),
1070+
'Test Value',
10711071
NetworkType.MIJIN_TEST,
10721072
);
10731073

@@ -1078,7 +1078,7 @@ describe('TransactionMapping - createFromDTO (Transaction.toJSON() feed)', () =>
10781078
expect(transaction.targetPublicKey).to.be.equal(account.publicKey);
10791079
expect(transaction.scopedMetadataKey.toHex()).to.be.equal(UInt64.fromUint(1000).toHex());
10801080
expect(transaction.valueSizeDelta).to.be.equal(1);
1081-
expect(Convert.uint8ToHex(transaction.value)).to.be.equal(Convert.uint8ToHex(new Uint8Array(10)));
1081+
expect(transaction.value).to.be.equal('Test Value');
10821082
});
10831083

10841084
it('should create MosaicMetadataTransaction', () => {
@@ -1088,7 +1088,7 @@ describe('TransactionMapping - createFromDTO (Transaction.toJSON() feed)', () =>
10881088
UInt64.fromUint(1000),
10891089
new MosaicId([2262289484, 3405110546]),
10901090
1,
1091-
Convert.uint8ToUtf8(new Uint8Array(10)),
1091+
'Test Value',
10921092
NetworkType.MIJIN_TEST,
10931093
);
10941094

@@ -1100,7 +1100,7 @@ describe('TransactionMapping - createFromDTO (Transaction.toJSON() feed)', () =>
11001100
expect(transaction.scopedMetadataKey.toHex()).to.be.equal(UInt64.fromUint(1000).toHex());
11011101
expect(transaction.valueSizeDelta).to.be.equal(1);
11021102
expect(transaction.targetMosaicId.toHex()).to.be.equal(new MosaicId([2262289484, 3405110546]).toHex());
1103-
expect(Convert.uint8ToHex(transaction.value)).to.be.equal(Convert.uint8ToHex(new Uint8Array(10)));
1103+
expect(transaction.value).to.be.equal('Test Value');
11041104
});
11051105

11061106
it('should create NamespaceMetadataTransaction', () => {
@@ -1110,7 +1110,7 @@ describe('TransactionMapping - createFromDTO (Transaction.toJSON() feed)', () =>
11101110
UInt64.fromUint(1000),
11111111
new NamespaceId([2262289484, 3405110546]),
11121112
1,
1113-
Convert.uint8ToUtf8(new Uint8Array(10)),
1113+
'Test Value',
11141114
NetworkType.MIJIN_TEST,
11151115
);
11161116

@@ -1122,6 +1122,6 @@ describe('TransactionMapping - createFromDTO (Transaction.toJSON() feed)', () =>
11221122
expect(transaction.scopedMetadataKey.toString()).to.be.equal(UInt64.fromUint(1000).toString());
11231123
expect(transaction.valueSizeDelta).to.be.equal(1);
11241124
expect(transaction.targetNamespaceId.toHex()).to.be.equal(new NamespaceId([2262289484, 3405110546]).toHex());
1125-
expect(Convert.uint8ToHex(transaction.value)).to.be.equal(Convert.uint8ToHex(new Uint8Array(10)));
1125+
expect(transaction.value).to.be.equal('Test Value');
11261126
});
11271127
});

0 commit comments

Comments
 (0)