Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update test dependencies and format code with Black #59

Merged
merged 3 commits into from
Jan 15, 2025

Conversation

Samk13
Copy link
Member

@Samk13 Samk13 commented Jan 15, 2025

❤️ Thank you for your contribution!

Description

  • Fix error:
./run-tests.sh 
lists of files in version control and sdist match
====================================================================== test session starts =======================================================================
platform linux -- Python 3.12.8, pytest-8.3.4, pluggy-1.5.0
rootdir: /home/samk13/INVENIO/issues/invenio-config
configfile: setup.cfg
testpaths: tests, invenio_config
plugins: invenio-3.0.0, github-actions-annotate-failures-0.2.0, flask-1.3.0, isort-4.0.0, cov-6.0.0, pycodestyle-2.4.1, pydocstyle-2.4.0, black-0.3.9
collected 0 items / 1 error                                                                                                                                      

============================================================================= ERRORS =============================================================================
_______________________________________________________________________ ERROR collecting . _______________________________________________________________________
../../../.pyenv/versions/invenio-config/lib/python3.12/site-packages/_pytest/nodes.py:110: in _create
    return super().__call__(*k, **kw)  # type: ignore[no-any-return,misc]
E   TypeError: Can't instantiate abstract class BlackItem without an implementation for abstract method 'collect'

During handling of the above exception, another exception occurred:
../../../.pyenv/versions/invenio-config/lib/python3.12/site-packages/pluggy/_hooks.py:513: in __call__
    return self._hookexec(self.name, self._hookimpls.copy(), kwargs, firstresult)
../../../.pyenv/versions/invenio-config/lib/python3.12/site-packages/pluggy/_manager.py:120: in _hookexec
    return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
../../../.pyenv/versions/invenio-config/lib/python3.12/site-packages/pytest_black.py:27: in pytest_collect_file
    return BlackItem.from_parent(parent, fspath=path)
../../../.pyenv/versions/invenio-config/lib/python3.12/site-packages/_pytest/nodes.py:621: in from_parent
    return super().from_parent(parent=parent, fspath=fspath, path=path, **kw)
../../../.pyenv/versions/invenio-config/lib/python3.12/site-packages/_pytest/nodes.py:233: in from_parent
    return cls._create(parent=parent, **kw)
../../../.pyenv/versions/invenio-config/lib/python3.12/site-packages/_pytest/nodes.py:125: in _create
    return super().__call__(*k, **known_kw)  # type: ignore[no-any-return,misc]
E   TypeError: Can't instantiate abstract class BlackItem without an implementation for abstract method 'collect'
==================================================================== short test summary info =====================================================================
ERROR . - TypeError: Can't instantiate abstract class BlackItem without an implementation for abstract method 'collect'
======================================================================== 1 error in 0.43s ========================================================================
ERROR: found no collectors for /home/samk13/INVENIO/issues/invenio-config/tests

ERROR: found no collectors for /home/samk13/INVENIO/issues/invenio-config/invenio_config

  • CI: use shared workflow for Python tests

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Frontend

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.

@Samk13 Samk13 force-pushed the update-pytest-versions branch from 3628796 to 22e40a4 Compare January 15, 2025 10:50
@slint slint merged commit b30ed75 into inveniosoftware:master Jan 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants