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

Using an account other than 0.0.2 to run the local node causes the deterministic deployment transaction test to fail. #2700

Open
ebadiere opened this issue Jul 12, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@ebadiere
Copy link
Collaborator

Description

Should execute eth_sendRawTransaction on Standard Web Socket for the deterministic deployment transaction and Should execute eth_sendRawTransaction on Ethers Web Socket Provider for the deterministic deployment transaction fail when using an account other than 0.0.2 when running with local node. The tests run a crypto transfer to the deterministic deployment signer to activate the account and allow it to run the deterministic deployer transaction. When an account taken from the local node start up is used, the crypt transfer fails and the deterministic deployment signer's address has a balance of zero.

Steps to reproduce

  1. In the .env file set the OPERATOR_ID_MAIN and the OPERATOR_KEY_MAIN to values from an account provided by the local node at start up.
  2. Run either test mentioned above.

Additional context

No response

Hedera network

other

Version

v0.52.0

Operating system

macOS

@ebadiere ebadiere added the bug Something isn't working label Jul 12, 2024
ebadiere added a commit that referenced this issue Jul 12, 2024
Opened issue: #2700

Signed-off-by: ebadiere <ebadiere@gmail.com>
ebadiere added a commit that referenced this issue Jul 23, 2024
#2690)

* fix: Updated the account from the 0.0.2 account to an account from the local node to track
transaction costs.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Turning off test to potentially get through CI.
Opened issue: #2700

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: WIP: Pushing up to collaborate

Signed-off-by: ebadiere <ebadiere@gmail.com>

fix: Updated the private key to the hex ECDSA value.

Signed-off-by: ebadiere <ebadiere@gmail.com>

fix: Bumped up the gasLimit to the required amount to activate a hollow account.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix:  Added key format support to the serviceClient.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Extracted the createPrivateKeyBasedOnFormat(key) to the Utils class.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Updated import of createPrivateKeyBasedOnFormat from Utils.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Removed memory leak check.  PR 26954 addresses memory leaks.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix:  Removed null test because an env var which is a string cannot be
set to null

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Removed the comment in the GH workflow

Signed-off-by: ebadiere <ebadiere@gmail.com>

---------

Signed-off-by: ebadiere <ebadiere@gmail.com>
Signed-off-by: Eric Badiere <ebadiere@gmail.com>
ebadiere added a commit that referenced this issue Aug 1, 2024
#2690)

* fix: Updated the account from the 0.0.2 account to an account from the local node to track
transaction costs.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Turning off test to potentially get through CI.
Opened issue: #2700

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: WIP: Pushing up to collaborate

Signed-off-by: ebadiere <ebadiere@gmail.com>

fix: Updated the private key to the hex ECDSA value.

Signed-off-by: ebadiere <ebadiere@gmail.com>

fix: Bumped up the gasLimit to the required amount to activate a hollow account.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix:  Added key format support to the serviceClient.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Extracted the createPrivateKeyBasedOnFormat(key) to the Utils class.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Updated import of createPrivateKeyBasedOnFormat from Utils.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Removed memory leak check.  PR 26954 addresses memory leaks.

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix:  Removed null test because an env var which is a string cannot be
set to null

Signed-off-by: ebadiere <ebadiere@gmail.com>

* fix: Removed the comment in the GH workflow

Signed-off-by: ebadiere <ebadiere@gmail.com>

---------

Signed-off-by: ebadiere <ebadiere@gmail.com>
Signed-off-by: Eric Badiere <ebadiere@gmail.com>
Signed-off-by: ebadiere <ebadiere@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

1 participant