pyiron / pyiron_base / 5580912521

72%

push

github-actions

Tab completed loading (#951)

* Black line length

* Add tests for loading and inspecting

* Introduce and test helper classes for loading and inspecting

* Don't deprecate the conversion argument

It gets used in iter methods in a reasonable way

* Use just inspect or load if conversion is not a variable

* Format black

* Update pyiron_base/project/generic.py

Co-authored-by: Marvin Poul <poul@mpie.de>

* Break test into subtests

* Directly check getattr, as suggested by @pmrv

* Move loader classes to their own module, per @jan-janssen's suggestion

* Use full import path

* Move tests to their own test module

* Break sub tests into separate tests

* PEP8

* Also work without a database, per @jan-janssen's concern

* Make sure loader values are always synced with project values

The project knows how to stay up-to-date with the state; instead of re-inventing this security, let's just directly use the project attributes.

* Store and restore all the settings

Let's see if this recovers the `project_path` configuration setting that the projectpath tests seem to be relying on

* Format black

* Add logging to try and debug remote ci

* Remove sibling tests

* Add path to debug print

* More logging

* More logging

* Log more about the loader project

* Add filters progressively in logging

* Log the filetable path

* Log the actual folder contents

* Narrow the search

* Just look at length and look before turning off the database too

* Log the actual recursion check results

* Revert changes, the problem is upstream in FileTable still

I don't see trouble on my local machine because I don't specify TOP_LEVEL_DIRS in my config but the CI does.

* Resolve some codacy nits

* Update class descriptions

* Test sub-project loading

* Update tests to account for extra job in TestWithFilledProject

Making some effort to avoid magic numbers

---------

Co-authored-by: pyi... (continued)

81 of 81 new or added lines in 6 files covered. (100.0%)

6911 of 10184 relevant lines covered (67.86%)

0.68 hits per line