Cache command method discovery in convention tests

[lex57ukr]

Summary

Improve convention test efficiency by caching discovered command methods. Reflection now runs once and is shared across all convention tests, making the pattern clearer for contributors adding new tests.

Related Issues

Fixes #141

Changes

• Add thread-safe Lazy<MethodInfo[]> field to cache discovered command methods
• Update existing convention tests to use the cached value
• Update how-to documentation to reflect the caching pattern

[Copilot]

Pull request overview

This PR optimizes convention test performance by introducing a caching mechanism for command method discovery. Instead of running reflection separately for each test, methods are discovered once and shared across all convention tests through a thread-safe lazy initialization pattern.

Changes:

• Added a Lazy<MethodInfo[]> field to cache discovered command methods with thread-safe initialization
• Refactored existing convention tests to use the cached value instead of calling DiscoverCommandMethods() directly
• Updated documentation to explain the caching pattern for future contributors

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tests/Keystone.Cli.UnitTests/Presentation/Cocona/CoconaCommandMethodConventionsTests.cs	Introduced cached lazy field and updated two test methods to consume it
docs/how-to/how-to-convention-tests.md	Updated documentation to reference the new caching pattern and guide contributors to use LazyCommandMethods.Value

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.