intentservice global state (`is_loaded`/`intents_active`) not reset between tests

Open Beginner friendly

#2,124 opened on Aug 27, 2026 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 2/5
Estimated time: 1-3 hours
Newbie friendliness: 78/100

Issue type: Bug
Clarity: Clearly specified
Activity status: Active
Tech stack: python
Domain: testing-qa

Research direction

Start with tests/langservice/conftest.py::clear_langprovider_state and compare its fixture setup with the intentservice globals is_loaded and intents_active. Run the two pytest command orderings from the issue, then verify the equivalent intentservice state cleanup makes results independent of test order.

Written by the indexing model from the issue text.

Description

bug needs-triage

intentservice.is_loaded/intents_active are module globals and no fixture resets them. Suite runs single-process, so whichever test last called load() decides the next test's state.

Observed while reviewing #1775: test_probe_prompt_translation[probes.tap.TAPIntent] flips result by run order alone:

python -m pytest -p no:cacheprovider   "tests/langservice/probes/test_probes_base.py::test_probe_prompt_translation[probes.tap.TAPIntent]" "tests/probes/test_probes.py::test_probe_metadata[probes.audio.AudioAchillesHeel]"  -v
# SKIPPED

python -m pytest -p no:cacheprovider    "tests/probes/test_probes.py::test_probe_metadata[probes.audio.AudioAchillesHeel]" "tests/langservice/probes/test_probes_base.py::test_probe_prompt_translation[probes.tap.TAPIntent]"  -v
# FAILED: assert 0 == 93 (real bug: TAPIntent.probe() never calls get_text)

Pre-existing, outside #1775's diff

Precedent: langservice has the same issue, already fixed via tests/langservice/conftest.py::clear_langprovider_state. Needs the equivalent for intentservice.

Dominant language: Python
Stars: 9.3k
Forks: 1.3k
Avg merge: 4d 18h
Merged PRs (30d): 12

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

[BUG] GkeTpuCluster queries metadata.google.internal for MEGASCALE_SLICE_ID even when TPU_SKIP_MDS_QUERY is set Open

Difficulty 2/5 1-3 hours Newbie friendliness 88/100

jax-ml/jax#40854 · Opened Sep 22, 2026
[pcied] Honour PCIE_DETACH_INFO on platforms that are not smartswitches Open

Difficulty 2/5 1-3 hours Newbie friendliness 82/100

sonic-net/sonic-platform-daemons#900 · Opened Sep 21, 2026
MCP client has no httpx timeout, so the event stream fails after 5 seconds Open

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

Azure/azure-functions-agents-runtime#228 · Opened Sep 21, 2026
feat(mcp): friendlier error when `run_cloud_sync` is called on a disabled connection Open

enhancement

Difficulty 2/5 1-3 hours Newbie friendliness 72/100

airbytehq/PyAirbyte#1205 · Opened Sep 21, 2026
pn7160: reader goes permanently deaf — post-read IRQ-low check fails reads that succeeded Open

Difficulty 2/5 1-3 hours Newbie friendliness 74/100

esphome/esphome#19452 · Opened Sep 21, 2026

More Python issues