Skip to content

test: add integration test for FeatureRepository with @Memoize decorator#27613

Merged
sean-brydon merged 1 commit intomainfrom
devin/1770214701-memoize-integration-test
Feb 4, 2026
Merged

test: add integration test for FeatureRepository with @Memoize decorator#27613
sean-brydon merged 1 commit intomainfrom
devin/1770214701-memoize-integration-test

Conversation

@eunjae-lee
Copy link
Copy Markdown
Contributor

@eunjae-lee eunjae-lee commented Feb 4, 2026

What does this PR do?

Adds integration tests for the FeatureRepository DI container to verify the @Memoize decorator behavior in CachedFeatureRepository.

The test exercises the getFeatureRepository() function from packages/features/di/containers/FeatureRepository.ts which returns a CachedFeatureRepository instance that uses @Memoize and @Unmemoize decorators for Redis caching.

Test coverage includes:

  • Repository instance caching via DI container
  • findBySlug with memoization
  • findAll with memoization
  • update with @Unmemoize cache invalidation
  • checkIfFeatureIsEnabledGlobally

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A - test only.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

Run the integration test locally:

VITEST_MODE=integration TZ=UTC yarn test packages/features/di/containers/FeatureRepository.integration-test.ts

Checklist

  • My code follows the style guidelines of this project
  • I have checked if my changes generate no new warnings

Link to Devin run: https://app.devin.ai/sessions/f702705bd61f4b98a2abc0c3cb26a7e3
Requested by: @eunjae-lee

@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@paragon-review
Copy link
Copy Markdown

paragon-review bot commented Feb 4, 2026

Paragon Review Unavailable

Hi @eunjae-lee! You are not toggled active by your team. Reach out to your admin.

@paragon-review
Copy link
Copy Markdown

paragon-review bot commented Feb 4, 2026

Paragon Review Unavailable

Hi @eunjae-lee! You are not toggled active by your team. Reach out to your admin.

@devin-ai-integration devin-ai-integration bot marked this pull request as ready for review February 4, 2026 14:37
@graphite-app graphite-app bot requested a review from a team February 4, 2026 14:37
@graphite-app graphite-app bot added consumer core area: core, team members only labels Feb 4, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@sean-brydon sean-brydon merged commit 3f4109d into main Feb 4, 2026
138 of 141 checks passed
@sean-brydon sean-brydon deleted the devin/1770214701-memoize-integration-test branch February 4, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consumer core area: core, team members only ready-for-e2e size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants