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

Refactor Ops tests to use pytest instead of unittest #1177

Open
benhoyt opened this issue Apr 8, 2024 · 0 comments
Open

Refactor Ops tests to use pytest instead of unittest #1177

benhoyt opened this issue Apr 8, 2024 · 0 comments
Assignees

Comments

@benhoyt
Copy link
Collaborator

benhoyt commented Apr 8, 2024

We've been moving to pytest for Harness and other examples in our docstrings and docs. However, the Ops tests themselves are still all written in unittest with ugly self.assertEqual(...) calls and the like. Let's go all in on pytest here too.

This is going to be a huge diff -- maybe we can do 1 file per PR to split it up?

Let's look to see if there are existing tools that do this. If not, we may want to consider writing our own AST-based hack that can do most of it.

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

No branches or pull requests

2 participants