Skip to content

Commit 00a887e

Browse files
committed
Fixed newBlock bug in listener
1 parent 840ee20 commit 00a887e

File tree

2 files changed

+12
-13
lines changed

2 files changed

+12
-13
lines changed

src/infrastructure/Listener.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -142,17 +142,16 @@ export class Listener {
142142
message: CreateTransactionFromDTO(message),
143143
});
144144
} else if (message.block) {
145-
const networkType = parseInt(message.block.version.toString(16).substr(0, 2), 16);
146145
this.messageSubject.next({
147146
channelName: ListenerChannelName.block, message: new BlockInfo(
148147
message.meta.hash,
149148
message.meta.generationHash,
150149
message.meta.totalFee ? UInt64.fromNumericString(message.meta.totalFee) : new UInt64([0, 0]),
151150
message.meta.numTransactions,
152151
message.block.signature,
153-
PublicAccount.createFromPublicKey(message.block.signerPublicKey, networkType),
154-
networkType,
155-
parseInt(message.block.version.toString(16).substr(2, 2), 16), // Tx version
152+
PublicAccount.createFromPublicKey(message.block.signerPublicKey, message.block.network),
153+
message.block.network,
154+
message.block.version,
156155
message.block.type,
157156
UInt64.fromNumericString(message.block.height),
158157
UInt64.fromNumericString(message.block.timestamp),
@@ -162,7 +161,7 @@ export class Listener {
162161
message.block.blockTransactionsHash,
163162
message.block.blockReceiptsHash,
164163
message.block.stateHash,
165-
extractBeneficiary(message, networkType), // passing `message` as `blockDTO`
164+
extractBeneficiary(message, message.block.network), // passing `message` as `blockDTO`
166165
),
167166
});
168167
} else if (message.status) {

test/model/blockchain/BlockInfo.spec.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,8 @@ describe('BlockInfo', () => {
3838
beneficiaryPublicKey: 'B4F12E7C9F6946091E2CB8B6D3A12B50D17CCBBF646386EA27CE2946A7423DCF',
3939
timestamp: new UInt64([ 0, 0 ]),
4040
type: 32768,
41-
version: 36865,
41+
version: 1,
42+
network: 144,
4243
},
4344
meta: {
4445
generationHash: '57F7DA205008026C776CB6AED843393F04CD458E0AA2D9F1D5F31A402072B2D6',
@@ -48,16 +49,15 @@ describe('BlockInfo', () => {
4849
},
4950
};
5051

51-
const network = parseInt(blockDTO.block.version.toString(16).substr(0, 2), 16);
5252
const blockInfo = new BlockInfo(
5353
blockDTO.meta.hash,
5454
blockDTO.meta.generationHash,
5555
blockDTO.meta.totalFee,
5656
blockDTO.meta.numTransactions,
5757
blockDTO.block.signature,
58-
PublicAccount.createFromPublicKey(blockDTO.block.signerPublicKey, network),
59-
network,
60-
parseInt(blockDTO.block.version.toString(16).substr(2, 2), 16), // Tx version
58+
PublicAccount.createFromPublicKey(blockDTO.block.signerPublicKey, blockDTO.block.network),
59+
blockDTO.block.network,
60+
blockDTO.block.version, // Tx version
6161
blockDTO.block.type,
6262
blockDTO.block.height,
6363
blockDTO.block.timestamp,
@@ -67,7 +67,7 @@ describe('BlockInfo', () => {
6767
blockDTO.block.blockTransactionsHash,
6868
blockDTO.block.blockReceiptsHash,
6969
blockDTO.block.stateHash,
70-
PublicAccount.createFromPublicKey(blockDTO.block.beneficiaryPublicKey, network),
70+
PublicAccount.createFromPublicKey(blockDTO.block.beneficiaryPublicKey, blockDTO.block.network),
7171
);
7272

7373
expect(blockInfo.hash).to.be.equal(blockDTO.meta.hash);
@@ -76,8 +76,8 @@ describe('BlockInfo', () => {
7676
expect(blockInfo.numTransactions).to.be.equal(blockDTO.meta.numTransactions);
7777
expect(blockInfo.signature).to.be.equal(blockDTO.block.signature);
7878
expect(blockInfo.signer.publicKey).to.be.equal(blockDTO.block.signerPublicKey);
79-
expect(blockInfo.networkType).to.be.equal(parseInt(blockDTO.block.version.toString(16).substr(0, 2), 16));
80-
expect(blockInfo.version).to.be.equal(parseInt(blockDTO.block.version.toString(16).substr(2, 2), 16));
79+
expect(blockInfo.networkType).to.be.equal(blockDTO.block.network);
80+
expect(blockInfo.version).to.be.equal(blockDTO.block.version);
8181
expect(blockInfo.type).to.be.equal(blockDTO.block.type);
8282
deepEqual(blockInfo.height, blockDTO.block.height);
8383
deepEqual(blockInfo.timestamp, blockDTO.block.timestamp);

0 commit comments

Comments
 (0)