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

tari-project / tari / 16731863361
62%

Build:
DEFAULT BRANCH: development
Ran 04 Aug 2025 08:27PM UTC
Jobs 1
Files 818
Run time 189min
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

04 Aug 2025 07:04PM UTC coverage: 54.203% (+0.02%) from 54.182%
16731863361

push

github

web-flow
fix: memo naming from payment_id (#7385)

Description
---
- Fixed memo naming where it still referenced payment_id.
- Added a unit test to verify that a wallet payment_id address can
provide its payment_id.
- Added debug logs to gRPC method `get_payment_id_address`.

Motivation and Context
---
These changes were a result of investigating issue #7363.

How Has This Been Tested?
---
System-level testing

What process can a PR reviewer use to test or verify this change?
---
Code review.

<!-- 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 chain -->


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Bug Fixes**
* Improved debug logging for payment ID and address handling in the
console wallet, aiding in troubleshooting.

* **Tests**
* Added new tests to verify correct retrieval and handling of memo
fields (payment ID) in addresses.
* Enhanced test assertions for memo field extraction and address
validation.

* **Style**
* Updated parameter names and documentation to consistently use "memo"
instead of "payment ID" for improved clarity.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

78 of 85 new or added lines in 3 files covered. (91.76%)

5 existing lines in 4 files now uncovered.

75560 of 139401 relevant lines covered (54.2%)

195388.17 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
7
87.4
0.37% base_layer/core/src/transactions/transaction_components/memo_field.rs

Uncovered Existing Lines

Lines Coverage ∆ File
1
93.35
-0.18% base_layer/core/src/proof_of_work/monero_rx/merkle_tree.rs
1
50.68
-0.04% base_layer/wallet/src/transaction_service/storage/sqlite_db.rs
1
75.98
-0.22% comms/core/src/connection_manager/peer_connection.rs
2
55.65
-0.11% comms/core/src/connectivity/manager.rs
Jobs
ID Job ID Ran Files Coverage
1 16731863361.1 04 Aug 2025 08:27PM UTC 818
54.2
GitHub Action Run
Source Files on build 16731863361
  • Tree
  • List 818
  • Changed 8
  • Source Changed 3
  • Coverage Changed 7
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #16731863361
  • d219520b on github
  • Prev Build on development (#16653736382)
  • Next Build on development (#16746310130)
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