Skip to content

test: More tests for Issue - #7627

Open
kuznetsss wants to merge 3 commits into
XRPLF:developfrom
kuznetsss:Add_tests_for_Issue
Open

test: More tests for Issue#7627
kuznetsss wants to merge 3 commits into
XRPLF:developfrom
kuznetsss:Add_tests_for_Issue

Conversation

@kuznetsss

Copy link
Copy Markdown
Contributor

High Level Overview of Change

Add a few more tests for Issue.

Context of Change

API Impact

  • Public API: New feature (new methods and/or new fields)
  • Public API: Breaking change (in general, breaking changes should only impact the next api_version)
  • libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
  • Peer protocol change (must be backward compatible or bump the peer protocol version)

@kuznetsss
kuznetsss requested review from a1q123456 and godexsoft June 25, 2026 14:24
godexsoft
godexsoft previously approved these changes Jun 25, 2026

@godexsoft godexsoft left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.3%. Comparing base (07c64f0) to head (dce21ce).
⚠️ Report is 65 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           develop   #7627     +/-   ##
=========================================
+ Coverage     82.0%   82.3%   +0.3%     
=========================================
  Files         1007    1024     +17     
  Lines        76890   78459   +1569     
  Branches      8971    8975      +4     
=========================================
+ Hits         63050   64568   +1518     
- Misses       13831   13882     +51     
  Partials         9       9             

see 93 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

a1q123456
a1q123456 previously approved these changes Jul 1, 2026
@kuznetsss
kuznetsss dismissed stale reviews from a1q123456 and godexsoft via ecbf8a9 July 1, 2026 14:36
@kuznetsss
kuznetsss force-pushed the Add_tests_for_Issue branch from ecbf8a9 to dce21ce Compare July 1, 2026 14:40
@github-actions

Copy link
Copy Markdown

This PR has conflicts, please resolve them in order for the PR to be reviewed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants