Skip to content

test: cover transcript parser edge cases#282

Open
haosenwang1018 wants to merge 1 commit intozilliztech:mainfrom
haosenwang1018:test/transcript-parser-edge-cases
Open

test: cover transcript parser edge cases#282
haosenwang1018 wants to merge 1 commit intozilliztech:mainfrom
haosenwang1018:test/transcript-parser-edge-cases

Conversation

@haosenwang1018
Copy link
Copy Markdown
Contributor

Summary

  • add coverage for find_turn_context() matching when the lookup string is longer than the stored turn prefix
  • add coverage for parse_transcript() safely ignoring assistant entries whose message.content is not a list
  • keep the patch scoped to transcript parser helper branches under issue test: improve test coverage across the project #114

Testing

  • uv run python -m pytest tests/test_transcript.py -q
  • uv run ruff check tests/test_transcript.py
  • uv run ruff format --check tests/test_transcript.py

Refs #114

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