github: add workflow for unit tests

[aconchillo]

No description provided.

[aconchillo]

This also adds a requirements-full.txt.

[Moishe]

The existing tests (under src/dailyai/tests/test_*) use unittest -- will this workflow run them?

[aconchillo]

yes, pytest just automatically looks for tests. You can see how it runs them in the CI step below. But I'll be more explicit though.

[Moishe]

ah yeah I just tried it, but it seems to miss the doctests. (7 tests with pytest vs 11 with unittest)

[aconchillo]

Updated.

[Moishe]

is there a way to extract the requirements from the toml file for testing? Otherwise we'll need to keep it in sync with requirements-full.txt which seems like it might be error-prone.

[aconchillo]

Oh, I see. Let me see what I can do...

[aconchillo]

OK, I have removed requirements-full.txt and we are just building the project now which forces to install the dependencies in the toml file.

[Moishe]

There's a load_tests function in test_aggregators.py that loads the aggregators module and then runs a DocTestSuite against that module. It doesn't seem to be picked up by pytest. (which is why pytest is reporting fewer tests than unittest). Not sure if there's a way to address that directly with pytest or if we should run unittest.

[aconchillo]

Oh, I see. How do you run that?

[aconchillo]

For the record, the way to run it with unitest is:

python -m unittest -v src/dailyai/tests/test_*

However, pytest doesn't like load_tests and since we are only loading the doctests, we can just do:

pytest --doctest-modules --ignore-glob="*to_be_updated*" src/dailyai

[aconchillo]

See pytest-dev/pytest#992

[Moishe]

Nice, thanks for figuring that out!