Page MenuHomePhabricator

Ensure teamcity runs more of the unit tests.
ClosedPublic

Authored by schancel on Apr 12 2018, 04:59.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
moar-testing
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2309
Build 2756: Bitcoin ABC Buildbot (legacy)
Build 2755: arc lint + arc unit

Event Timeline

Fix build of secp256k1 and univalue

1/ What about leveldb ?
2/ This doesn't seems like it'll fail if tests or even the build fails

1/ What about leveldb ?
2/ This doesn't seems like it'll fail if tests or even the build fails

It is "set -e" at the top, I believe it'll fail?

@schancel The code LGTM, don't have context about what it is supposed to cover.

Leveldb seems to be built in a strange way. No makefiles are being generated by ../configure

This approach is wrong, the leveldb thing was kind of a tell. This will lead to have to maintain a list of tests that will inevitably get out of sync and have pieces missing. Why isn't it possible to use make check ? If there is something missing from there, then it's probably a better idea to modify it to do what it's not doing properly now rather than create a list of test to maintain.

This approach is wrong, the leveldb thing was kind of a tell. This will lead to have to maintain a list of tests that will inevitably get out of sync and have pieces missing. Why isn't it possible to use make check ? If there is something missing from there, then it's probably a better idea to modify it to do what it's not doing properly now rather than create a list of test to maintain.

Yes, I need it to do ./src/test/test_bitcoin --log_format=JUNIT > test_bitcoin.xml

When you have some time maybe you can help me figure out how to do this.

Ok, please make a task about this. In the meantime, we can do as this.

This revision is now accepted and ready to land.Apr 27 2018, 23:40
This revision was automatically updated to reflect the committed changes.