• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

tari-project / tari / 16621916274
62%

Build:
DEFAULT BRANCH: development
Ran 30 Jul 2025 01:35PM UTC
Jobs 1
Files 818
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

30 Jul 2025 12:00PM UTC coverage: 54.226% (+0.003%) from 54.223%
16621916274

push

github

web-flow
feat: enhance wallet connectivity monitoring (#7372)

Description
---
Closes #7360 

Enhance wallet connectivity check to proactively check it during wallet
scanning.

Previously grpc call `check_connectivity` falsely returned `Offline`
status while syncing. This was caused because it relied on `is_online`
method in `https.rs` which check if last http call was done in less than
a minute ago but initially no http calls were made and returned
incorrect value.
Now this grpc method relies on pro-actively updated value checked every
5 minutes which verifies if tip of the scanned block has changed since
the previous check.

How Has This Been Tested?
---

Replaced wallet in TU and check `get_state` and `check_connectivity`
methods calls results while scanning wallet. In both cases I received
`Online` status.

```
❯ grpcurl -plaintext -import-path tari/applications/minotari_app_grpc/proto -proto wallet.proto -d '{}' 127.0.0.1:42027 tari.rpc.Wallet/CheckConnectivity
{
  "status": "Online"
}
```

```
14:07:05 INFO  Wallet status: GetStateResponse { scanned_height: 54930, balance: Some(GetBalanceResponse { available_balance: 3107794619, pending_incoming_balance: 190000, pending_outgoing_balance: 2111889, timelocked_balance: 0 }), network: Some(NetworkStatusResponse { status: Online, avg_latency_ms: 18, num_node_connections: 11 }) }
14:07:05 INFO  Initial wallet scanning: 90.81590476977763% (54930/60485)
```

<!-- Checklist -->
<!-- 1. Is the title of your PR in the form that would make nice release
notes? The title, excluding the conventional commit
tag, will be included exactly as is in the CHANGELOG, so please think
about it carefully. -->


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 ch... (continued)

0 of 3 new or added lines in 2 files covered. (0.0%)

9 existing lines in 3 files now uncovered.

75493 of 139218 relevant lines covered (54.23%)

195785.14 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
0.0
0.0% clients/rust/base_node_wallet_client/src/client/http.rs
2
0.0
0.0% base_layer/wallet/src/output_manager_service/handle.rs

Uncovered Existing Lines

Lines Coverage ∆ File
1
75.98
-0.22% comms/core/src/connection_manager/peer_connection.rs
3
60.03
-0.4% comms/dht/src/connectivity/mod.rs
5
55.11
-0.43% comms/core/src/connectivity/manager.rs
Jobs
ID Job ID Ran Files Coverage
1 16621916274.1 30 Jul 2025 01:35PM UTC 818
54.23
GitHub Action Run
Source Files on build 16621916274
  • Tree
  • List 818
  • Changed 11
  • Source Changed 4
  • Coverage Changed 9
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #16621916274
  • bb0b2fa9 on github
  • Prev Build on development (#16604848589)
  • Next Build on development (#16644501184)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc