Skip to content

Revert removal of Video Recorder#15336

Merged
nohwnd merged 3 commits intomicrosoft:mainfrom
nohwnd:video-recorder
Mar 17, 2026
Merged

Revert removal of Video Recorder#15336
nohwnd merged 3 commits intomicrosoft:mainfrom
nohwnd:video-recorder

Conversation

@nohwnd
Copy link
Copy Markdown
Member

@nohwnd nohwnd commented Nov 27, 2025

Description

Puts back Video and Voice Recorder data collector for VSTest that was incorrectly removed as part of TPv0 removal.

Related issue

Fix #15292

@nohwnd
Copy link
Copy Markdown
Member Author

nohwnd commented Nov 27, 2025

@copilot can you add an integration test that will use the data collector (use the example in the added MD file), and then check in TestResults folder for presence of non-empty .wmv file. This is a windows only tool, so make the test windows only.

Copy link
Copy Markdown
Member

@Youssef1313 Youssef1313 left a comment

Choose a reason for hiding this comment

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

Can we have a test here?

@nohwnd
Copy link
Copy Markdown
Member Author

nohwnd commented Dec 9, 2025

Don't close this. I will get back to it.

Copilot AI review requested due to automatic review settings March 16, 2026 14:05
Copy link
Copy Markdown
Contributor

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

Restores the Video and Voice Recorder data collector artifacts into the VSTest VSIX and Microsoft.TestPlatform NuGet package after they were removed unintentionally during TPv0 cleanup.

Changes:

  • Re-add VideoRecorder extension content to the VSIX packaging inputs.
  • Add VideoRecorder binaries to Microsoft.TestPlatform.nuspec and update package verification to account for the new exe.
  • Add documentation explaining how to use the Video and Voice Recorder data collector.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
src/package/Microsoft.VisualStudio.TestTools.TestPlatform.V2.CLI/Microsoft.VisualStudio.TestTools.TestPlatform.V2.CLI.csproj Includes VideoRecorder extension folder in the VSIX packaging inputs.
src/package/Microsoft.TestPlatform/Microsoft.TestPlatform.nuspec Packages VideoRecorder engine/exe and related assemblies into the NuGet tools layout.
eng/verify-nupkgs.ps1 Updates NuPkg verification (file count + exe arch list) for the added VideoRecorder payload.
docs/extensions/video-and-voice-recorder.md Documents usage and shipped file layout for the Video/Voice recorder collector.

Comment thread eng/verify-nupkgs.ps1 Outdated
Comment thread src/package/Microsoft.TestPlatform/Microsoft.TestPlatform.nuspec
Comment thread docs/extensions/video-and-voice-recorder.md
Comment thread docs/extensions/video-and-voice-recorder.md
@nohwnd
Copy link
Copy Markdown
Member Author

nohwnd commented Mar 16, 2026

Test is added.


if (_useVsixRunner)
{
var runnerInfo = new RunnerInfo
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I want to refine this later, the runner info is not super informative about what is being run.

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.

vstest.console.exe v18 no longer finds Screen and Voice Recorder.

3 participants