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

TestHarness Enhanced Documentation #285

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

ericpassmore
Copy link
Contributor

  • added description to TestHarness python classes
  • corrected error codes specified in docs
  • additional overview to PerformanceHarness README
  • oneline description of python files in TestHarness
  • example call and output from trx_generator

```

## Output
Why is this 0us
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is 0 because this is logging _total_us which is never updated. Just initialized to 0. Seems you found a bug. I don't guess anyone ever noticed.

Copy link
Contributor Author

@ericpassmore ericpassmore Jun 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

forgot I had that question, converted PR to draft. I'll continue to add documentation updates, and better document the why 0us.

This example is from Jungle Spring Beta2 test net. I believe an intermediate node is filtering the submitted transactions to the P2P endpoint, and they are not making onto the blockchain. Need some additional tests to verify.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Time is zero, because _total_us is never updated.

@ericpassmore ericpassmore marked this pull request as draft June 13, 2024 19:08
## Command
Sleep is needed otherwise subjective transaction may reject transactions as too soon.
```
CHAIN_ID=$(cleos --url $HTTP_URL get info | grep chain_id | cut -d: -f2 | sed 's/[ ",]//g')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not suggesting that you need to change this, but if you're open to using jq you can do this shorter via cleos get info | jq -r .chain_id

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

Successfully merging this pull request may close these issues.

3 participants