Alan-Jowett / CoPilot-For-Consensus / 20155523918
82%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 3
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 73.152% (-3.0%) from 76.171%
20155523918

push

github

web-flow 
feat: Add ValidatingEventSubscriber and integrate schema validation in parsing service (#101)

* feat: add ValidatingEventSubscriber and integrate schema validation in parsing service

- Add ValidatingEventSubscriber wrapper for inbound event validation
- Integrate ValidatingEventSubscriber in parsing service
- Add copilot_schema_validation to parsing service dependencies
- Use ValidatingEventPublisher for outbound events
- Use ValidatingDocumentStore for storage operations
- Provides comprehensive schema validation defense across event lifecycle

* fix: address PR review comments for ValidatingEventSubscriber

- Fix connect() return type to match EventSubscriber interface (returns None, raises on failure)
- Use Tuple from typing module for type hint consistency
- Make subscribe() parameters required (event_type and callback)
- Add comprehensive test suite for ValidatingEventSubscriber (25 test cases)
  - Tests for initialization, validation logic, callback wrapping
  - Tests for strict/non-strict modes and error handling
  - Tests for edge cases (missing schemas, malformed events)
  - Integration tests for full event subscription flow

* fix: correct validate_json mock patch path in tests

- Patch copilot_schema_validation.validate_json instead of copilot_events.validating_subscriber.validate_json
- validate_json is imported dynamically inside _validate_event method, not at module level
- Fixes AttributeError in test suite (8 failing tests)

---------

Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>

188 of 257 relevant lines covered (73.15%)

0.73 hits per line

Jobs
ID Job ID Ran Files Coverage
1 20155523918.1 3
73.15
 GitHub Action Run
Source Files on build 20155523918