tari-project
/
tari
/
16884686988
/
1
62%
development: 62%
development: 62%
|
Ran
|
Files
817
|
Run time
39s
|
Badge
Embed ▾
README BADGES
|
coverage: 54.251% (+0.01%) from 54.238%
push
github
web-flowfix: ci integration tests workflow (#7389) Description --- After merging HTTP wallet connecting wallet to base node has changed but it was not reflected in the [integration tests](https://github.com/tari-project/tari/pull/7215/files#diff-5ebed9b65a60af6d0d090d184ab0e425eL200). This PR fixes it since there is no longer gRPC method `set_base_node` but instead wallets needs to receive base_node HTTP address. This also changes integration tests workflow to exit early if first test fails and exit with non-zero exit code. How Has This Been Tested? --- Run integration tests locally and check CI if it's still showing green check mark for failed tests. Breaking Changes --- - [x] None - [ ] Requires data directory on base node to be deleted - [ ] Requires hard fork - [ ] Other - Please specify <!-- Does this include a breaking change? If so, include this line as a footer --> <!-- BREAKING CHANGE: Description what the user should do, e.g. delete a database, resync the chain --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Tests now expose HTTP ports for base node and wallet processes; base node startup disables the splash screen by default. * **Bug Fixes** * Fixed a minor logging syntax issue in wallet integration tests. * **Chores** * Upgraded test framework dependency and reformatted dependency lists. * CI/integration workflows: improved logging, explicit error handling, safer shell options, extended binary test timeout, and expanded workflow triggers. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
75708 of 139552 relevant lines covered (54.25%)
195209.52 hits per line
