Skip to content

feat: add integration tests#18

Merged
gauthierdmn merged 2 commits intomainfrom
feat/e2e-testing
Mar 4, 2026
Merged

feat: add integration tests#18
gauthierdmn merged 2 commits intomainfrom
feat/e2e-testing

Conversation

@gauthierdmn
Copy link
Owner

No description provided.

@gauthierdmn gauthierdmn self-assigned this Mar 3, 2026
Copy link

@nominalbot nominalbot bot left a comment

Choose a reason for hiding this comment

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

The PR adds integration testing infrastructure with CI workflow support. However, there is a critical naming mismatch: all test files are placed under tests/integration/ but every import path and both CI workflow commands reference tests/end_to_end/, which does not exist. This means every integration test will fail at import time, and the CI jobs will find zero tests. Additionally, no test in the repo carries the agent_in_the_loop marker, so the new end-to-end.yml workflow will also collect nothing. The end-to-end.yml workflow is also missing the python-version specifier that other jobs include.

@gauthierdmn gauthierdmn force-pushed the feat/e2e-testing branch 7 times, most recently from 2cd18eb to d64a2d7 Compare March 4, 2026 01:35
@gauthierdmn gauthierdmn merged commit 11fcc26 into main Mar 4, 2026
10 checks passed
@gauthierdmn gauthierdmn deleted the feat/e2e-testing branch March 4, 2026 02:56
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