Skip to content

Conversation

@SmartArray
Copy link

@SmartArray SmartArray commented Apr 7, 2022

Fix: Functional Tests

160 commits later, this PR finally fixes all 158 failing functional test by adapting them to the DigiByte network and chainparams.

Before this PR can be tested and merged, #89 needs to be accepted and merged because the functional tests will only run through if the core-related issues are fixed.

In order to fix the tests, it was required to

  • generate new test vectors
  • respect lower COIN_MATURITY parameter
  • add scrypt as a python library that will be called during the tests
  • add -easypow that turns off MultiAlgo during tests that are executed in regtest environment
  • add a testnet checkpoint
  • adapt higher absolute fees
  • fix several mempool bugs
  • and more

Yoshi Jäger and others added 30 commits January 17, 2022 08:03
This will be needed to postpone difficulty adjustment modifications to a block in the future throughout the functional tests
Yoshi Jäger added 20 commits April 6, 2022 14:00
…hain tx required due to small maturity of DigiByte coins
…-date with test_framework.

2/3 subtests are failing. Core code needs to be adjusted.
Copy link

@JaredTate JaredTate left a comment

Choose a reason for hiding this comment

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

ACK. Once again amazing work @SmartArray. Having these tests working is a fundamental step towards a very stable 8.22. The client compiles, runs, and works great.

Copy link
Member

@gto90 gto90 left a comment

Choose a reason for hiding this comment

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

ACK

@ghost
Copy link

ghost commented Apr 20, 2022

❤️

@ycagel ycagel self-requested a review May 5, 2022 01:51
Copy link
Member

@ycagel ycagel left a comment

Choose a reason for hiding this comment

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

cACK

@ycagel ycagel merged commit 5439490 into DigiByte-Core:feature/8.22.0 May 5, 2022
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.

4 participants