Skip to content

Conversation

@caioolivv
Copy link
Collaborator

@caioolivv caioolivv commented Dec 10, 2025

Description

Fail tests if backends are not available. Tests should be ran at all times.

Main changes

  • Chnage missing backend erros from pytest.skip to pytest.fail

Checklist

Besides passing all CI checks and coverage is at 100%, make sure you also checked the following items
(check details in CONTRIBUTING).

For developers

  • Notebooks: notebooks related to this PR have been updated and all notebooks can run correctly.
  • Build the documentation: All documentation builds correctly.

For reviewers

  • Notebooks: notebooks related to this PR have been updated and all notebooks can run correctly.
  • Build the documentation: All documentation builds correctly.

For developers (part 2)

After the PR has been approved by two reviewers:

  • Update the code version in clmm/__ini__.py.
  • Keep only relevant points in the squash and merge commit message.
  • If any dependencies have been altered, update environment.yml, pyproject.toml, INSTALL.md, and README.md. A maintainer should also be notified to change the requirements on conda-forge.
  • Update clmm/theory/_ccl_supported_versions.py and clmm/theory/ccl.py if pyccl's version constraints have been altered.

…this is already tested in `test_base` from `test_theory_be.py`
@caioolivv caioolivv self-assigned this Dec 10, 2025
@github-actions github-actions bot changed the title Issue/701/fail_missing_backend_tests Issue/701/fail_missing_backend_tests [version:1.16.11] Dec 10, 2025
@coveralls
Copy link

coveralls commented Dec 10, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling f599ec2 on issue/701/fail_missing_backend_tests
into 0dc0be9 on main.

@caioolivv caioolivv requested a review from m-aguena December 10, 2025 10:40
* fix: Correct typo in pull request template for version update instruction

* fix: Update pull request template to remove old checklist items

* fix: Update NumCosmo version to v0.24 in installation files

* fix: Remove reference to updating CCL version file in documentation

* fix: Remove Travis CI configuration file
Copy link
Member

@m-aguena m-aguena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@caioolivv thanks for repporting this. I guess the issue here is that we don't want to force the user to have all backends installed, but we do want the CI to run with all backends. Do you think there is a way we can achieve this?

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.

Having CLMM tests skip if a backend is not available means some bugs might be left unseen

4 participants