Skip to content

Conversation

@vagisha-nidhi
Copy link
Contributor

@vagisha-nidhi vagisha-nidhi commented Jan 31, 2019

Description

Test projects targeting .NetFramework 3.5 throw error.
Change : We will run these tests in compat mode(default using .NetFramework 4.0)
We are unable to get information from assembly metadata for a .NetFramework 3.5 project, hence by default .NetFramework 4.0 is chosen as the running framework.

image

Related issue

#1896

</data>
<data name="Framework35NotSupported" xml:space="preserve">
<value>Framework35 is not supported. For projects targeting .Net Framework 3.5, please use Framework40 to run tests in CLR 4.0 "compatibility mode".</value>
<value>Framework35 is not supported. For projects targeting .Net Framework 3.5, tests are running in "compatibility mode".</value>
Copy link
Contributor

Choose a reason for hiding this comment

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

Framework35 is not supported. For projects targeting .Net Framework 3.5, test will run in CLR 4.0 "compatibility mode".

Copy link
Contributor

@singhsarab singhsarab left a comment

Choose a reason for hiding this comment

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

Please change the string and attach the final output users sees. Approving with suggestions.

@singhsarab
Copy link
Contributor

@vagisha-nidhi Please validate the VS scenario before pushing this in.

@vagisha-nidhi vagisha-nidhi merged commit e16098b into microsoft:master Jan 31, 2019
singhsarab pushed a commit to singhsarab/vstest that referenced this pull request Mar 5, 2019
* Bug Fix running NETFramework 3.5 in compat mode
singhsarab added a commit that referenced this pull request Mar 5, 2019
* Translation Layer connection timeout (#1843)

* Adding the missing assemblyInfo files and updating the copyrights (#1859)

* Stop trying to connect if the test host exits unexpectedly  (#1853)

* Running NETFramework 3.5 in compat mode (#1906)

* Fixing the timeouts. (#1909)

* Fixing the connection timeout (#1910)

* TestPlatform.PR (#1837)
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