Skip to content

Conversation

@coderabbitai
Copy link
Contributor

@coderabbitai coderabbitai bot commented Oct 30, 2025

Unit test generation was requested by @EggerMarc.

The following files were modified:

  • TESTING.md
  • TEST_SUMMARY.md
  • package.json
  • package/__tests__/adapters/cache.test.ts
  • package/__tests__/adapters/index.test.ts
  • package/__tests__/adapters/queries/get-usage.test.ts
  • package/__tests__/adapters/queries/insert-usage.test.ts
  • package/__tests__/adapters/queries/reset-usage.test.ts
  • package/__tests__/realtime/usage-tracker.test.ts
  • package/__tests__/realtime/websocket-server.test.ts
  • package/__tests__/utils.test.ts

@coderabbitai coderabbitai bot requested a review from EggerMarc October 30, 2025 10:08
@coderabbitai coderabbitai bot mentioned this pull request Oct 30, 2025
@coderabbitai
Copy link
Contributor Author

coderabbitai bot commented Oct 30, 2025

Important

Review skipped

CodeRabbit bot authored PR detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Owner

Choose a reason for hiding this comment

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

No need to add summary and stats here. Just outline how to add testing, how to check coverage, and if there's a roadmap on the testing side

});
beforeEach(() => {
mockRedis = new MockRedis();
// Note: In real tests, you'd use a test Redis instance or redis-mock
Copy link
Owner

Choose a reason for hiding this comment

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

Will this run? We're just throwing some random URL, will Redis know to initialize?

Copy link
Owner

Choose a reason for hiding this comment

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

We should store the mockAdapter somewhere else so as not to define it in every unit test

Copy link
Owner

@EggerMarc EggerMarc left a comment

Choose a reason for hiding this comment

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

Need to add a general mock adapter / cache / ws generator function to share across unit tests.

@EggerMarc EggerMarc merged commit 9b8f267 into optimization Oct 30, 2025
@EggerMarc EggerMarc deleted the coderabbitai/utg/5ea9450 branch October 30, 2025 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant