Alan-Jowett / CoPilot-For-Consensus / 20218110446

78%

push

github

Enable schema validation in test fixtures across all services (#219)

* Add schema validation to test fixtures

- Fix collection-to-schema mapping in ValidatingDocumentStore to use lowercase names
- Update messages.schema.json to allow null for in_reply_to field
- Add schema validation to parsing, orchestrator, summarization, and chunking test fixtures
- Update test data to include required fields (archive_id, created_at, etc.)
- Add  operator support to test fixtures where needed
- Schema validation catches real bugs in chunking service (None for start_offset/end_offset)

* Fix chunks schema to allow null for optional start_offset and end_offset

The schema defined these as integers, but the chunker implementations
set them to None when they don't calculate character offsets.
Updated to use anyOf pattern to allow either integer or null.

* Embedding: enable schema validation in tests and fix ValidatingDocumentStore update_document to validate merged document\n\n- Wrap InMemoryDocumentStore with ValidatingDocumentStore + FileSchemaProvider\n- Add created_at to embedding test chunks\n- Fix import of FileSchemaProvider from copilot_schema_validation\n- Update ValidatingDocumentStore.update_document to validate merged doc (fetch current, merge, validate)\n- Reinstall adapters to ensure tests use current workspace code

* Ingestion: enable event schema validation in tests and adapt assertions\n\n- Wrap NoopPublisher with ValidatingEventPublisher using FileSchemaProvider\n- Adjust tests to reference underlying publisher for published_events\n- Add attribute delegation in ValidatingEventPublisher to expose base publisher test utilities\n- All ingestion tests passing with validation

* Fix NoopSubscriber to block like RabbitMQ subscriber

NoopSubscriber.start_consuming() now blocks using a threading.Event
until stop_consuming() is called, matching the behavior of RabbitMQ's
start_consuming(). This prevents CI hangs where:

1. Service starts subscriber in non... (continued)

12 of 13 new or added lines in 3 files covered. (92.31%)

513 of 800 relevant lines covered (64.13%)

0.64 hits per line