openbraininstitute / neurodamus / 13813033482 / 2 – unit
90%
main: 90%

DEFAULT BRANCH: main
Ran
Files 48
Run time 4s
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: 65.501% (+2.4%) from 63.056%
unit – 13813033482.2

push

github

web-flow 
Unit tests for reporting (#119)

## Context
For unit tests we don't build libsonata-report, therefore we can't
create SONATA reports. Alternatively, we can dump the reporting
variables in ASCII format, as per dission in #98.
This PR adds unit tests to check reporting functionalities, mainly
`Node.enable_reports()` and `report.py`. For compartment reports, a test
dumps the reporting values to ASCII report files and check the values.

## Scope
- Remove the unused legacy parameter `ReportingBufferSize`
(https://github.com/openbraininstitute/neurodamus/issues/146)
- In `Node.enable_reports()`, remove the checks for sonata configuration
parameters which should have been checked by libsonata parser.
- In `tests/utils.py`, new functions for recording compartment reports,
and write/read the ascii report files.

## Testing
New `tests/unit/test_ringcells_reports.py` for testing the variables
reports for NEURON and CoreNEURON. It also takes over some tests from
`tests/integration-e2e/test_reports.py`.
However, the test for creating synapse report stays in
`tests/integration-e2e/test_reports.py` because we can't create it with
the current ring circuit (lack of synapses per cell).

## Review
* [x] PR description is complete
* [x] Coding style (imports, function length, New functions, classes or
files) are good
* [x] Unit/Scientific test added
* [ ] Updated Readme, in-code, developer documentation

---------

Co-authored-by: Alessandro Cattabiani <cattabiani@users.noreply.github.com>

5901 of 9009 relevant lines covered (65.5%)

0.66 hits per line

Source Files on job unit - 13813033482.2