Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unit tests should all use standardized fixtures #503

Open
15 tasks
alexkjames opened this issue Jan 12, 2024 · 1 comment
Open
15 tasks

unit tests should all use standardized fixtures #503

alexkjames opened this issue Jan 12, 2024 · 1 comment
Assignees
Labels
copilot Issue that can potentially be handled by GitHub copilot's "workspace" feature enhancement

Comments

@alexkjames
Copy link
Contributor

Currently our unit tests use a mixture of bespoke testing objects and well defined pytest fixtures. We should move to use pytest fixtures across the board (as much as is reasonable anyway)

  • coherence
  • correns
  • ensembleseries
  • geoseries
  • multiplegeoseries
  • multiplepsd
  • multipleseries
  • multivariatedecomp
  • psd
  • resolution
  • scalogram
  • series
  • ssares
  • tsbase
  • tsutils
  • [ ]
@alexkjames alexkjames self-assigned this Jan 12, 2024
@CommonClimate
Copy link
Collaborator

Yes! Let me add that most of our tests use series that are on the long side. We should endeavor to use as few data points as possible to make the tests faster, thereby accelerating our unit test cycles.

@CommonClimate CommonClimate added the copilot Issue that can potentially be handled by GitHub copilot's "workspace" feature label Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
copilot Issue that can potentially be handled by GitHub copilot's "workspace" feature enhancement
Projects
Development

No branches or pull requests

2 participants