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

fix(cli): check if node is ACTIVE after it is started #616

Merged
merged 9 commits into from
Dec 11, 2023

Conversation

leninmehedy
Copy link
Member

Description

This pull request changes the following:

  • fix(cli): check if node is ACTIVE after it is started
  • add E2E tests to ensure query and create account transaction can be invoked

Related Issues

Copy link

github-actions bot commented Dec 7, 2023

Unit Test Results

  26 files  ±0    26 suites  ±0   54s ⏱️ +2s
109 tests ±0  105 ✔️ ±0  4 💤 ±0  0 ±0 
110 runs  ±0  106 ✔️ ±0  4 💤 ±0  0 ±0 

Results for commit 6923034. ± Comparison against base commit af42af8.

♻️ This comment has been updated with latest results.

Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
…uery calls

Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
@leninmehedy leninmehedy force-pushed the 515-node-start-e2e-test branch from 1f134d6 to 4e21148 Compare December 8, 2023 00:38
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>

# Conflicts:
#	fullstack-network-manager/.npmrc
#	fullstack-network-manager/src/commands/node.mjs
#	fullstack-network-manager/src/core/constants.mjs
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jeromy-cannon jeromy-cannon merged commit 2dc488f into main Dec 11, 2023
15 checks passed
@jeromy-cannon jeromy-cannon deleted the 515-node-start-e2e-test branch December 11, 2023 18:46
swirlds-automation added a commit that referenced this pull request Dec 15, 2023
## [0.15.0](v0.14.0...v0.15.0) (2023-12-15)

### Features

* **cli:** show elapsed time for each step ([#620](#620)) ([af42af8](af42af8))
* enable self-signed TLS support for Hedera Mirror Node Explorer in the Helm charts ([#630](#630)) ([7cb24d8](7cb24d8))

### Bug Fixes

* **cli:** check if node is ACTIVE after it is started ([#616](#616)) ([2dc488f](2dc488f))
* resolve issues with package.json not being updated during releases ([cc4f1d6](cc4f1d6))
@swirlds-automation
Copy link
Contributor

🎉 This PR is included in version 0.15.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLI[Chart]: Invoke a transaction using SDK after deployment
3 participants