Context
Tests are disabled in CI with if: false in .github/workflows/ci.yml. Contributors can't verify their changes pass tests before merging. This is a trust killer for open source adoption — new contributors won't submit PRs if they can't validate their work.
Current State
The CI workflow exists but the test job is gated with if: false, effectively skipping all tests on every push/PR.
Proposed Changes
- Remove the
if: false gate from the test job
- Ensure test database setup works in CI (PostgreSQL service container)
- Add Qdrant service container for vector search tests (or mock it)
- Add Redis service container
- Fix any broken tests (known issue:
intuno_sdk import errors — either mock or add as test dependency)
- Consider adding a test coverage threshold
Priority
High — this should be one of the first things fixed for open source credibility.
Files
Context
Tests are disabled in CI with
if: falsein.github/workflows/ci.yml. Contributors can't verify their changes pass tests before merging. This is a trust killer for open source adoption — new contributors won't submit PRs if they can't validate their work.Current State
The CI workflow exists but the test job is gated with
if: false, effectively skipping all tests on every push/PR.Proposed Changes
if: falsegate from the test jobintuno_sdkimport errors — either mock or add as test dependency)Priority
High — this should be one of the first things fixed for open source credibility.
Files
.github/workflows/ci.yml