Skip to content

Conversation

@SurbhiJainUSC
Copy link
Collaborator

@SurbhiJainUSC SurbhiJainUSC commented Jan 14, 2026

Description

  • Move all integration tests to tests/integration
  • Move all unit tests to tests/unit
  • Move all smoke tests to tests/integration/smoke

Tests

CI tests

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

@codecov
Copy link

codecov bot commented Jan 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@SurbhiJainUSC SurbhiJainUSC force-pushed the integration_tests_restructure branch from 2111c89 to 9a05ddf Compare January 14, 2026 21:02
@SurbhiJainUSC SurbhiJainUSC changed the title Move all integration tests to tests/integration Move integration tests to tests/integration and unit tests to tests/unit Jan 14, 2026
@SurbhiJainUSC SurbhiJainUSC force-pushed the integration_tests_restructure branch 15 times, most recently from f2873a9 to d5e2aa9 Compare January 16, 2026 01:09
@SurbhiJainUSC SurbhiJainUSC force-pushed the integration_tests_restructure branch from d5e2aa9 to 0da3c0f Compare January 16, 2026 01:12
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