Skip to content

Fix CI and DEV workflows to use the current test directory - #199

Merged
un33k merged 1 commit into
un33k:masterfrom
naotohayashi1991:item-369/fix-workflow-test-paths
Sep 22, 2026
Merged

un33k merged 1 commit into
un33k:masterfrom
naotohayashi1991:item-369/fix-workflow-test-paths

Conversation

@naotohayashi1991

Copy link
Copy Markdown
Contributor

The CI and DEV workflows still reference root-level test.py and test_release.py after the tests moved into tests/. Their pycodestyle step fails with E902, and pytest exits with code 4 without collecting tests.

Update both workflows to run pycodestyle and pytest against tests/, matching the current test layout and including the uppercase regression tests.

Validation on Linux with CPython 3.12.3 and the text-unidecode backend: 123 tests passed; the workflows' pycodestyle and flake8 commands passed; git diff --check passed. The full interpreter matrix and hosted GitHub Actions have not been run.

AI disclosure: this patch and description were prepared using OpenAI Codex.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 98.758%. remained the same — naotohayashi1991:item-369/fix-workflow-test-paths into un33k:master

@un33k

un33k commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Thank you, @naotohayashi1991 — great catch. When the suite moved into tests/, these two workflows were left pointing at the old root test.py/test_release.py, which is exactly why the pycodestyle step hit E902 and pytest exited without collecting anything. Pointing both at tests/ is the correct fix.

Verified locally: pytest tests collects all 123 tests and the workflow's pycodestyle ... slugify tests setup.py passes clean. Merging now. Since this only updates CI workflow files (no packaged code changes), it doesn't need a release of its own.

🚀 Generated with Dojo ⛩️

@un33k
un33k merged commit e9f287d into un33k:master Sep 22, 2026
2 checks passed
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.

3 participants