Skip to content

test: improve lpad sqllogictest coverage - #23969

Open
2010YOUY01 wants to merge 1 commit into
apache:mainfrom
2010YOUY01:codex/coverage-lpad
Open

test: improve lpad sqllogictest coverage#23969
2010YOUY01 wants to merge 1 commit into
apache:mainfrom
2010YOUY01:codex/coverage-lpad

Conversation

@2010YOUY01

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

  • Closes #.

Rationale for this change

Codecov reports low test coverage for this function, so this PR adds more SLT tests for it.

The tests were AI-generated, and I manually verified that the results are correct.

See Codecov bot comment -> ☔ View full report in Codecov by Harness -> Indirect Changes for the updated test coverage.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions Bot added the sqllogictest SQL Logic Tests (.slt) label Jul 29, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.76%. Comparing base (02a902c) to head (237ee49).
⚠️ Report is 113 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #23969      +/-   ##
==========================================
+ Coverage   80.71%   80.76%   +0.05%     
==========================================
  Files        1089     1096       +7     
  Lines      368293   373282    +4989     
  Branches   368293   373282    +4989     
==========================================
+ Hits       297250   301468    +4218     
- Misses      53332    53831     +499     
- Partials    17711    17983     +272     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants