-
Notifications
You must be signed in to change notification settings - Fork 631
Add Allure reports to Garnet Github #1457
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
base: main
Are you sure you want to change the base?
Conversation
…em running in GH action
…g mock data for allure results for now
…ed since mock data
…ng debug and cluster tests
…f putting it on every test
…all the debug code too.
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 pull request adds comprehensive Allure test reporting infrastructure to the Garnet project. Allure is a test reporting framework that provides visual test execution reports with history tracking and detailed categorization.
Key Changes:
- Integration of Allure reporting framework across all test projects
- Creation of AllureTestBase class for test categorization and environment labeling
- Updated CI/CD workflows to generate and publish Allure reports
- Documentation updates for developers on Allure test requirements
Reviewed changes
Copilot reviewed 163 out of 164 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| test/Garnet.test/AllureUtils.cs | New base class for Allure test infrastructure with environment labeling |
| Directory.Packages.props | Added Allure.Net.Commons and Allure.NUnit package dependencies |
| test/Garnet.test/*.cs | Updated 60+ test classes to inherit from AllureTestBase and add [AllureNUnit] attribute |
| .github/workflows/nightly.yml | Major refactoring to support Allure report generation, history tracking, and publishing |
| .github/workflows/ci.yml | Added Allure wiring verification to ensure all tests are properly configured |
| test/Allure/*.ps1 | New PowerShell scripts for Allure data collection and report generation |
| website/docs/dev/onboarding.md | Added documentation requiring Allure-enabled tests for new features |
| README.md | Added badge linking to Allure test report |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
TalZaccai
left a comment
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.
Added some minor comments. Please see why Tsavorite tests are failing...
Fixed |
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 162 out of 163 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Full test reporting using Allure (https://github.com/allure-framework).
See website here: http://10.159.0.74:8080/ Local to DarrenGe machine so expected to be slower than when released.