Skip to content

Fix CI coverage.xml failure and increase test coverage#28

Merged
Pablo1990 merged 4 commits intomainfrom
copilot/increase-test-coverage
Apr 9, 2026
Merged

Fix CI coverage.xml failure and increase test coverage#28
Pablo1990 merged 4 commits intomainfrom
copilot/increase-test-coverage

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026

  • Fix CI workflow: add pip install -e . step, remove --doctest-modules, fix --cov scope
  • Fix ismember_rows in utils.py: replace deprecated np.in1d with np.isin, flatten bool/index arrays to fix edge_valence IndexError
  • Fix coverage config in pyproject.toml: remove parameters/* from omit, add analysis scripts
  • Fix check_if_cells_are_the_same in test_geo.py to handle empty Geo objects
  • Fix broken tests in test_vertexModel.py: remove invalid set_option kwarg from Set/VertexModelVoronoiFromTimeImage calls
  • Add Tests/test_set.py: ~25 unit tests covering Set defaults, all presets, derived parameters, copy, wound_default
  • Add comprehensive tests for Kg classes (kgVolume, kgViscosity, kgSubstrate, kgTriEnergyBarrier, kgTriAREnergyBarrier, kgSurfaceCellBasedAdhesion) using wing_disc_150.mat
  • Add comprehensive tests for cell.py, face.py, tris.py methods
  • Add comprehensive tests for geo.py free functions and Geo methods
  • Add comprehensive tests for utils.py pure functions
  • Verify total coverage exceeds 60%

Copilot AI linked an issue Apr 9, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Increase test coverage to over 60 percent Fix CI coverage.xml failure and increase test coverage Apr 9, 2026
Copilot stopped work on behalf of Pablo1990 due to an error April 9, 2026 09:19
Copilot AI requested a review from Pablo1990 April 9, 2026 10:06
@Pablo1990 Pablo1990 marked this pull request as ready for review April 9, 2026 11:17
@Pablo1990 Pablo1990 merged commit 833bf7a into main Apr 9, 2026
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.

Increase test coverage

2 participants