Skip to content

Add unit tests for LeadingCapture#1361

Merged
matux merged 2 commits into
masterfrom
matux/unit-tests-lead
Oct 10, 2025
Merged

Add unit tests for LeadingCapture#1361
matux merged 2 commits into
masterfrom
matux/unit-tests-lead

Conversation

@matux

@matux matux commented Oct 10, 2025

Copy link
Copy Markdown
Contributor

Description of the change

Adds dedicated unit test suite for the LeadingCapture component extracted in the previous PR.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Maintenance
  • New release

@matux matux requested a review from Copilot October 10, 2025 11:41
@matux matux self-assigned this Oct 10, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a comprehensive unit test suite for the LeadingCapture component that was extracted in a previous PR. The tests ensure proper functionality of replay capture scheduling, exporting, and state management.

  • Comprehensive test coverage for all public methods and edge cases
  • Mock setup for dependencies including recorder, tracing, and telemetry components
  • Integration tests for the complete capture workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@matux matux force-pushed the matux/unit-tests-lead branch from c205372 to 7f9b77f Compare October 10, 2025 11:43
Base automatically changed from matux/extract-capture to master October 10, 2025 13:10
@matux matux merged commit 0a7e97a into master Oct 10, 2025
6 checks passed
@matux matux deleted the matux/unit-tests-lead branch October 10, 2025 13:11
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.

3 participants