Test: Create test for get_instance for White Whole

[nanoman657]

Test the get_instance function for the White_whole dataset.

White_whole has a get_instance function like other complete connectome datasets. However, it has been untested, which creates risk for further development of the Connectome Toolbox. This PR tests the function, while also including abstracted caching logic which could also be extended to other data readers.

Notes:

• This functionality was tested using unittest, and should be pytest compatible. I can convert to pytest if needed.
• The imports have been rearranged to conform with PEP 8 import guidelines

[nanoman657]

Hi @pgleeson , at your earliest convenience, would you let me know if I'm on the right track with this PR? If I am, I'll clean it up and make additional tests.

[pgleeson]

Will look it over tomorrow @nanoman657, thanks.

[pgleeson]

Thanks again for this. The reason that the WhiteDataReader didn't have a get_instnce method was because it was really a helperclass for White_whole, White_A etc. which do use specific spreadsheets and so are full connectome dataset. Test classes can maybe be added for these "real" connectome datasets

[pgleeson]

By the way, do base your PRs on the latets main in the https://github.com/openworm/ConnectomeToolbox/ repo.

[nanoman657]

Thanks @pgleeson for your feedback. To address your comment, I have adjusted the commits to address the White_whole.py file instead. I have also rebased on the latest Connectome Toolbox repository. Please let me know if the edits made are satisfactory.

[pgleeson]

Thanks, testing this out, unfortunately the GHA tests take an hour to run, so difficult not to get distracted by other things...

[nanoman657]

@pgleeson No worries! I see that some of the builds have failed, so I'll work to correct those.

[pgleeson]

Thanks @nanoman657, have tested this and merged it in. Note this change I made:

ConnectomeToolbox/cect/tests/test_white_whole.py

Lines 43 to 46 in 8e3875b

	if not Path(white_cache).is_file():
	self.cache_files_to_cleanup.append(white_cache)
	else:
	print(f"Cache file {white_cache} already exists - not cleaning it up")

i.e. if the test is run, and the cache file to generate already exists, don't delete it. In this case the cache file has been added to the repo.

[pgleeson]

Also, this (0967715) puts a limit on the version of hiveplotlib to use, which prevents the test from failing.