Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(accountUpdateTransaction): Implement AccountUpdateTransaction E2E tests: TCK #224

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
64 commits
Select commit Hold shift + click to select a range
0415d0e
docs: Update test specification document for AccountCreateTransaction…
rwalworth Apr 23, 2024
4bfc6e9
revert: unintended changes in test_CreateAccount.js
rwalworth Apr 23, 2024
6bf2191
docs: Update test specifications template markdown file to be correct
rwalworth Apr 23, 2024
f58de75
docs: Remove JSONRPC API document for account create as that informat…
rwalworth Apr 24, 2024
5a46641
docs: Update language used to determine if a request is successful. F…
rwalworth Apr 24, 2024
0853ffc
docs: address PR comments
rwalworth Apr 25, 2024
f5e113c
docs: Address PR comments. Keep folder names kebab-case
rwalworth Apr 26, 2024
d665531
docs: address PR comments
rwalworth Apr 29, 2024
a1bc8b3
docs: Add valid value test for max automatic token associations
rwalworth May 6, 2024
ff81c5a
docs: fix typo
rwalworth May 6, 2024
01ea468
docs: fix table formatting
rwalworth May 6, 2024
0bf5231
docs: remove required/optional columns from output parameter tables
rwalworth May 6, 2024
f0a8c76
Update account create transaction tests for key and initial balance
rwalworth May 7, 2024
0893d5b
feat(accountCreateTransaction): Update account create transaction tes…
rwalworth May 8, 2024
7cfaa87
feat(accountCreateTransaction): add remaining tests
rwalworth May 9, 2024
9e1fa95
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 9, 2024
e3b51bf
feat(accountCreateTransaction): remove unnecessary includes and mark …
rwalworth May 9, 2024
1f6b04f
fix: node account ID should contain the entire ID. Numbering issue in…
rwalworth May 15, 2024
add1435
fix: use signerKeys to explicitly sign transactions. Update to correc…
rwalworth May 15, 2024
c5990ee
fix(accountCreateTransaction): fix remaining tests
rwalworth May 16, 2024
0a46162
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 16, 2024
b8dd150
fix: add keyList type to tests as they're necessary according to the …
rwalworth May 20, 2024
f4ed3f8
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 20, 2024
1bcd404
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 20, 2024
3f041ee
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 20, 2024
6a2e81e
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 20, 2024
d54fbed
refactor: add comments, keep consistent between tests
rwalworth May 20, 2024
a6bd6b5
docs: update max value for max auto token associations
rwalworth May 20, 2024
70982a6
fix: update staking account ID and staking node ID test to passing
rwalworth May 22, 2024
cbb3f32
fix: update staked node ID to ID that will work for both testnet and …
rwalworth May 22, 2024
d8c3607
feat(accountUpdateTransaction): create new test file and start implem…
rwalworth May 23, 2024
746838d
feat(accountUpdateTransaction): add tests for Key property
rwalworth May 24, 2024
944085f
feat(accountUpdateTransaction): add auto renew period tests
rwalworth May 28, 2024
e104298
feat(accountUpdateTransaction): finish account update tests
rwalworth May 29, 2024
da36275
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 29, 2024
d429506
fix: update to use commonTransactionParams
rwalworth May 30, 2024
cd7afd8
fix: generatePrivateKey should be generateKey
rwalworth May 31, 2024
d90e545
fix: no longer allowing random types for generateKey
rwalworth May 31, 2024
22c74ab
fix: add explicity key types for all generateKey calls
rwalworth May 31, 2024
21d940a
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth May 31, 2024
5ba42ac
refactor: remove update files that are now in test_accountUpdateTrans…
rwalworth May 31, 2024
898dc2d
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth Jun 3, 2024
fcf33a7
fix: use maxTransactionFee to allow for an account to be created with…
rwalworth Jun 5, 2024
353d096
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth Jun 13, 2024
df4596e
fix: maxTransactionFee wasn't large enough and not documented
rwalworth Jun 18, 2024
3d49c82
Merge branch 'main' into 00217-tck-implement-json-rpc-methods-for-acc…
rwalworth Jun 18, 2024
bed9e3f
Merge branch '00205-implement-accountcreatetransaction-e2e-tests-tck'…
rwalworth Jun 20, 2024
c2f29de
feat: add all accountUpdateTransaction tests
rwalworth Jun 25, 2024
555f8af
feat(memo): add test for invalid memo checking
rwalworth Jul 11, 2024
b47d9c6
fix: update response code for max auto association tests for account …
rwalworth Jul 17, 2024
7fdc5ab
feat: fix typos in code and docs
rwalworth Jul 22, 2024
44d35f4
Merge branch 'main' into 00217-tck-implement-json-rpc-methods-for-acc…
rwalworth Jul 22, 2024
466cc7c
refactor: remove getNodeType
rwalworth Jul 23, 2024
476de2a
Merge branch 'main' into 00205-implement-accountcreatetransaction-e2e…
rwalworth Jul 24, 2024
73138e8
fix: add '0x' to alias bytes and revert unlimited max auto token asso…
rwalworth Jul 29, 2024
2d0f684
revert: undo max auto token associations changes
rwalworth Aug 1, 2024
7a6ae68
Merge branch '00205-implement-accountcreatetransaction-e2e-tests-tck'…
rwalworth Aug 5, 2024
c4e8025
fix: address PR comments
rwalworth Aug 6, 2024
ce78e3f
fix: add more keys to account update key keylist test
rwalworth Aug 6, 2024
27da111
Merge branch 'main' into 00217-tck-implement-json-rpc-methods-for-acc…
rwalworth Aug 6, 2024
75158ff
Update test/crypto-service/test_accountUpdateTransaction.js
rwalworth Aug 7, 2024
dbbe673
refactor: fix formatting
rwalworth Aug 7, 2024
1b9cf76
fix: floating point numbers need to be parsed as ints. Fix return sta…
rwalworth Aug 8, 2024
94917b2
fix: uncomment tests
rwalworth Aug 8, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion consensusInfoClient.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { AccountBalanceQuery, AccountId, AccountInfoQuery, Client, TokenCreateTr
class ConsensusInfoClient {
constructor() {
if (process.env.NODE_IP && process.env.NODE_ACCOUNT_ID && process.env.MIRROR_NETWORK) {
const node = { [process.env.NODE_IP]: new AccountId(parseInt(process.env.NODE_ACCOUNT_ID)) };
const node = { [process.env.NODE_IP]: AccountId.fromString(process.env.NODE_ACCOUNT_ID) };
this.sdkClient = Client.forNetwork(node);
} else {
this.sdkClient = Client.forTestnet();
Expand Down
97 changes: 47 additions & 50 deletions test-specifications/crypto-service/accountUpdateTransaction.md

Large diffs are not rendered by default.

124 changes: 0 additions & 124 deletions test/crypto-service/test_UpdateAccountKey.js

This file was deleted.

121 changes: 0 additions & 121 deletions test/crypto-service/test_UpdateAccountMemo.js

This file was deleted.

Loading