-
Notifications
You must be signed in to change notification settings - Fork 325
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
Add UWP testhost deps in meta package #1351
Changes from 1 commit
0300647
f84feae
503775a
bc52e0e
5aee3e8
20cd2ed
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,12 +9,24 @@ | |
<requireLicenseAcceptance>true</requireLicenseAcceptance> | ||
<licenseUrl>http://www.microsoft.com/web/webpi/eula/net_library_eula_enu.htm</licenseUrl> | ||
<iconUrl>http://go.microsoft.com/fwlink/?LinkID=288859</iconUrl> | ||
<description>The MSbuild targets and properties for building the .Net core test projects.</description> | ||
<description>The MSbuild targets and properties for building .NET Core, .NET Full framework and Universal Windows test projects.</description> | ||
<copyright>© Microsoft Corporation. All rights reserved.</copyright> | ||
<projectUrl>https://github.com/microsoft/vstest/</projectUrl> | ||
<tags>vstest visual-studio unittest testplatform mstest microsoft test</tags> | ||
<dependencies> | ||
<dependency id="Microsoft.TestPlatform.TestHost" version="$Version$" /> | ||
<dependency id="Microsoft.CodeCoverage" version="1.0.3" /> | ||
<group targetFramework="uap10.0"> | ||
<dependency id="System.ComponentModel.Primitives" version="4.1.0" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why did we choose to not provide TestHost in section? CodeCoverage is not provided because it's not supported in UWP yet. A comment in here may help our future selves :) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. please do not hardcode any version, set them as min version. You can look at objectmodel nuspec, there we specify min version of dependencies. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done |
||
<dependency id="System.ComponentModel.TypeConverter" version="4.1.0" /> | ||
<dependency id="System.Runtime.InteropServices.RuntimeInformation" version="4.0.0" /> | ||
<dependency id="Newtonsoft.Json" version="9.0.1" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. NewtonSoft.Json is also needed by TestHost. How do we keep versions in sync? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Using JsonNetVersion from TestPlatform.Dependencies.props. |
||
</group> | ||
<group targetFramework="netcoreapp1.0"> | ||
<dependency id="Microsoft.TestPlatform.TestHost" version="$Version$" /> | ||
<dependency id="Microsoft.CodeCoverage" version="1.0.3" /> | ||
</group> | ||
<group targetFramework="net45"> | ||
<dependency id="Microsoft.CodeCoverage" version="1.0.3" /> | ||
</group> | ||
</dependencies> | ||
</metadata> | ||
<files> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,6 +12,7 @@ | |
<iconUrl>http://go.microsoft.com/fwlink/?LinkID=288859</iconUrl> | ||
<projectUrl>https://github.com/microsoft/vstest/</projectUrl> | ||
<copyright>© Microsoft Corporation. All rights reserved.</copyright> | ||
<tags>vstest visual-studio unittest testplatform mstest microsoft test</tags> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Searching for packages with There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done. |
||
</metadata> | ||
<files> | ||
<file src="Microsoft.TestPlatform.Build\netstandard1.3\Microsoft.TestPlatform.targets" target="runtimes\any\native" /> | ||
|
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.
suggest: .NET test projects
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.
Done.