Skip to content

Add CI/test config, require Python 3.12, and update tests to package imports - #18

Merged
IliTheButterfly merged 2 commits into
devfrom
codex/fix-pytest-and-add-testing-documentation-6df34o
Apr 21, 2026
Merged

Add CI/test config, require Python 3.12, and update tests to package imports#18
IliTheButterfly merged 2 commits into
devfrom
codex/fix-pytest-and-add-testing-documentation-6df34o

Conversation

@IliTheButterfly

Copy link
Copy Markdown
Owner

Motivation

  • Enable automated test runs on push and pull requests and standardize the local/CI test environment.
  • Move the project to a minimum of Python 3.12 and add dev test dependencies so tests run reproducibly.
  • Align editor and test configuration with the src/ layout and package imports so tests import eb_macro_gen directly.

Description

  • Added a GitHub Actions workflow at .github/workflows/tests.yml to run pytest on ubuntu-latest with Python 3.12.
  • Updated pyproject.toml to require Python >=3.12, added a [project.optional-dependencies] dev group with pytest>=8, and added [tool.pytest.ini_options] with pythonpath and testpaths.
  • Adjusted test modules in tests/ (for example test_objects.py, test_reverse_operators.py, and test_syntax_regressions.py) to import the package as eb_macro_gen instead of using src.-prefixed imports.
  • Updated .vscode/settings.json to enable pytest, set python.testing.pytestArgs to run tests, and add src/ to python.analysis.extraPaths.
  • Extended ReadMe.md to document the Python 3.12 requirement and provide instructions for running tests and configuring VS Code.

Testing

  • Ran pytest (via pytest -q) against the test suite with the updated configuration.
  • All tests completed successfully.

Codex Task

@IliTheButterfly
IliTheButterfly merged commit 530838d into dev Apr 21, 2026
6 checks passed
@IliTheButterfly
IliTheButterfly deleted the codex/fix-pytest-and-add-testing-documentation-6df34o branch April 21, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant