neurobagel / federation-api / 7037895045

97%

push

github

[REF] Refactor tests (#37)

* add new query param that can receive a list of nodes

* rename query param

* add validation for node url list query param

* update docstring

* add test for unrecognized node URLs

* rename expected env file

* add startup event for creating federation node index

* rename variable storing user-defined local nodes

* raise warning and use local nodes when public nodes fetching fails

* switch to using federation node index in endpoints

* move dynamic default setting + validation of node url param out of query model

* update function docstring

* add and implement function for adding trailing slash to urls

* add test for check_nodes_are_recognized()

* update variable name and docstring typo in test

* add TODO for self for future

* refactor node url param validation into separate function and test

* remove unneeded test module

* add tests of federation node index creation and nodes endpoint

* update function return type hint

* update README

* add test case for whitespace in local nodes parsing

* add test case for query node URL list with duplicates

* update README

* update regex for local nodes to be whitespace-insensitive

* Update docstrings

Co-authored-by: Sebastian Urchs <surchs@users.noreply.github.com>

* add template fed.env file

* test utility to add trailing slash

* simplify some tests of utility functions

* simplify env variable mocking in tests

* Update docstring

Co-authored-by: Sebastian Urchs <surchs@users.noreply.github.com>

---------

Co-authored-by: Sebastian Urchs <surchs@users.noreply.github.com>

28 of 28 new or added lines in 2 files covered. (100.0%)

241 of 246 relevant lines covered (97.97%)

1.39 hits per line