perf: enhance account key update logic to batch transactions instead of loading all with sleep

[jeromy-cannon]

Description

This pull request changes the following:

• enhance account key update logic to batch transactions instead of loading all with sleep
• removed unused reference to @jest/test-sequencer
• added logic to properly close WebSocketServer connections
• removed multiple sleep commands
• updated for check for Terminating Netty string
• removed extra creation of NodeClient to simplify and increase performance
• restructured mirror node call to better leverage listr2 subtasks for user experience
• restructured update of account keys to better leverage listr2 subtasks
• fixed bug was not reporting skips and rejections correctly in the account key updates loop

Related Issues

• Closes Enhance account key update logic to batch transactions instead of loading all with sleep #104
• Closes Remove jest/test-sequencer from package.json as it is no longer used #76
• Closes Refactor update special account keys using Listr #71
• Closes Incorrect status reported by account manager #59

[github-actions]

Unit Test Results

 1 files  ±0  13 suites  ±0   20s ⏱️ ±0s
83 tests ±0  83 ✅ +2  0 💤 ±0  0 ❌  - 2 
90 runs  +1  90 ✅ +3  0 💤 ±0  0 ❌  - 2 

Results for commit 50965e0. ± Comparison against base commit 3d91617.

♻️ This comment has been updated with latest results.

[github-actions]

E2E Test Results

  1 files  ±  0    6 suites  ±0   13m 18s ⏱️ + 11m 49s
736 tests +  1  736 ✅ +  6  0 💤 ±0  0 ❌  - 5 
736 runs  +714  736 ✅ +719  0 💤 ±0  0 ❌  - 5 

Results for commit 50965e0. ± Comparison against base commit 3d91617.

This pull request removes 702 and adds 703 tests. Note that renamed tests count towards both.

NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 10 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 10 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 100 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 100 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 1000 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 1000 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 11 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 11 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 12 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 12 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 13 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 13 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 14 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 14 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 15 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 15 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 16 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 16 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 17 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] [beforeAll - ERROR]: only genesis account should have genesis key for all special accounts special account 17 should not have genesis key
…

NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] checkNetworkNodeProxyUp should succeed ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] checkNetworkNodeProxyUp should succeed
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 10 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 10 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 100 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 100 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 1000 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 1000 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 11 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 11 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 12 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 12 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 13 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 13 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 14 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 14 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 15 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 15 should not have genesis key
NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 16 should not have genesis key ‑ NodeCommand node start should succeed [release v0.42.5, keyFormat: pfx] only genesis account should have genesis key for all special accounts special account 16 should not have genesis key
…

♻️ This comment has been updated with latest results.

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
Report missing for 3d916171	✅ 87.82%

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (3d91617)	Report Missing	Report Missing	Report Missing
Head commit (50965e0)	2220	1444	65.05%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#105)	156	137	87.82%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

_{You may notice some variations in coverage metrics with the latest Coverage engine update. For more details, visit the documentation}

Footnotes

1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct. ↩

[codecov]

Codecov Report

Attention: Patch coverage is 88.70968% with 21 lines in your changes are missing coverage. Please review.

Project coverage is 64.65%. Comparing base (8fd976b) to head (50965e0).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   63.73%   64.65%   +0.91%     
==========================================
  Files          27       27              
  Lines        2570     2634      +64     
  Branches      422      432      +10     
==========================================
+ Hits         1638     1703      +65     
+ Misses        932      931       -1     

Files	Coverage Δ
src/commands/account.mjs	67.83% <100.00%> (-2.50%)	⬇️
src/core/constants.mjs	92.06% <100.00%> (ø)
src/core/k8.mjs	84.32% <100.00%> (+5.32%)	⬆️
src/core/account_manager.mjs	85.65% <89.77%> (+0.23%)	⬆️
src/commands/node.mjs	55.50% <81.81%> (+1.76%)	⬆️

... and 1 file with indirect coverage changes

[leninmehedy]

Please address my comments and merge when you are ready.

[swirlds-automation]

🎉 This PR is included in version 0.22.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀