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

Cleanup functional tests / Structure them like libraries tests #46810

Merged
merged 20 commits into from
Jan 15, 2021

Conversation

steveisok
Copy link
Member

@steveisok steveisok commented Jan 11, 2021

You should now be able to run the functional tests locally the same way as running the libraries tests.

Example:
./dotnet.sh build /t:Test src/tests/FunctionalTests/wasm/Interpreter/console/ /p:TargetOS=Browser /p:TargetArchitecture=wasm /p:Configuration=Release

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost
Copy link

ghost commented Jan 11, 2021

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

Issue Details
Author: steveisok
Assignees: -
Labels:

area-Infrastructure-mono

Milestone: -

MaximLipnin and others added 8 commits January 12, 2021 14:46
# Conflicts:
#	src/tests/FunctionalTests/wasm/AOT/console/Wasm.Aot.Console.Test.csproj
#	src/tests/FunctionalTests/wasm/Interpreter/browser/Wasm.Interpreter.Browser.Test.csproj
#	src/tests/FunctionalTests/wasm/Interpreter/console/Wasm.Interpreter.Console.Test.csproj
Copy link
Member

@akoeplinger akoeplinger left a comment

Choose a reason for hiding this comment

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

Just a couple questions, but looks great otherwise!

src/libraries/tests.proj Show resolved Hide resolved
src/tests/FunctionalTests/iOS/Simulator/AOT/Program.cs Outdated Show resolved Hide resolved
eng/testing/tests.targets Show resolved Hide resolved
@steveisok steveisok merged commit c260375 into dotnet:master Jan 15, 2021
@steveisok steveisok deleted the cleanup-func-tests branch January 15, 2021 13:36
@ghost ghost locked as resolved and limited conversation to collaborators Feb 14, 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.

5 participants