Skip to content

Chore: Break up the core integration tests#5432

Merged
izeigerman merged 4 commits intomainfrom
chore-break-up-core-integration-test
Sep 24, 2025
Merged

Chore: Break up the core integration tests#5432
izeigerman merged 4 commits intomainfrom
chore-break-up-core-integration-test

Conversation

@izeigerman
Copy link
Collaborator

No description provided.

@izeigerman izeigerman requested review from a team and Copilot September 23, 2025 19:33
Copy link
Contributor

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 pull request breaks up the core integration tests into separate, more focused modules. The changes aim to improve test organization and maintainability by separating different types of integration tests.

  • Extracts utility functions into a shared utils.py module
  • Separates tests into focused modules: test_run.py, test_restatement.py, test_plan.py, test_multi_repo.py, and test_model_kinds.py
  • Maintains test functionality while improving code organization

Reviewed Changes

Copilot reviewed 15 out of 17 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/core/integration/utils.py New module containing shared utility functions for integration tests
tests/core/integration/test_run.py Tests for run functionality extracted from main integration tests
tests/core/integration/test_restatement.py Tests for restatement functionality extracted from main integration tests
tests/core/integration/test_plan.py Tests for plan functionality extracted from main integration tests
tests/core/integration/test_multi_repo.py Tests for multi-repository functionality extracted from main integration tests
tests/core/integration/test_model_kinds.py Tests for various model kinds functionality extracted from main integration tests

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

@@ -0,0 +1,1939 @@
from __future__ import annotations
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This will likely become the next candidate for breakup due to the vagueness of its scope. I’m looking for feedback on how to improve it

izeigerman and others added 2 commits September 23, 2025 12:52
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@izeigerman izeigerman force-pushed the chore-break-up-core-integration-test branch from 6cb434c to 3046be2 Compare September 23, 2025 19:52
@izeigerman izeigerman merged commit e7e4841 into main Sep 24, 2025
35 of 36 checks passed
@izeigerman izeigerman deleted the chore-break-up-core-integration-test branch September 24, 2025 01:19
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.

3 participants