Skip to content

Conversation

blva
Copy link
Collaborator

@blva blva commented Aug 27, 2025

Proposed changes

  • this fixes the issue where we were validating the size of the response. now we just check if the expected org has been found

Checklist

@blva blva marked this pull request as ready for review August 27, 2025 10:41
@Copilot Copilot AI review requested due to automatic review settings August 27, 2025 10:41
@blva blva requested a review from a team as a code owner August 27, 2025 10:41
Copy link
Contributor

@Copilot Copilot AI 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 fixes an Atlas integration test by removing a hard-coded length validation that was causing test failures. The test now focuses on validating the presence of expected organizations rather than enforcing a specific response size.

  • Removes hard-coded length assertion that was brittle and causing test failures
  • Maintains validation of the actual organization data content

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@kmruiz kmruiz left a comment

Choose a reason for hiding this comment

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

Added a suggestion, but it's not really blocking.

@blva blva enabled auto-merge (squash) August 27, 2025 11:04
@blva blva changed the title fix: update Atlas integration tests fix: update atlas integration tests -- MCP-98 Aug 27, 2025
@blva blva merged commit cead349 into main Aug 27, 2025
16 of 18 checks passed
@blva blva deleted the MCP-98 branch August 27, 2025 11:18
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.

2 participants