Skip to content

chore: consolidate pytest config and replace parameterized with native parametrize - #729

Merged
Aidan Holland (thehappydinoa) merged 1 commit into
mainfrom
chore/pytest-consolidate-and-drop-parameterized
Aug 28, 2026
Merged

chore: consolidate pytest config and replace parameterized with native parametrize#729
Aidan Holland (thehappydinoa) merged 1 commit into
mainfrom
chore/pytest-consolidate-and-drop-parameterized

Conversation

@thehappydinoa

@thehappydinoa Aidan Holland (thehappydinoa) commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

  • Deletes pytest.ini and moves all settings into [tool.pytest.ini_options] in pyproject.toml
  • Removes the parameterized dependency entirely
  • Converts all test classes from unittest.TestCase to plain pytest classes
  • Rewrites CensysTestCase base to use an autouse fixture instead of setUp/addCleanup
  • Replaces all @parameterized.expand@pytest.mark.parametrize and @parameterized_class → autouse fixtures with params=

Test plan

  • 459 tests pass, 100% coverage
  • uv run ruff check . && uv run ruff format --check . clean
  • uv run mypy censys clean
  • Pre-commit hooks green

🤖 Generated with Claude Code

…t.mark.parametrize

- Move pytest.ini config into [tool.pytest.ini_options] in pyproject.toml and delete pytest.ini
- Replace parameterized library with native @pytest.mark.parametrize across all test files
- Convert all CensysTestCase subclasses from unittest.TestCase to plain pytest classes
- Rewrite CensysTestCase base using autouse fixture for mocker/responses setup
- Drop parameterized from dev dependencies and uv.lock

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@thehappydinoa
Aidan Holland (thehappydinoa) merged commit 1d0ccb0 into main Aug 28, 2026
12 checks passed
@thehappydinoa
Aidan Holland (thehappydinoa) deleted the chore/pytest-consolidate-and-drop-parameterized branch August 28, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant