This checks whether network upgrade code named 'great wall' has activated.
Details
Details
- Reviewers
deadalnix jasonbcox - Group Reviewers
Restricted Project - Commits
- rSTAGING1fcc2a0a4cb0: Add function 'IsGreatWallEnabled'
rABC1fcc2a0a4cb0: Add function 'IsGreatWallEnabled'
Added test
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- gw-activation
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 4623 Build 7309: Bitcoin ABC Buildbot (legacy) Build 7308: arc lint + arc unit
Event Timeline
| src/consensus/activation.cpp | ||
|---|---|---|
| 62 | Please check if this needs to be broken up into separate functions similar to magnetic anomaly. Iirc the inner function is called directly from another context. | |
| src/test/activation_tests.cpp | ||
| 13 | These tests probably don't add much value on their own. Activation of the upgrade features tends to be covered in their own tests, which would overlap with this test entirely. | |
| src/consensus/activation.cpp | ||
|---|---|---|
| 62 | If it is needed, it can be done. | |
| src/test/activation_tests.cpp | ||
| 13 | It adds value because the tests you are mentioning do not exist. This can be removed if/when obsolete and if you feel strongly about it, you'll even be able to do it. There is no point in delaying @dagurval 's work for this. | |
| src/test/activation_tests.cpp | ||
|---|---|---|
| 25 | Hmm does this work? nTime single block timestamp but the activation is based on median-time-past. | |
| src/test/activation_tests.cpp | ||
|---|---|---|
| 25 ↗ | (On Diff #6812) | Can you make sure mtp is tested, by producing 11 headers instead ? It's fairly easy to declare an array of 11 CBlockIndex |
| src/test/activation_tests.cpp | ||
|---|---|---|
| 13 | Fair. | |