Skip to content

test(api)-Unit Test cases For settingsservice[#309]#333

Merged
leeeunkoo merged 2 commits intoeclipse-pullpiri:mainfrom
akshaylg0314:main
Oct 10, 2025
Merged

test(api)-Unit Test cases For settingsservice[#309]#333
leeeunkoo merged 2 commits intoeclipse-pullpiri:mainfrom
akshaylg0314:main

Conversation

@akshaylg0314
Copy link
Contributor

@akshaylg0314 akshaylg0314 commented Oct 3, 2025

📝 PR Description
test(api)-Unit Test cases For settingsservice[#309]

🔗 Related Issue
Closes #309

🧪 Test Method
cargo deny check

cargo test
cargo tarpaulin
📸 Screenshots
image

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

@akshaylg0314
Copy link
Contributor Author

Hey @leeeunkoo @daeyoung-jeong-lge @youngtaekiim
Please review and merge this test code
thanks!!

Copy link
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 good.

@daeyoung-jeong-lge
Copy link
Contributor

@akshaylg0314 Could you attach the screenshot as liike you did at #312. Thanks.

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

@leeeunkoo leeeunkoo merged commit 37076b4 into eclipse-pullpiri:main Oct 10, 2025
6 checks passed
@leeeunkoo leeeunkoo added this to the Setting Dashboard milestone Oct 10, 2025
@leeeunkoo leeeunkoo added the enhancement New feature or request label Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[TASK] Add test codes for SetttingsService

3 participants