Skip to content

test: Implement tests for Navigation history dialog#4811

Merged
kelson42 merged 4 commits intokiwix:mainfrom
harshsomankar123-tech:dialog
Apr 18, 2026
Merged

test: Implement tests for Navigation history dialog#4811
kelson42 merged 4 commits intokiwix:mainfrom
harshsomankar123-tech:dialog

Conversation

@harshsomankar123-tech
Copy link
Copy Markdown
Contributor

@harshsomankar123-tech harshsomankar123-tech commented Apr 9, 2026

FIX #4809

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.51%. Comparing base (fc45ebb) to head (ee83510).
⚠️ Report is 28 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4811      +/-   ##
============================================
+ Coverage     60.49%   60.51%   +0.01%     
+ Complexity     1675     1671       -4     
============================================
  Files           333      333              
  Lines         16020    16020              
  Branches       2225     2225              
============================================
+ Hits           9692     9694       +2     
- Misses         4855     4863       +8     
+ Partials       1473     1463      -10     

☔ View full report in Codecov by Sentry.
📢 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.

@harshsomankar123-tech
Copy link
Copy Markdown
Contributor Author

@MohitMaliFtechiz Please Take A look

Copy link
Copy Markdown
Collaborator

@MohitMaliFtechiz MohitMaliFtechiz left a comment

Choose a reason for hiding this comment

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

@harshsomankar123-tech There is a small change needed. After that, we are good to merge.

Comment thread buildSrc/src/main/kotlin/plugin/AllProjectConfigurer.kt Outdated
Copy link
Copy Markdown
Collaborator

@MohitMaliFtechiz MohitMaliFtechiz left a comment

Choose a reason for hiding this comment

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

@harshsomankar123-tech Thank you 🎉. Once CI is green, we will merge this one.

@kelson42 kelson42 merged commit 71066c0 into kiwix:main Apr 18, 2026
28 of 29 checks passed
@harshsomankar123-tech harshsomankar123-tech deleted the dialog branch April 18, 2026 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add UI tests for NavigationHistoryDialogScreen Compose components

3 participants