Skip to content

Rework fixtures to avoid skips#557

Merged
vitenti merged 2 commits into
masterfrom
reduce-skips
Oct 6, 2025
Merged

Rework fixtures to avoid skips#557
vitenti merged 2 commits into
masterfrom
reduce-skips

Conversation

@marcpaterno
Copy link
Copy Markdown
Collaborator

@marcpaterno marcpaterno commented Oct 3, 2025

Description

This PR modifies fixtures in test_ccl_factory.py to reduce the number of skips. This reduces the number of skips due to "CAMB extra parameters are only compatible with PUR_CCL_MODE from > 380 to 0..

Type of change

  • Refactoring

Checklist:

  • I have run bash pre-commit-check and fixed any issues
  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation
  • I have 100% test coverage for my changes (please check this after the CI system has verified the coverage)

@marcpaterno marcpaterno requested a review from vitenti October 3, 2025 00:47
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.0%. Comparing base (ca6245f) to head (4f7447e).
⚠️ Report is 98 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #557   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files          50       50           
  Lines        5115     5115           
  Branches      580      580           
=======================================
  Hits         5115     5115           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vitenti vitenti merged commit 510f9fc into master Oct 6, 2025
11 checks passed
@vitenti vitenti deleted the reduce-skips branch October 6, 2025 23:52
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