Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Fix misc. live tests#795

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
joseharriaga merged 3 commits intomainfromjoseharriaga/FixLiveTests
Oct 30, 2025
Merged

Fix misc. live tests#795

joseharriaga merged 3 commits intomainfromjoseharriaga/FixLiveTests
Oct 30, 2025

Conversation

@joseharriaga
Copy link
Collaborator

  • Extend delays for VectorStore tests to mitigate flakiness
  • Upgrade Moderation tests to the new omni-moderation-latest model to avoid failures when using the older text-moderation-stable model
  • Move image generation tool in Responses to ResponsesToolTests.cs
  • Add a delay to file search tool test in Responses to mitigate flakiness
  • Re-record as needed

Copy link

CopilotAI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the moderations tests to use the newomni-moderation-latest model instead of the deprecatedtext-moderation-stable model. The changes include:

  • Updating the default model for moderations test scenarios
  • Adjusting test assertions to match the new model's score thresholds
  • Updating session recordings to reflect the new API responses
  • Reorganizing test methods between test files
  • Adding necessary delays for consistency in vector store tests

Reviewed Changes

Copilot reviewed 11 out of 21 changed files in this pull request and generated 5 comments.

Show a summary per file
FileDescription
tests/Utility/TestHelpers.csUpdates default moderation model from "text-moderation-stable" to "omni-moderation-latest"
tests/Moderations/ModerationsTests.csAdjusts violence score thresholds from 0.5 to 0.2 and updates model name assertion
tests/SessionRecords/ModerationsTests/*.jsonUpdates recorded API responses for the new omni-moderation model with additional fields
tests/SessionRecords/VectorStoresTests/*.jsonUpdates session recordings with new test data and reduced file counts
tests/Responses/ResponsesToolTests.csAdds System.Threading using statement and moves image generation tests from ResponsesTests
tests/Responses/ResponsesTests.csRemoves image generation tests (moved to ResponsesToolTests)
tests/Assistants/Assistants.VectorStoresTests.csIncreases delays from 1-2 seconds to 10 seconds for vector store operations

💡Add Copilot custom instructions for smarter, more guided reviews.Learn how to get started.

joseharriagaand others added2 commitsOctober 30, 2025 16:48
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@joseharriagajoseharriaga merged commitb1ad4b1 intomainOct 30, 2025
1 check passed
@joseharriagajoseharriaga deleted the joseharriaga/FixLiveTests branchOctober 30, 2025 23:51
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

Copilot code reviewCopilotCopilot left review comments

@ShivangiRejaShivangiRejaShivangiReja approved these changes

@trrwilsontrrwilsonAwaiting requested review from trrwilsontrrwilson is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@joseharriaga@ShivangiReja
[8]ページ先頭
©2009-2025 Movatter.jp