Skip to content

Conversation

NoodlesNZ
Copy link

This PR modernizes the testing framework replacing the unittest framework with pytest. pytest provides clearer assertions, powerful fixtures, and an extensive plugin ecosystem, which will make tests easier to write, maintain, and extend going forward.

@NoodlesNZ NoodlesNZ requested a review from izar as a code owner October 4, 2025 03:16
@Copilot Copilot AI review requested due to automatic review settings October 4, 2025 03:16
@NoodlesNZ NoodlesNZ changed the title Convert unitests to Pytest Convert unit tests to Pytest Oct 4, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Modernize tests by migrating from unittest to pytest for clearer assertions and better fixtures, and update packaging to poetry-core.

  • Replace unittest.TestCase classes and assertions with plain pytest-style tests and assertions
  • Update exception assertions to pytest.raises and remove unittest-only helpers
  • Update pyproject.toml to use poetry-core and add pytest dependency

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

File Description
tests/test_pytmfunc.py Convert unittest assertions and subTest blocks to pytest style; minor refactors in test bodies.
tests/test_private_func.py Convert unittest assertions to pytest style; adjust exception assertions; minor cleanup.
pyproject.toml Switch build backend to poetry-core; reorder Python constraints; add pytest to dependencies.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

1 participant