Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 17, 2026

Summary

  • What: Increase token count in test mock responses from 5 (minimum threshold) to 8
  • Why: Avoid brittle tests that rely on exact minimum validation thresholds

Changes

  • Updated three test mock responses in test_orchestrate_content.py:
    • test_skip_humanized_modules (line 93)
    • test_modules_dict_with_modules_key (line 176)
    • test_modules_plain_array (line 198)
  • Changed from 5-token responses to 8-token responses by appending "with enough tokens" or "with sufficient length"

Before/After

# Before - exactly at 5 token minimum
return {"text": "Humanized summary for array format", "ttft": 0.002}

# After - comfortably above threshold
return {"text": "Humanized summary for array format with sufficient length", "ttft": 0.002}

Testing

  • All 9 tests in test_orchestrate_content.py pass
  • Code review: no issues
  • Security scan: no vulnerabilities

Checklist

  • Mock responses updated to 8 tokens
  • All tests passing
  • Code review completed
  • Security check completed

Reviewers


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback on marking flagged modules in content validation Fix test mock responses with borderline token counts Jan 17, 2026
Copilot AI requested a review from 73junito January 17, 2026 21:55
Copy link
Owner

@73junito 73junito left a comment

Choose a reason for hiding this comment

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

viewed

@73junito 73junito marked this pull request as ready for review January 18, 2026 17:27
Copilot AI review requested due to automatic review settings January 18, 2026 17:27
@73junito 73junito merged commit f6af38b into chore/humanize-modules Jan 18, 2026
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants