Skip to content

Improve test coverage for the summarizer module #106

Description

@sepehr-safari

Good First Issue

The summarizer module (packages/toolkit/src/core/summarizer.ts) has only 5 tests. Improve coverage by adding edge case tests.

What to add

  • Test with empty sessions array
  • Test with session having null timestamps
  • Test with session having very large duration
  • Test with multiple sessions (different transaction IDs)
  • Test with session having zero events
  • Test summarizeSession() directly with various inputs

Files to modify

  • packages/toolkit/src/core/summarizer.test.ts

How to claim

Comment "I'd like to work on this" and it will be assigned to you.

Please hold one open claim at a time. Once the pull request for it is merged,
say which issue you want next and it will be assigned. That keeps issues from
sitting claimed while another is still in review, so other newcomers can see
what is genuinely free.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions