Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

Conversation

@huangminghuang
Copy link
Contributor

The test can be occasionally fail because the balance of defproducera was determined by the number of successful balance transfers in 398-426. The balance of defproducera could be accumulated up to 0.0106 CUR before considered error in line 429. Therefore, the way to make sure the transfer in line 513 to fail is to make the balance transfer amount in line 513 greater than 0.0106 CUR.

Change Description

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

The test can be occasionally fail because the balance of defproducera was determined by the number of  successful balance transfers in 398-426. The balance of defproducera could be accumulated up to 0.0106 CUR before considered error in line 429. Therefore, the way to make sure the transfer in line 513 to fail is to make the balance transfer amount in line 513 greater than 0.0106 CUR.
@brianjohnson5972 brianjohnson5972 changed the title fix balance transfer issue fix balance transfer issue - release/2.1.x Dec 18, 2020
@brianjohnson5972 brianjohnson5972 merged commit bbb4e85 into release/2.1.x Dec 18, 2020
@huangminghuang huangminghuang deleted the huangminghuang/fix-nodeos-run-test branch December 28, 2020 15:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.