Add pytest to Github actions

[NielsRogge]

This PR adds pytest to the Github actions, to make sure tests are run each time a PR is made.

[NielsRogge]

@RobbeSneyders the test pipeline failed when not adding dataclasses_json:

ModuleNotFoundError: No module named 'dataclasses_json'

Even though there's a step before that installs all dependencies (which includes dataclasses_json). Are we sure that this step:

pip install poetry==1.4.0
poetry install

installs the dependencies of the package properly?

[RobbeSneyders]

Since Poetry is not running in a virtual environment, it create and manages its own. This is cleaner for CI/CD, as Poetry's owns dependencies are not installed in the virtual environment. You can run a command in the virtual environment by running poetry run <command>.

See the docs for more info.

[NielsRogge]

@RobbeSneyders thanks, have added back the "io" module to make all tests pass

[PhilippeMoussalli]

@RobbeSneyders thanks, have added back the "io" module to make all tests pass

Can you remove io and test_io altogether, those functionalities are not used anymore

[PhilippeMoussalli]

Thanks for adding this. Just remove all the io stuff and it's ready to merge

[NielsRogge]

@PhilippeMoussalli that means I can also remove the copy_folder and copy_file methods which rely on it?

[PhilippeMoussalli]

@PhilippeMoussalli that means I can also remove the copy_folder and copy_file methods which rely on it?

yes they can be removed

[NielsRogge]

I'll merge this and remove the io files in a separate PR, as they are still used in the soon-to-be-legacy storage helpers