Skip to content

Tests for WatchedSet#22

Merged
vittorius merged 2 commits intomainfrom
add-tests-for-watched-set
Jan 17, 2026
Merged

Tests for WatchedSet#22
vittorius merged 2 commits intomainfrom
add-tests-for-watched-set

Conversation

@vittorius
Copy link
Owner

@vittorius vittorius commented Jan 16, 2026

  • Added tests for WatchedSet using mocks
  • Moved test list with FS events and notifications to PathWatcher
    • because of mocks used in WatchedSet tests, PathWatcher is the type that will be truly tested for FS events and notifications using a temporary FS
  • Minor improvements

PR train:

@vittorius vittorius changed the title Add tests for watched set Tests for WatchedSet Jan 16, 2026
Copy link

@bloodfeast bloodfeast left a comment

Choose a reason for hiding this comment

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

Good test coverage

@vittorius vittorius force-pushed the notify-zed-of-error-in-lsp branch from e427076 to 3bceeb0 Compare January 17, 2026 15:25
Base automatically changed from notify-zed-of-error-in-lsp to main January 17, 2026 15:33
@vittorius vittorius force-pushed the add-tests-for-watched-set branch from ae8c397 to 90e4677 Compare January 17, 2026 15:35
@vittorius vittorius merged commit 923092e into main Jan 17, 2026
8 checks passed
@vittorius vittorius deleted the add-tests-for-watched-set branch January 17, 2026 15:39
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.

2 participants