Skip to content

Commit 4d287e8

Browse files
authored
Patched with latest catbuffer and openAPI (#576)
* Fixed #575 - Patched with latest catbuffer and openAPI * Fixed typo
1 parent 2b78872 commit 4d287e8

File tree

11 files changed

+33
-33
lines changed

11 files changed

+33
-33
lines changed

e2e/infrastructure/TransactionHttp.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -596,7 +596,7 @@ describe('TransactionHttp', () => {
596596
const signedTransaction = accountLinkTransaction.signWith(account, generationHash);
597597

598598
return helper.announce(signedTransaction).then((transaction: AccountKeyLinkTransaction) => {
599-
expect(transaction.remotePublicKey, 'RemotePublicKey').not.to.be.undefined;
599+
expect(transaction.linkedPublicKey, 'linkedPublicKey').not.to.be.undefined;
600600
expect(transaction.linkAction, 'LinkAction').not.to.be.undefined;
601601
return signedTransaction;
602602
});

package-lock.json

Lines changed: 9 additions & 9 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@
7373
},
7474
"dependencies": {
7575
"bluebird": "^3.7.2",
76-
"catbuffer-typescript": "0.0.18",
76+
"catbuffer-typescript": "0.0.19",
7777
"crypto-js": "^4.0.0",
7878
"diff": "^4.0.2",
7979
"futoin-hkdf": "^1.3.1",
@@ -89,7 +89,7 @@
8989
"ripemd160": "^2.0.2",
9090
"rxjs": "^6.5.3",
9191
"rxjs-compat": "^6.5.3",
92-
"symbol-openapi-typescript-node-client": "0.8.10",
92+
"symbol-openapi-typescript-node-client": "0.8.11",
9393
"tweetnacl": "^1.0.3",
9494
"utf8": "^3.0.0",
9595
"ws": "^7.2.3"

src/infrastructure/transaction/CreateTransactionFromDTO.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,7 @@ const CreateStandaloneTransactionFromDTO = (transactionDTO, transactionInfo): Tr
349349
transactionDTO.version,
350350
Deadline.createFromDTO(transactionDTO.deadline),
351351
UInt64.fromNumericString(transactionDTO.maxFee || '0'),
352-
transactionDTO.remotePublicKey,
352+
transactionDTO.linkedPublicKey,
353353
transactionDTO.linkAction,
354354
transactionDTO.signature,
355355
transactionDTO.signerPublicKey

src/infrastructure/transaction/SerializeTransactionToJSON.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ export const SerializeTransactionToJSON = (transaction: Transaction): any => {
5252
case TransactionType.ACCOUNT_KEY_LINK:
5353
const accountLinkTx = transaction as AccountKeyLinkTransaction;
5454
return {
55-
remotePublicKey: accountLinkTx.remotePublicKey,
55+
linkedPublicKey: accountLinkTx.linkedPublicKey,
5656
linkAction: accountLinkTx.linkAction,
5757
};
5858
case TransactionType.ADDRESS_ALIAS:

src/model/transaction/AccountKeyLinkTransaction.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export class AccountKeyLinkTransaction extends Transaction {
4444
/**
4545
* Create a link account transaction object
4646
* @param deadline - The deadline to include the transaction.
47-
* @param remotePublicKey - The public key of the remote account.
47+
* @param linkedPublicKey - The public key of the remote account.
4848
* @param linkAction - The account link action.
4949
* @param maxFee - (Optional) Max fee defined by the sender
5050
* @param signature - (Optional) Transaction signature
@@ -53,7 +53,7 @@ export class AccountKeyLinkTransaction extends Transaction {
5353
*/
5454
public static create(
5555
deadline: Deadline,
56-
remotePublicKey: string,
56+
linkedPublicKey: string,
5757
linkAction: LinkAction,
5858
networkType: NetworkType,
5959
maxFee: UInt64 = new UInt64([0, 0]),
@@ -65,7 +65,7 @@ export class AccountKeyLinkTransaction extends Transaction {
6565
TransactionVersion.ACCOUNT_KEY_LINK,
6666
deadline,
6767
maxFee,
68-
remotePublicKey,
68+
linkedPublicKey,
6969
linkAction,
7070
signature,
7171
signer,
@@ -77,7 +77,7 @@ export class AccountKeyLinkTransaction extends Transaction {
7777
* @param version
7878
* @param deadline
7979
* @param maxFee
80-
* @param remotePublicKey
80+
* @param linkedPublicKey
8181
* @param linkAction
8282
* @param signature
8383
* @param signer
@@ -91,7 +91,7 @@ export class AccountKeyLinkTransaction extends Transaction {
9191
/**
9292
* The public key of the remote account.
9393
*/
94-
public readonly remotePublicKey: string,
94+
public readonly linkedPublicKey: string,
9595
/**
9696
* The account link action.
9797
*/
@@ -118,7 +118,7 @@ export class AccountKeyLinkTransaction extends Transaction {
118118
const signature = payload.substring(16, 144);
119119
const transaction = AccountKeyLinkTransaction.create(
120120
isEmbedded ? Deadline.create() : Deadline.createFromDTO((builder as AccountKeyLinkTransactionBuilder).getDeadline().timestamp),
121-
Convert.uint8ToHex(builder.getRemotePublicKey().key),
121+
Convert.uint8ToHex(builder.getLinkedPublicKey().key),
122122
builder.getLinkAction().valueOf(),
123123
networkType,
124124
isEmbedded ? new UInt64([0, 0]) : new UInt64((builder as AccountKeyLinkTransactionBuilder).fee.amount),
@@ -160,7 +160,7 @@ export class AccountKeyLinkTransaction extends Transaction {
160160
TransactionType.ACCOUNT_KEY_LINK.valueOf(),
161161
new AmountDto(this.maxFee.toDTO()),
162162
new TimestampDto(this.deadline.toDTO()),
163-
new KeyDto(Convert.hexToUint8(this.remotePublicKey)),
163+
new KeyDto(Convert.hexToUint8(this.linkedPublicKey)),
164164
this.linkAction.valueOf(),
165165
);
166166
return transactionBuilder.serialize();
@@ -176,7 +176,7 @@ export class AccountKeyLinkTransaction extends Transaction {
176176
this.versionToDTO(),
177177
this.networkType.valueOf(),
178178
TransactionType.ACCOUNT_KEY_LINK.valueOf(),
179-
new KeyDto(Convert.hexToUint8(this.remotePublicKey)),
179+
new KeyDto(Convert.hexToUint8(this.linkedPublicKey)),
180180
this.linkAction.valueOf(),
181181
);
182182
}
@@ -196,6 +196,6 @@ export class AccountKeyLinkTransaction extends Transaction {
196196
* @returns {boolean}
197197
*/
198198
public shouldNotifyAccount(address: Address): boolean {
199-
return super.isSigned(address) || Address.createFromPublicKey(this.remotePublicKey, this.networkType).equals(address);
199+
return super.isSigned(address) || Address.createFromPublicKey(this.linkedPublicKey, this.networkType).equals(address);
200200
}
201201
}

test/core/utils/TransactionMapping.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -571,7 +571,7 @@ describe('TransactionMapping - createFromPayload', () => {
571571
const transaction = TransactionMapping.createFromPayload(signedTransaction.payload) as AccountKeyLinkTransaction;
572572

573573
expect(transaction.linkAction).to.be.equal(1);
574-
expect(transaction.remotePublicKey).to.be.equal(account.publicKey);
574+
expect(transaction.linkedPublicKey).to.be.equal(account.publicKey);
575575
});
576576

577577
it('should create an VrfKeyLinkTransaction object with link action', () => {
@@ -867,7 +867,7 @@ describe('TransactionMapping - createFromDTO (Transaction.toJSON() feed)', () =>
867867

868868
const transaction = TransactionMapping.createFromDTO(accountLinkTransaction.toJSON()) as AccountKeyLinkTransaction;
869869

870-
expect(transaction.remotePublicKey).to.be.equal(account.publicKey);
870+
expect(transaction.linkedPublicKey).to.be.equal(account.publicKey);
871871
expect(transaction.linkAction).to.be.equal(LinkAction.Link);
872872
});
873873

test/core/utils/TransactionMappingWithSignatures.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -691,7 +691,7 @@ describe('TransactionMapping - createFromPayload with optional sigature and sign
691691
let transaction = TransactionMapping.createFromPayload(signedTransaction) as AccountKeyLinkTransaction;
692692

693693
expect(transaction.linkAction).to.be.equal(1);
694-
expect(transaction.remotePublicKey).to.be.equal(account.publicKey);
694+
expect(transaction.linkedPublicKey).to.be.equal(account.publicKey);
695695
expect(transaction.signature).to.be.equal(testSignature);
696696
expect(transaction.signer?.publicKey).to.be.equal(account.publicKey);
697697

test/infrastructure/SerializeTransactionToJSON.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ describe('SerializeTransactionToJSON', () => {
7171

7272
const json = accountLinkTransaction.toJSON();
7373

74-
expect(json.transaction.remotePublicKey).to.be.equal(account.publicKey);
74+
expect(json.transaction.linkedPublicKey).to.be.equal(account.publicKey);
7575
expect(json.transaction.linkAction).to.be.equal(LinkAction.Link);
7676
});
7777

test/infrastructure/TransactionHttp.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ describe('TransactionHttp', () => {
7676
it('Test getTransactionStatus method', async () => {
7777
const hash = 'abc';
7878
const transactionStatusDTO = new TransactionStatusDTO();
79-
transactionStatusDTO.code = TransactionStatusTypeEnum.FailureAccountLinkInvalidAction;
79+
transactionStatusDTO.code = TransactionStatusTypeEnum.FailureAccountLinkInconsistentUnlinkData;
8080
transactionStatusDTO.deadline = '1234';
8181
transactionStatusDTO.hash = hash;
8282
transactionStatusDTO.group = TransactionStateTypeEnum.Failed;
@@ -90,14 +90,14 @@ describe('TransactionHttp', () => {
9090

9191
expect(transactionStatus.deadline.toString()).to.be.equal('1234');
9292
expect(transactionStatus.hash).to.be.equal(hash);
93-
expect(transactionStatus.code).to.be.equal('Failure_AccountLink_Invalid_Action');
93+
expect(transactionStatus.code).to.be.equal('Failure_AccountLink_Inconsistent_Unlink_Data');
9494
expect(transactionStatus.group).to.be.equal('failed');
9595
});
9696

9797
it('Test getTransactionsStatuses method', async () => {
9898
const hash = 'abc';
9999
const transactionStatusDTO = new TransactionStatusDTO();
100-
transactionStatusDTO.code = TransactionStatusTypeEnum.FailureAccountLinkInvalidAction;
100+
transactionStatusDTO.code = TransactionStatusTypeEnum.FailureAccountLinkInconsistentUnlinkData;
101101
transactionStatusDTO.deadline = '1234';
102102
transactionStatusDTO.hash = hash;
103103
transactionStatusDTO.group = TransactionStateTypeEnum.Failed;
@@ -112,7 +112,7 @@ describe('TransactionHttp', () => {
112112

113113
expect(transactionStatus.deadline.toString()).to.be.equal('1234');
114114
expect(transactionStatus.hash).to.be.equal(hash);
115-
expect(transactionStatus.code).to.be.equal('Failure_AccountLink_Invalid_Action');
115+
expect(transactionStatus.code).to.be.equal('Failure_AccountLink_Inconsistent_Unlink_Data');
116116
expect(transactionStatus.group).to.be.equal('failed');
117117
});
118118
});

0 commit comments

Comments
 (0)