feat: validate GenericRedisCacheContext

[wschurman]

Why

#601 exposed another validation gap in assumptions made about cache keys. We need certain assumptions to be true, namely:

• delimiter is nonzero-length (otherwise issue fix!: replace makeKeyFn with delimiter to strengthen guarantee of proper escaping #601 fixes still would exist)
• delimiter doesn't include escape character (otherwise escape breaks)
• ttlSecondsPositive and negative are positive integers
• cacheKeyVersionsToInvalidateFn returns > 0 versions (otherwise silently doesn't invalidate)

How

Add validation.

Test Plan

Add test.

[wschurman]

• fix: correctness of TSMockitoExtensions isEqualWithEntityAware for SerializableKeyMaps of different length #614
• fix: fix possible composite key collision in StubDatabaseAdapter test utility #613
• chore: update comment for IEntityMetricsAdapter for implementors #612
• fix: pick method crash when object has hasOwnProperty key #611
• fix: fix behavior of AuthorizationResultBasedEntityAssociationLoader when field value is empty string #610
• feat: support numeric enums in StrictEnumField #609
• feat: validate GenericRedisCacheContext #608 👈 (View in Graphite)
• fix: skip when AllowIfAllSubRulesAllowPrivacyPolicyRule is empty #607
• fix: correct cycle guard behavior for deletion #606
• fix: handle potential infinite recursion for canViewerDeleteAsync #605
• feat: clean up transactional dataloader at transaction end #604
• fix: correct negative caching behavior in GenericSecondaryEntityCache #603
• fix: update cache key generation for secondary caches #602
• fix!: replace makeKeyFn with delimiter to strengthen guarantee of proper escaping #601
• fix: prevent collision in companion when entities share same name or same table name #600
• chore: remove unnecessary ViewerScopedEntityMutatorFactory #599
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (717d203) to head (c41a1b8).

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #608   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          109       109           
  Lines        17699     17729   +30     
  Branches      1526      1535    +9     
=========================================
+ Hits         17699     17729   +30     

Flag	Coverage Δ
integration	28.64% <30.55%> (-0.01%)	⬇️
unittest	94.52% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[wschurman]

Merge activity

• May 21, 4:55 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• May 21, 5:21 PM UTC: Graphite rebased this pull request as part of a merge.
• May 21, 5:22 PM UTC: @wschurman merged this pull request with Graphite.