-
-
Notifications
You must be signed in to change notification settings - Fork 0
feat: Add PublicApiAttribute for marking PublicApi test definitions #967
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
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
715f981 to
0848f06
Compare
Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
0848f06 to
b1bef72
Compare
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #967 +/- ##
==========================================
+ Coverage 83.54% 83.82% +0.28%
==========================================
Files 84 88 +4
Lines 231 235 +4
Branches 17 17
==========================================
+ Hits 193 197 +4
Misses 32 32
Partials 6 6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Implements
PublicApiAttributeacross all test frameworks (TUnit, XUnit.V3, MSTest, NUnit) to enable consistent marking of PublicApi definitions in test projects, mirroring the existingFunctionalTestAttributepattern.Changes
PublicApiAttributeto each framework, inheriting from the appropriate base (CategoryTraitBaseAttribute,TestTraitBaseAttribute, orCategoryAttribute)PublicApiconstant toInternals.csin each frameworkUsage
The attribute follows the same semantics as other test categorization attributes: applicable to Assembly, Class, or Method with
AllowMultiple = true.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.