neurobagel / api

https://api.neurobagel.org/
MIT License
4 stars 3 forks source link

[FIX] Ensure pipeline variables are returned from the graph in an aggregate query #349

Closed alyssadai closed 1 month ago

alyssadai commented 1 month ago

Changes proposed in this pull request:

Note: In this case, some tests started failing after the graph response handling logic was updated, since the mocked aggregate response from the graph used in those tests no longer matched the expected format (good). However, we are not yet to write a test that would fail under the original bug conditions (bad), since this would require a graph with data to test against. See https://github.com/neurobagel/api/issues/283

Checklist

This section is for the PR reviewer

For new features:

For bug fixes:

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 11266360515

Details


Totals Coverage Status
Change from base Build 11261813541: 0.1%
Covered Lines: 744
Relevant Lines: 779

💛 - Coveralls
neurobagel-bot[bot] commented 1 month ago

:rocket: PR was released in v0.4.0 :rocket: