edanalytics / lightbeam

CLI tool for validating and transmitting payloads from JSONL files into an Ed-Fi API.
Apache License 2.0
10 stars 1 forks source link

fixes a bug with validating local descriptor values within nested payloads #29

Closed tomreitz closed 5 months ago

tomreitz commented 6 months ago

I found a bug when trying to validate an assessment record where the assessment.performanceLevels[x].performanceLevelDescriptor referred to a value that definitely existed locally (in a performanceLevelDescriptors.jsonl file) but lightbeam validate failed anyway. Turns out there was a bug in the recursive calls within has_invalid_descriptor_values() of validate.py; this PR fixes it.