Skip to content

Conversation

@acesiddhu
Copy link
Contributor

@acesiddhu acesiddhu commented Jan 29, 2018

  1. Added CUIT package in VS VSIX

Description

Enable packaging of CUIT package to run CUIT test in xcopy mode in vstest package
Also included this in vsix.
list of dlls

Microsoft.VisualStudio.Diagnostics.Measurement.dll
Microsoft.VisualStudio.QualityTools.Sqm.dll
Interop.UIAutomationClient.dll
Microsoft.ShDocVw.dll
Microsoft.VisualStudio.OLE.Interop.dll
Microsoft.VisualStudio.QualityTools.CodedUITestFramework.dll
Microsoft.VisualStudio.TestTools.UITest.Common.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.MSAA.dll
Microsoft.VisualStudio.TestTools.UITest.ExtensionUtilities.dll
Microsoft.VisualStudio.TestTools.UITest.Framework.dll
Microsoft.VisualStudio.TestTools.UITest.Logging.dll
Microsoft.VisualStudio.TestTools.UITest.Playback.dll
Microsoft.VisualStudio.TestTools.UITest.WindowsStoreUtility.dll
Microsoft.VisualStudio.TestTools.UITesting.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE.Communication.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE.Communication.Interop.dll
Microsoft.VisualStudio.TestTools.UITest.Playback.Engine.dll
Microsoft.VisualStudio.TestTools.UITest.Synchronization.dll
UIAComWrapper.dll
IEPropertyConfiguration.xml
IEPropertyConfiguration.xsd
Microsoft.VisualStudio.TestTools.UITest.Extension.CrossBrowser.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE.EventHelper.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE64.Communication.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE64.EventHelper.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE.Communication.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.IE.Communication.Interop.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.Uia.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.UiaWidget.dll
Microsoft.VisualStudio.TestTools.UITest.Extension.UiaWidget.UIAHtmlElementUtilities.dll

Related issue

#1353

2) Added CUIT package in VS VSIX
Copy link
Contributor

@smadala smadala left a comment

Choose a reason for hiding this comment

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

Please list the dll/files we are bringing with this package in PR description.

  • update sign.proj and verify-sign.ps1 if requried.
  • Add acceptance test if this enables any scenario

@acesiddhu
Copy link
Contributor Author

@smadala added list of dlls
we cant add acceptance tests because CUIT needs to have ui access to machine and machine should have active desktop session.

I will check the sign proj thingi

</ItemGroup>
<ItemGroup>
<PackageReference Include="Microsoft.VisualStudio.cuit">
<Version>15.6.0-preview-1329213</Version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Use TestPlatformExternalsVersion.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not importing that props because that guy includes mstest v2 as well. my test needs mstest v1 reference

// Copyright (c) Microsoft Corporation. All rights reserved.
// Licensed under the MIT license. See LICENSE file in the project root for full license information.

namespace BlameUnitTestProject
Copy link
Contributor

Choose a reason for hiding this comment

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

namespace.

using Microsoft.VisualStudio.TestTools.UnitTesting;
using Microsoft.VisualStudio.TestTools.UITest.Extension;
using Keyboard = Microsoft.VisualStudio.TestTools.UITesting.Keyboard;
using System.Threading;
Copy link
Contributor

Choose a reason for hiding this comment

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

Threading required?

@acesiddhu acesiddhu merged commit a0f0183 into microsoft:master Jan 31, 2018
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