Skip to content

Comments

Re-enabled CT#450

Open
skywojciechowskim wants to merge 1 commit intomasterfrom
FreeTimeFixes
Open

Re-enabled CT#450
skywojciechowskim wants to merge 1 commit intomasterfrom
FreeTimeFixes

Conversation

@skywojciechowskim
Copy link
Contributor

Summary: Re-enabled CT
Type: Fix
Test Plan: UT/CT, Fullstack
Jira: NO-JIRA

Copilot AI review requested due to automatic review settings February 20, 2026 12:47
@github-actions
Copy link

Pull request must be merged with a description containing the required fields,

Summary:
Type: Feature/Fix/Cleanup
Test Plan:
Jira:

If there is no jira releated to this change, please put 'Jira: NO-JIRA'.

Description can be changed by editing the top comment on your pull request and making a new commit.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Re-enables the previously disabled Flush component test by adding missing wrapper expectations so it can run reliably again in the MediaPipeline component test suite.

Changes:

  • Re-enabled the flushAudioSourceSuccess component test (removed DISABLED_ prefix).
  • Added Glib/GStreamer wrapper expectations for audio sink type name and optional unref calls.
  • Adjusted flush expectations by removing a now-redundant strict gstObjectUnref(m_audioSink) expectation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 85.2%
Functions coverage stays unchanged and is: 92.5%

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.

1 participant