Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

fix balance transfer issue - release/2.1.x - #9813

Merged
brianjohnson5972 merged 1 commit into
release/2.1.xfrom
huangminghuang/fix-nodeos-run-test
Dec 18, 2020
Merged

brianjohnson5972 merged 1 commit into
release/2.1.xfrom
huangminghuang/fix-nodeos-run-test

Conversation

@huangminghuang

Copy link
Copy Markdown
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.

2 participants