Skip to content

TST: pytest-dev gives 3 errors in devinfra jobs from suspicious skips #20830

Open

Description

@pllim
  • astropy/cosmology/_src/tests/flrw/test_flrw.py::TestFLRW::test_luminosity_distance_pandas - pytest.PytestWarning: The reason 'w(z) is abstract' of pytest.mark.skip looks like a condition. Did you mean pytest.mark.skipif?
  • astropy/cosmology/_src/tests/io/test_mapping.py::TestToFromMapping::test_fromformat_subclass_partial_info_mapping - pytest.PytestWarning: The reason 'N/A' of pytest.mark.skip looks like a condition. Did you mean pytest.mark.skipif?
  • astropy/modeling/tests/test_quantities_fitting.py::test_compound_fitting_with_units[LevMarLSQFitter] - pytest.PytestWarning: The reason 'Flaky and ill-conditioned' of pytest.mark.skip looks like a condition. Did you mean pytest.mark.skipif?

I see #20813 is addressing the 2 cosmology ones, but are you also planning on addressing the modeling test, @neutrinoceros ?

Activity

  1. neutrinoceros commented on Nov 11, 2026

    @neutrinoceros
    Contributor

    I have a PR upstream that will adress the most common case and create an escape path for remaining false positives.
    There's nothing we can do about it from our side until that's resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions