tari-project / tari / 16884686988

62%

Build Type

push

github

Committed by web-flow

Commit Message

fix: 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 -->

Run Details

8 of 10 new or added lines in 5 files covered. (80.0%)

7 existing lines in 2 files now uncovered.

75708 of 139552 relevant lines covered (54.25%)

195209.52 hits per line