microsoft/ApplicationInspector

More Comprehensive Rules Verifier Self-Test Report

Open

#507 ouverte le 31 août 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C# (4 102 stars) (355 forks)batch import
enhancementgood first issue

Description

Rule self-tests only confirm that the rule as a whole matched a sample. It may be the case that a rule looks for multiple different things in the same category and you'd like to have self-tests to audit that you have samples for every pattern in every rule. The same perhaps for conditions.

Other potential enhancements:

  1. Mechanism to indicate where the expected match in the must-match should be

Guide contributeur

Stack technique
csharp
Domaine
testingtooling
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
unclear
Prérequis
C#understanding of JSON rules
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Examine the existing rule self test infrastructure in the repository, specifically how rules are tested via must match samples. Identify if there is a way to ensure every pattern within a rule has a corresponding sample. Consider implementing a report that lists patterns missing sample coverage, similar to the suggestion in the issue. Also look for any prior discussions or PRs referenced in the repository that attempted similar enhancements.