Skip to content

test(statemanager)-Unit test cases for statemanager with code coverage[#321]#396

Merged
youngtaekiim merged 1 commit into
eclipse-pullpiri:mainfrom
akshaylg0314:main
Dec 12, 2025
Merged

test(statemanager)-Unit test cases for statemanager with code coverage[#321]#396
youngtaekiim merged 1 commit into
eclipse-pullpiri:mainfrom
akshaylg0314:main

Conversation

@akshaylg0314
Copy link
Copy Markdown
Contributor

📝 PR Description
test(statemanager)-Unit test cases for statemanager with code coverage[#321]

🔗 Related Issue
Closes #321

🧪 Test Method
sudo cargo tarpaulin --skip-clean --exclude-files "/filtergateway/"
cargo test

Screenshot:

image

✅ Checklist
[✅] Code conventions are followed
[✅] Tests are added/modified
[✅] Documentation is updated (if necessary)

@akshaylg0314
Copy link
Copy Markdown
Contributor Author

Hey @daeyoung-jeong-lge @youngtaekiim @leeeunkoo
Please review and merge these changes
thanks!!

@daeyoung-jeong-lge
Copy link
Copy Markdown
Contributor

@akshayg0314 Is there any possibility to make the coverage ratio to be higher than 80%?

@akshaylg0314
Copy link
Copy Markdown
Contributor Author

akshaylg0314 commented Dec 11, 2025

@akshayg0314 Is there any possibility to make the coverage ratio to be higher than 80%?

Hey @daeyoung-jeong-lge we will try to make it higher than 80% in next version with integration test cases, but with only unit test cases only 77% code coverage we are able to do right now.
thanks!!

Copy link
Copy Markdown
Contributor

@daeyoung-jeong-lge daeyoung-jeong-lge left a comment

Choose a reason for hiding this comment

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

Looks OK

@daeyoung-jeong-lge
Copy link
Copy Markdown
Contributor

@youngtaekiim I finished my review. Please merge this. Thanks.

@youngtaekiim youngtaekiim merged commit 23d279f into eclipse-pullpiri:main Dec 12, 2025
6 checks passed
eunhyeYu pushed a commit to eunhyeYu/pullpiri that referenced this pull request Mar 11, 2026
test(statemanager)-Unit test cases for statemanager with code coverage[eclipse-pullpiri#321]
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.

[TASK] Add test codes for StateManager

4 participants