Skip to content

test(DevLogs): Bump test coverage for DevLogs component#254

Merged
its-me-abhishek merged 1 commit intoCCExtractor:mainfrom
ShivaGupta-14:feat/192-devlogs-test-coverage
Dec 8, 2025
Merged

test(DevLogs): Bump test coverage for DevLogs component#254
its-me-abhishek merged 1 commit intoCCExtractor:mainfrom
ShivaGupta-14:feat/192-devlogs-test-coverage

Conversation

@ShivaGupta-14
Copy link
Copy Markdown
Contributor

@ShivaGupta-14 ShivaGupta-14 commented Dec 7, 2025

DevLogs Test Coverage Improvements

Description

Bumped test coverage for the DevLogs component by adding comprehensive unit tests.

Previously, the component lacked proper test coverage for its core functionality, making future changes riskier to validate.

Tests added:

  • Loading, empty, and error states
  • Log filtering by level (INFO/WARN/ERROR)
  • Copy single log and copy-all functionality

Fixes: #192

Checklist

  • Ran npx prettier --write . (for formatting)
  • Ran gofmt -w . (for Go backend)
  • Ran npm test (for JS/TS testing)
  • Added unit tests, if applicable
  • Verified all tests pass
  • Updated documentation, if needed

Additional Notes

Self-Review

I've reviewed my changes and confirm:

  • All tests pass locally
  • Mocks are correctly set up for UI components and fetch
  • Tests cover the main component behaviors (loading, filtering, copying)
  • No duplicate or redundant tests
  • Code follows existing test patterns in the project

Screenshots

  • Before:

    Screenshot 2025-12-08 at 1 52 13 AM
  • After:

    Screenshot 2025-12-08 at 1 51 28 AM

- Cover loading, empty, and error states
- Test log filtering by level (INFO/WARN/ERROR)
- Test copy single log and copy all functionality

Fixes: CCExtractor#192
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 7, 2025

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

@its-me-abhishek its-me-abhishek merged commit 2b2709c into CCExtractor:main Dec 8, 2025
5 checks passed
@ShivaGupta-14 ShivaGupta-14 deleted the feat/192-devlogs-test-coverage branch December 10, 2025 11:47
ShivaGupta-14 added a commit to ShivaGupta-14/ccsync that referenced this pull request Dec 25, 2025
)

- Cover loading, empty, and error states
- Test log filtering by level (INFO/WARN/ERROR)
- Test copy single log and copy all functionality

Fixes: CCExtractor#192
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.

Bump test coverage for Devlogs

2 participants