Skip to content

chore(testing): cleanup test running#51

Merged
JonZeolla merged 5 commits into
mainfrom
fix/config/pytest-markers
Aug 20, 2025
Merged

chore(testing): cleanup test running#51
JonZeolla merged 5 commits into
mainfrom
fix/config/pytest-markers

Conversation

@JonZeolla

@JonZeolla JonZeolla commented Aug 20, 2025

Copy link
Copy Markdown
Member

Contributor Comments

This adds the slow marker to our pytest configs in pyproject.toml; required to cleanup a warning as seen here:

=============================== warnings summary ===============================
tests/test_cookiecutter.py:215
  /home/runner/work/ai-native-python/ai-native-python/tests/test_cookiecutter.py:215: PytestUnknownMarkWarning: Unknown pytest.mark.slow - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    @pytest.mark.slow

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=========================== short test summary info ============================
FAILED tests/test_cookiecutter.py::test_default_project - Failed: Command failed with exit code 1. See output above.
====== 1 failed, 2 passed, 36 deselected, 1 warning in 469.09s (0:07:49) =======

This also runs a task update to fix the tests which were previously failing (as shown above)

It also updates the code of conduct to work around a link validation issue with lychee which was changed upstream

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

@JonZeolla JonZeolla enabled auto-merge (squash) August 20, 2025 16:16

@ai-coding-guardrails ai-coding-guardrails Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice work! 😎

I didn't find anything of concern

Reviewed with 🤟 by Zenable

@JonZeolla JonZeolla changed the title chore(testing): configure the slow marker chore(testing): cleanup test running Aug 20, 2025
@JonZeolla JonZeolla merged commit 1fda4f2 into main Aug 20, 2025
7 checks passed
@JonZeolla JonZeolla deleted the fix/config/pytest-markers branch August 20, 2025 17:15
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