-
Notifications
You must be signed in to change notification settings - Fork 178
Add telemetry for test results #2048
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 telemetry for test results #2048
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds telemetry tracking for test results across three test types (standard tests, page scripting tests, and BCPT tests), enabling AL-Go partners to monitor test execution metrics over time. The implementation collects test counts (total, passed, failed, skipped) and execution time, emitting telemetry events that can be queried through Application Insights.
Key changes:
- Added telemetry emission for test result metrics in
TestResultAnalyzer.ps1 - Updated documentation with KQL query examples and telemetry event specifications
- Tracked metrics include: TotalTests, TotalPassed, TotalFailed, TotalSkipped, and TotalTime
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
| Scenarios/EnablingTelemetry.md | Added documentation for test result telemetry events including KQL query examples and dimension descriptions |
| Actions/AnalyzeTests/TestResultAnalyzer.ps1 | Imported TelemetryHelper module and added telemetry emission logic to three test result analysis functions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
…rup/enrichedtelemetry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
|
❔What, Why & How
It might be interesting to track the number of tests being run in your workflow over time, and to see whether tests are failing.
This PR would add that to the existing AL-Go telemetry. In the near future we could look into adding a page to the starter-dashboard as well, so an AL-Go partner could quite easily get an overview of their test runs across all their repos.
✅ Checklist