Skip to content

Conversation

@terragon-labs
Copy link

@terragon-labs terragon-labs bot commented Jul 11, 2025

Summary

  • Completely removes the test suite from the repository
  • Deletes all test files under src/croniter/tests/
  • Removes test dependencies from requirements/test.txt and requirements/tox.txt
  • Cleans up setup.cfg by removing coverage configuration
  • Updates tox.ini to only run linting environment

Changes

Test Files

  • Deleted all test files including:
    • test_croniter.py
    • test_croniter_dst_repetition.py
    • test_croniter_hash.py
    • test_croniter_random.py
    • test_croniter_range.py
    • test_croniter_speed.py

Dependencies

  • Removed requirements/test.txt which contained pytest, coverage, mock, flake8, setuptools
  • Removed tox-current-env from requirements/tox.txt
  • Updated tox.ini to only run lint environment, removing test and coverage environments

Configuration

  • Removed coverage run and report sections from setup.cfg

Rationale

  • This PR removes all testing infrastructure and test code, possibly to prepare for a different testing strategy or to clean up the repository.

Test plan

  • No tests remain in the repository after this change
  • Linting environment remains active and can be run via tox
  • Manual verification that no test files or test dependencies remain

🌿 Generated by Terry


ℹ️ Tag @terragon-labs to ask questions and address PR feedback

📎 Task: https://www.terragonlabs.com/task/ed8d2c2e-cfd4-4753-80bf-a810d4ddfc6b

- Removed coverage configuration from setup.cfg
- Updated tox.ini to only run lint environment
- Removed test and coverage environments from tox.ini
- Cleaned up requirements/tox.txt by removing tox-current-env dependency

Co-authored-by: terragon-labs[bot] <terragon-labs[bot]@users.noreply.github.com>
@terragon-labs terragon-labs bot requested a review from k6l3 July 11, 2025 02:34
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