-
Notifications
You must be signed in to change notification settings - Fork 339
Remove CUIT (Coded UI Test) from NuGet packages and test projects #15177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: drognanar <[email protected]>
…ject Co-authored-by: drognanar <[email protected]>
test/TestAssets/TestAssets.sln
Outdated
| {CF46C8A0-E9FA-40E9-96CA-DCD3797546D8}.Debug|Any CPU.Build.0 = Debug|Any CPU | ||
| {CF46C8A0-E9FA-40E9-96CA-DCD3797546D8}.Release|Any CPU.ActiveCfg = Release|Any CPU | ||
| {CF46C8A0-E9FA-40E9-96CA-DCD3797546D8}.Release|Any CPU.Build.0 = Release|Any CPU | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot please remove the empty lines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty lines from the TestAssets.sln file. Fixed in commit 8b4d2c1.
test/TestAssets/TestAssets.sln
Outdated
| EndProject | ||
| Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "CUITTestProject", "CUITTestProject\CUITTestProject.csproj", "{CF46C8A0-E9FA-40E9-96CA-DCD3797546D8}" | ||
| EndProject | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot please remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The empty line at line 34 in TestAssets.sln was already removed in commit f2bb4bc.
| </ItemGroup> | ||
| </When> | ||
| </Choose> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot please remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line at line 57. Fixed in commit 3fee095.
| <VSToolsPath Condition="'$(VSToolsPath)' == ''">$(MSBuildExtensionsPath32)\Microsoft\VisualStudio\v$(VisualStudioVersion)</VSToolsPath> | ||
| <ReferencePath>$(ProgramFiles)\Common Files\microsoft shared\VSTT\$(VisualStudioVersion)\UITestExtensionPackages</ReferencePath> | ||
| <IsCodedUITest>False</IsCodedUITest> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot please remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line at line 20. Fixed in commit 3fee095.
Co-authored-by: drognanar <[email protected]>
Co-authored-by: drognanar <[email protected]>
| <PackageReference Include="Microsoft.VisualStudio.TraceDataCollector" Version="$(MicrosoftInternalCodeCoverageVersion)" PrivateAssets="All" GeneratePathProperty="true" /> | ||
| <PackageReference Include="Microsoft.Internal.CodeCoverage" Version="$(MicrosoftInternalCodeCoverageVersion)" PrivateAssets="All" GeneratePathProperty="true" /> | ||
| <PackageReference Include="Microsoft.VisualStudio.CUIT" Version="$(TestPlatformExternalsVersion)" PrivateAssets="All" GeneratePathProperty="true" /> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line. Fixed in commit c0d5dd0.
| <MicrosoftInternalTestPlatformExtensionsFiles Include="$(PkgMicrosoft_Internal_TestPlatform_Extensions)\tools\net451\**\*"></MicrosoftInternalTestPlatformExtensionsFiles> | ||
| <MicrosoftInternalCodeCoverageFiles Include="$(PkgMicrosoft_Internal_CodeCoverage)\contentFiles\any\any\**\*"></MicrosoftInternalCodeCoverageFiles> | ||
| <MicrosoftVisualStudioCUITFiles Include="$(PkgMicrosoft_VisualStudio_CUIT)\tools\net451\**\*"></MicrosoftVisualStudioCUITFiles> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line. Fixed in commit c0d5dd0.
| <Copy SourceFiles="@(MicrosoftInternalTestPlatformExtensionsFiles)" DestinationFiles="$(OutDir)\Microsoft.Internal.TestPlatform.Extensions\%(RecursiveDir)%(Filename)%(Extension)" /> | ||
| <Copy SourceFiles="@(MicrosoftInternalCodeCoverageFiles)" DestinationFiles="$(OutDir)\Microsoft.Internal.CodeCoverage\%(RecursiveDir)%(Filename)%(Extension)" /> | ||
| <Copy SourceFiles="@(MicrosoftVisualStudioCUITFiles)" DestinationFolder="$(OutDir)\Microsoft.VisualStudio.CUIT" /> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line. Fixed in commit c0d5dd0.
| <file src="net48\Microsoft.VisualStudio.TestPlatform.ObjectModel.dll" target="tools\net462\Common7\IDE\Extensions\TestPlatform\Microsoft.VisualStudio.TestPlatform.ObjectModel.dll" /> | ||
| <file src="net48\Microsoft.VisualStudio.QualityTools\Microsoft.VisualStudio.TestTools.UITest.Extension.IE.Communication.sxs.manifest" target="tools\net462\Common7\IDE\Extensions\TestPlatform\Microsoft.VisualStudio.TestTools.UITest.Extension.IE.Communication.sxs.manifest" /> | ||
| <file src="net48\Microsoft.VisualStudio.QualityTools\Microsoft.VisualStudio.TestTools.UITest.Playback.Engine.sxs.manifest" target="tools\net462\Common7\IDE\Extensions\TestPlatform\Microsoft.VisualStudio.TestTools.UITest.Playback.Engine.sxs.manifest" /> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line. Fixed in commit c0d5dd0.
| <file src="net48\Microsoft.VisualStudio.CUIT\Microsoft.VisualStudio.TestTools.UITest.Extension.Uia.dll" target="tools\net462\Common7\IDE\Extensions\TestPlatform\CUITPlugins\Microsoft.VisualStudio.TestTools.UITest.Extension.Uia.dll" /> | ||
| <file src="net48\Microsoft.VisualStudio.CUIT\Microsoft.VisualStudio.TestTools.UITest.Extension.UiaWidget.dll" target="tools\net462\Common7\IDE\Extensions\TestPlatform\CUITPlugins\Microsoft.VisualStudio.TestTools.UITest.Extension.UiaWidget.dll" /> | ||
| <file src="net48\Microsoft.VisualStudio.CUIT\Microsoft.VisualStudio.TestTools.UITest.Extension.UiaWidget.UIAHtmlElementUtilities.dll" target="tools\net462\Common7\IDE\Extensions\TestPlatform\CUITPlugins\Microsoft.VisualStudio.TestTools.UITest.Extension.UiaWidget.UIAHtmlElementUtilities.dll" /> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line. Fixed in commit c0d5dd0.
| <TestPlatformBinFolder>$(ArtifactsBinDir)Microsoft.TestPlatform\$(Configuration)\$(NetFrameworkRunnerTargetFramework)\</TestPlatformBinFolder> | ||
| <MsDiaFolder>$(TestPlatformBinFolder)Microsoft.Internal.Dia\</MsDiaFolder> | ||
| <CUITFolder>$(TestPlatformBinFolder)Microsoft.VisualStudio.CUIT\</CUITFolder> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line. Fixed in commit c0d5dd0.
| <VsixSourceItem Include="$(CUITFolder)Microsoft.VisualStudio.TestTools.UITesting.dll" /> | ||
| <VsixSourceItem Include="$(CUITFolder)UIAComWrapper.dll" /> | ||
| </ItemGroup> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the empty line. Fixed in commit c0d5dd0.
Co-authored-by: drognanar <[email protected]>
|
@copilot the pipeline failed with error below please fix by updating the value in verify-nupkg to 586. |
Co-authored-by: drognanar <[email protected]>
This PR removes Microsoft.VisualStudio.CUIT from the vstest NuGet packages and eliminates CUIT-related test projects from the repository as part of the CUIT deprecation effort.
Changes Made
Removed CUIT Test Project
test/TestAssets/CUITTestProject/directory and all its contentsCUITTestProjectfromTestAssets.slnsolution fileCUITTest.csintegration test classUpdated NuGet Packages
Microsoft.TestPlatform.nuspec(30+ DLL and configuration files)Microsoft.VisualStudio.CUITpackage reference fromMicrosoft.TestPlatform.csprojUpdated CLI Package
Microsoft.VisualStudio.TestTools.TestPlatform.V2.CLI.csprojCleaned Up Existing Projects
BingWebTest.csprojIsCodedUITestproperty and Choose/When blocks for CUIT dependenciesImpact
Verification
Fixes #15176.
Warning
Firewall rules blocked me from connecting to one or more addresses
I tried to connect to the following addresses, but was blocked by firewall rules:
1javsblobprodcus364.vsblob.vsassets.iodotnet restore --no-cache(dns block)1k9vsblobprodcus379.vsblob.vsassets.iodotnet restore --no-cache(dns block)1s1vsblobprodcus386.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)2zrvsblobprodcus388.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)4myvsblobprodcus32.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)dotnet restore --no-cache(dns block)4vyvsblobprodcus361.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)51yvsblobprodcus36.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)5dkvsblobprodcus355.vsblob.vsassets.iodotnet restore --no-cache(dns block)7tjvsblobprodcus341.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)80zvsblobprodcus35.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)8xbvsblobprodcus382.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)c50vsblobprodcus330.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)dotnet restore --no-cache(dns block)cflvsblobprodcus383.vsblob.vsassets.iodotnet restore --no-cache(dns block)d0svsblobprodcus381.vsblob.vsassets.iodotnet restore --no-cache(dns block)fdpvsblobprodcus345.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)frdvsblobprodcus327.vsblob.vsassets.iodotnet restore --no-cache(dns block)h6tvsblobprodcus346.vsblob.vsassets.iodotnet restore --no-cache(dns block)imzvsblobprodcus368.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)jd4vsblobprodcus366.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)jrqvsblobprodcus343.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)kmuvsblobprodcus389.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)l49vsblobprodcus358.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)l7avsblobprodcus319.vsblob.vsassets.iodotnet restore --no-cache(dns block)lylvsblobprodcus31.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)m6xvsblobprodcus342.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)dotnet restore --no-cache(dns block)mfjvsblobprodcus373.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)o3svsblobprodcus318.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)ofvvsblobprodcus315.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)dotnet restore --no-cache(dns block)pe3vsblobprodcus354.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)dotnet restore --no-cache(dns block)pe4vsblobprodcus351.vsblob.vsassets.iodotnet restore --no-cache(dns block)/home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js(dns block)pkvvsblobprodcus321.vsblob.vsassets.iodotnet restore --no-cache(dns block)sc4vsblobprodcus331.vsblob.vsassets.iodotnet restore --no-cache(dns block)uy6vsblobprodcus34.vsblob.vsassets.iodotnet restore --no-cache(dns block)xupvsblobprodcus332.vsblob.vsassets.iodotnet build test/TestAssets/TestAssets.sln --configuration Release(dns block)dotnet restore --no-cache(dns block)ytvvsblobprodcus310.vsblob.vsassets.iodotnet restore --no-cache(dns block)If you need me to access, download, or install something from one of these locations, you can either:
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.