As per title. Add CTOR ordering to test block construction. Use
NULLDUMMY instead of CLEANSTACK for checking consensus vs non-consensus
script cache behavior.
Details
Details
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- unit-test
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 3969 Build 6010: Bitcoin ABC Buildbot (legacy) Build 6009: arc lint + arc unit
Event Timeline
| src/test/test_bitcoin.cpp | ||
|---|---|---|
| 170 ↗ | (On Diff #5897) | Is this comment still relevant? Either I'm missing something or it's already done above. |
Comment Actions
This is a better approach toward fixing that test.
| src/test/test_bitcoin.cpp | ||
|---|---|---|
| 163 ↗ | (On Diff #5898) | I don't think this really warrant to modify the test framework. |
| src/test/txvalidationcache_tests.cpp | ||
| 202 ↗ | (On Diff #5898) | It's the last item. You are pushing onto a stack so the item you push first end up being the last item on the stack. |
| src/test/txvalidationcache_tests.cpp | ||
|---|---|---|
| 198 ↗ | (On Diff #5904) | Either comment is out of date, or this appears to be missing OP_1 |
| src/test/txvalidationcache_tests.cpp | ||
|---|---|---|
| 198 ↗ | (On Diff #5904) | Oh I see, I think this comment needs to be moved above ^^ |
| src/script/interpreter.cpp | ||
|---|---|---|
| 1084 | Revert | |
| 1088 | revert | |
| src/test/txvalidationcache_tests.cpp | ||
| 185 | There is no point in bringing the key store in there. You are building the script to begin with so anything you'll pull of the store you already have. | |
| 188 | I'm not sure why you want to do P2SH here instead of bare multisig. If there is a reason, it should be mentioned. if there is no reason, then don't do it. | |
| 200 | This would benefit being scoped to destruct the temporaries. | |
| 202 | This is not spending, this is mining. | |
| 243 | You can sign the exact same ways as before. You just need to push the non nulldummy element. | |
| src/test/txvalidationcache_tests.cpp | ||
|---|---|---|
| 243 | +1. doing this a different way is confusing | |
| src/test/txvalidationcache_tests.cpp | ||
|---|---|---|
| 243 | I did that | |