Skip to content

fix(examples): avoid smoke-test collection name collisions - #3812

Merged
LauraGPT merged 1 commit into
mainfrom
fix/example-smoke-collection-20261101
Nov 1, 2026
Merged

LauraGPT merged 1 commit into
mainfrom
fix/example-smoke-collection-20261101

Conversation

@LauraGPT

@LauraGPT LauraGPT commented Nov 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add package markers to the MCP and OpenAI API example directories so their
standalone smoke_test.py scripts have distinct import names under pytest.
Both documented CLI paths remain unchanged. No model, inference, Docker
publication policy, pytest discovery setting or exclusion is changed.

Add a real subprocess regression for prepend, append and importlib collection,
plus both direct --help invocations. A small path-filtered, read-only CI job
runs this and the existing example metadata contracts on Python 3.10 and 3.12,
retaining exact source-commit, dependency and JUnit evidence.

Validation

  • Before the fix: 2 failures (prepend/append import-file mismatch), 3 passing
    controls (importlib and both standalone help commands).
  • After: 12 focused pytest cases pass, zero skips; the existing MCP unittest
    discovery command passes 21 cases. No live service/model/provider call.
  • Bare offline python -m pytest -q was attempted, not silently narrowed. The
    example name collision is gone. Four pre-existing collection errors remain:
    runtime/python/utils/test_cer.py, test_rtf.py, test_rtf_gpu.py parse
    mandatory CLI arguments on import; the separate funasr_onnx dependency is
    missing for tests/test_funasr_onnx_paraformer_online_empty_final.py.
    Two existing optional skips remain. This is NOT a passing full-suite claim.
  • All 4159 original blobs are unchanged. The repair consists of four new files.
    No GPU/runtime change, pretrained ASR quality result, Nano work, release or
    PyPI upload is implied. Actual code review is by the maintainer implementing
    this fix, not claimed as an independent GitHub review.
  • Exact-head CI and downloaded evidence must pass before normal merge.
    No administrator bypass, self approval or protection change.

Namespace both standalone smoke CLIs without renaming commands or hiding tests.

Signed-off-by: LauraGPT <zhifu.gzf@alibaba-inc.com>
@LauraGPT
LauraGPT merged commit 272c981 into main Nov 1, 2026
5 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.

1 participant