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.

Sign up for GitHub

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

Jump to bottom

Handle Test Information from the New Testing Platform #42494

Merged
merged 16 commits into from
Aug 9, 2024
Merged

Handle Test Information from the New Testing Platform #42494

merged 16 commits into from
Aug 9, 2024

Conversation

mariam-abdulla
Copy link
Member

@mariam-abdulla mariam-abdulla commented Aug 1, 2024 •
edited
Loading

We receive the below:

  1. Test states
  2. Attachments
  3. Test session start/end events
  4. Handshake info (pid, architecture, tfm, os, host type, module path)
    and we send handshake info back.

We also store the output/error data of the testing platform.

@mariam-abdulla mariam-abdulla requested a review from a team as a code owner August 1, 2024 08:51
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-DotNet Test untriaged Request triage from a team member labels Aug 1, 2024
@mariam-abdulla mariam-abdulla changed the title Handle Test Results from the New Testing Platform Handle Test Information from the New Testing Platform Aug 1, 2024
MarcoRossignoli
MarcoRossignoli approved these changes Aug 5, 2024
View reviewed changes
Copy link
Member

@MarcoRossignoli MarcoRossignoli left a comment

Choose a reason for hiding this comment

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

Generally good some nits

@MarcoRossignoli MarcoRossignoli enabled auto-merge (squash) August 9, 2024 15:12
@MarcoRossignoli MarcoRossignoli merged commit f8d2772 into main Aug 9, 2024
37 checks passed
@MarcoRossignoli MarcoRossignoli deleted the dev/mabdullah/HandleTestsInfoFromTestHost branch August 9, 2024 16:23
@Evangelink
Copy link
Member

Relates to #45927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Evangelink Evangelink Evangelink left review comments

@MarcoRossignoli MarcoRossignoli MarcoRossignoli approved these changes

Assignees
No one assigned
Labels
Area-DotNet Test untriaged Request triage from a team member
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mariam-abdulla @Evangelink @MarcoRossignoli