Update .claude/skills/review-pr/SKILL.md

Co-authored-by: Guillaume CARRE <122804551+g-carre@users.noreply.github.com>

Author: ChengYanJin

.claude/skills/review-pr/SKILL.md

@@ -52,7 +52,9 @@ gh pr diff <number> --repo <owner/repo>
 | Python / buildchain | Correct doit task dependencies and file targets; proper pytest fixtures; behave step definitions match feature files; no hardcoded paths (use constants.py) |
 | Security | No secrets/tokens in plain text; no OWASP-relevant issues (injection, XSS, insecure defaults) |
 | Breaking changes | Public API changes, CRD schema changes, Salt pillar schema changes, UI prop interface changes |
-
+| Test coverage | New logic paths have corresponding unit tests; changed functions have updated tests; no test files deleted without replacement, no orphan test that would be never run |
+| Test impact | Changes to shared code (`map.jinja`, `constants.py`, CRD types, API types) are reflected in dependent tests across all affected layers |
+| BDD / integration | New user-facing behavior has matching behave scenarios in `tests/`; modified Salt orchestrations have updated integration steps |
 4. **Deliver your review:**
 
 ### If CI mode: post to GitHub