[FIX] Exclude sessions missing a queried property from matches

alyssadai commented 1 month ago

Closes #324

Changes proposed in this pull request:

Use the BOUND function in subquery for matching phenotypic sessions to exclude any results where a variable being filtered for does not have a value ("unbound" results - these are included by default, leading to inaccurate counts of matches)
- e.g., because all phenotypic attributes are optional, when we query for healthy controls, the default matching behaviour includes any sessions that do not have the isSubjectGroup edge - which would be any session that has diagnosis info, since these two properties are mutually exclusive
Make having a phenotypic session non-optional in query (since subjects should not end up in the graph if they don't have any phenotypic info)
Minor refactoring of query template for efficiency

This PR also includes a small fix to the context of an auth behaviour test that assumes authentication is enabled (this test was failing for me locally b/c I had auth turned off in development).

Checklist

This section is for the PR reviewer

[x] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see our Contributing Guidelines for more info)
[x] PR has a label for the release changelog or skip-release (to be applied by maintainers only)
[x] PR links to GitHub issue with mention Closes #XXXX
[x] Tests pass
[x] Checks pass