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

Clean up the documentation for the library testing #59401

Merged
merged 15 commits into from
Nov 8, 2021

Conversation

MaximLipnin
Copy link
Contributor

No description provided.

@ghost
Copy link

ghost commented Sep 21, 2021

Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: MaximLipnin
Assignees: -
Labels:

area-Infrastructure-libraries

Milestone: -

MaximLipnin and others added 3 commits September 22, 2021 09:39
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
@mdh1418
Copy link
Member

mdh1418 commented Oct 1, 2021

Can we also add instructions on some guidelines for debugging the tests being ran?

For example using Console.Writeline in various libraries, except System.Private.CoreLib, we need to use Internal.Console.Writeline for iOS/MacCatalyst/tvOS; for Android, we need a few extra steps in order to log these statement lines.

It might also be worth documenting how to use a local xharness build? Would this be the right place to do it? If so, we can put it in the shared general testing README.md in runtime/docs/workflow/testing/libraries.

@akoeplinger
Copy link
Member

/cc @lambdageek who was also looking at the docs recently

@ghost ghost closed this Nov 3, 2021
@ghost
Copy link

ghost commented Nov 3, 2021

Draft Pull Request was automatically closed for inactivity. Please let us know if you'd like to reopen it.

@steveisok steveisok reopened this Nov 4, 2021
@marek-safar marek-safar marked this pull request as ready for review November 4, 2021 07:27
@MaximLipnin
Copy link
Contributor Author

Let's merge this version for now. If there are any other important details, please let me know and I'll add them to the docs.

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