You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After #1494 we now have code coverage reports generated in CI. While overall our coverage percentage is high (over 80%) I noticed a few areas that were essentially untested while scrolling through the reports.
Partial Evaluation commands in the CLI
EST sub_entity_literals function
json_schema.rs functions fully_qualify_type_references and conditionally_qualify_type_references
Test coverage could improve in other areas, but these should be easy wins. Feel free to comment with other areas in need of testing, but try to limit the scope of this issue to function with essentially zero coverage
Describe alternatives you've considered
No response
Additional context
No response
Is this something that you'd be interested in working on?
👋 I may be able to implement this internal improvement
⚠️ This feature might incur a breaking change
The text was updated successfully, but these errors were encountered:
john-h-kastner-aws
added
internal-improvement
Refactoring, minor performance improvement, or other changes that Cedar users may never notice
pending-triage
The cedar maintainers haven't looked at this yet. Automicaly added to all new issues.
and removed
pending-triage
The cedar maintainers haven't looked at this yet. Automicaly added to all new issues.
labels
Mar 3, 2025
Describe the improvement you'd like to request
After #1494 we now have code coverage reports generated in CI. While overall our coverage percentage is high (over 80%) I noticed a few areas that were essentially untested while scrolling through the reports.
sub_entity_literals
functionjson_schema.rs
functionsfully_qualify_type_references
andconditionally_qualify_type_references
Test coverage could improve in other areas, but these should be easy wins. Feel free to comment with other areas in need of testing, but try to limit the scope of this issue to function with essentially zero coverage
Describe alternatives you've considered
No response
Additional context
No response
Is this something that you'd be interested in working on?
The text was updated successfully, but these errors were encountered: