Skip to content

Comments

Optimize tests#136

Open
aknierim wants to merge 119 commits intomainfrom
optimize_tests
Open

Optimize tests#136
aknierim wants to merge 119 commits intomainfrom
optimize_tests

Conversation

@aknierim
Copy link
Member

@aknierim aknierim commented Jan 27, 2026

Updates the entire test framework with unit tests that run faster than the old tests. Also, the new tests cover more edge cases than the previous tests.

Changes

  • Removed tomli from dependencies and test requirements in pyproject.toml, added pytest-mock as a new test dependency
  • Updated type hints across the codebase
  • Added dataset_type as an to FITSWriter for consistency with other writer classes
  • Updated linting configuration to exclude only examples from ruff checks, not tests
  • Updated code coverage exclusion patterns
  • Removed unused submodule pyvisgen.fits.data
  • Updated __getitem__ method of ValidBaselineSubset dataclass to return ValidBaselinesSubset object, similar to Baselines dataclass
  • pyvisgen.simulation.array.Array now has updated property names relative_pos and antenna_pairs. Removed calc_relative_pos method that would just get the relative_pos property

@aknierim aknierim marked this pull request as ready for review February 12, 2026 10:46
@aknierim aknierim requested review from ChAr-De, Kevin2 and tgross03 and removed request for Kevin2 February 12, 2026 10:46
@aknierim aknierim changed the title [wip] Optimize tests Optimize tests Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant