openbraininstitute / libsonata / 27198033396

93%

master: 93%

Build Type

Pull #49

github

Committed by cattabiani

Commit Message

Reject empty variable_name in non-LFP reports

parseMandatory only checks key existence, not that the value is
non-empty. An empty variable_name would silently pass the regex
validation. Now it throws a SonataError.

Also moves the variable_name validation (empty check + regex) into
the else branch so it only runs for non-LFP reports, which never
have a variable_name.

Pull Request Pull Request #49: Move electrodes_file to per-report LFP blocks

Coverage Stats

16 of 23 new or added lines in 1 file covered. (69.57%)

14 existing lines in 1 file now uncovered.

2523 of 2715 relevant lines covered (92.93%)

50.81 hits per line