Skip to content
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

[iOS][tvOS][tests] Re add ios tvos simulator functional tests #52606

Merged
merged 2 commits into from
May 12, 2021

Conversation

mdh1418
Copy link
Member

@mdh1418 mdh1418 commented May 11, 2021

The iOS simulator and tvOS simulator AOT functional tests had been removed in a previous commit, but instead, they will be brought back for local development usage, and skipped on CI until we add a way to bypass the Global property cannot be modified error.

@ghost
Copy link

ghost commented May 11, 2021

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

The iOS simulator and tvOS simulator AOT functional tests had been removed in a previous commit, but instead, they will be brought back for local development usage, and skipped on CI until we add a way to bypass the Global property cannot be modified error.

Author: mdh1418
Assignees: -
Labels:

area-System.Net

Milestone: -

@@ -0,0 +1,18 @@
<Project Sdk="Microsoft.NET.Sdk">
Copy link
Member

Choose a reason for hiding this comment

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

We could try something like

<Project Sdk="Microsoft.NET.Sdk" TreatAsLocalProperty="MonoForceInterpreter" >

and then set MonoForceInterpreter to false in this project.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll try that in a follow up PR, one to just target running this particular test project on CI.


<!-- Can't AOT in interp mode -->
<ProjectExclusions Condition="'$(MonoForceInterpreter)' == 'true'"
Include="$(RepoRoot)\src\tests\FunctionalTests\iOS\Simulator\AOT\**\*.Test.csproj" />
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Perhaps, no need to add a backslash after $(RepoRoot)

Copy link
Member Author

Choose a reason for hiding this comment

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

It seems the other instances of $(RepoRoot) are of similar style

@mdh1418 mdh1418 force-pushed the add_ios_tvos_simulator_functional_tests branch 2 times, most recently from ade150d to 65eadfd Compare May 12, 2021 16:23
@mdh1418
Copy link
Member Author

mdh1418 commented May 12, 2021

The 3 failing lanes in the previous run (before having to fix the merge conflict in tests.proj) were unrelated.

@mdh1418
Copy link
Member Author

mdh1418 commented May 12, 2021

The failing lane is unrelated

@mdh1418 mdh1418 merged commit 6e89441 into dotnet:main May 12, 2021
@mdh1418 mdh1418 deleted the add_ios_tvos_simulator_functional_tests branch May 12, 2021 19:00
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jun 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants