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

Add a console logger sample for the runner #2246

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

Evangelink
Copy link
Member

@Evangelink Evangelink commented Feb 2, 2024

Contributes to #2242

Provides a sample of extending the platform to provide a custom report generator through some simple (naive) console logger.

@Evangelink Evangelink merged commit e33d8d0 into microsoft:main Feb 2, 2024
7 checks passed
@Evangelink Evangelink deleted the sample-console-logger branch February 2, 2024 15:20
@ray440
Copy link

ray440 commented Feb 2, 2024

Just checked TestResultConsoleLogger. That was quick! 👍

Looks like we can log the results comprehensively.

There's no suite & method info. Can the logger also incorporate something like ITestDiscoveryEventsHandler?

BTW: I tried to compile this project but it's setup to use net9.0 alpha. Is this feature ready for .net8.0?

@Evangelink
Copy link
Member Author

BTW: I tried to compile this project but it's setup to use net9.0 alpha. Is this feature ready for .net8.0?

That's a mistake from me :) Fixing here #2254

There's no suite & method info. Can the logger also incorporate something like ITestDiscoveryEventsHandler?

This interface shows when tests are discovered, you can do the same also with the TestNodeStateProperty.

@ray440
Copy link

ray440 commented Feb 5, 2024

That's a mistake from me :) Fixing here #2254

Right I should have known. I also needed to update global.json.

This interface shows when tests are discovered, you can do the same also with the TestNodeStateProperty.

OK got it to compile.
Looks like I can get suite/method info by searching TestNode.Properties

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.

None yet

2 participants