Implement request-safe assessment cache with TTL and tests

[nicolasgioanni]

This pull request refactors the assessment configuration caching logic in gohighlevel-assessment.ts to avoid sharing in-flight asynchronous discovery operations between parallel requests. It also adds new tests to ensure that assessment configuration discovery is not shared and that failed discoveries are retried correctly. The changes improve cache safety, correctness, and test coverage.

Assessment Configuration Caching Improvements

• Refactored the assessment configuration cache to store the resolved AssessmentConfiguration object directly instead of a promise, ensuring that parallel calls do not share in-flight discovery operations. Now, each call to getAssessmentConfiguration will independently trigger discovery if the cache is expired or empty, preventing accidental sharing of incomplete or failed discoveries.
• Updated the test cache reset function to clear the new assessmentConfiguration variable instead of the removed promise.

Test Coverage Enhancements

• Added a test to verify that in-flight assessment configuration discovery is not shared between parallel requests, ensuring that each request independently fetches configuration as needed. (Fb7f17b9L1040R1040)
• Added a test to confirm that the system retries assessment configuration discovery after a failed attempt, by simulating a failure and ensuring a subsequent successful discovery is possible. (Fb7f17b9L1040R1040)
• Improved an existing test to suppress error logs for cleaner test output.

[nicolasgioanni]

Fix implemented