Skip to content

Commit 13d073b

Browse files
fix: move scripts folder from root to appropriate location (#1661)
* fix: move scripts folder from root to appropriate location (fixes #1659) - Move all smoke test scripts from /scripts to src/praisonai/tests/integration/ - Update documentation in moved files to reflect new paths - Per AGENTS.md architecture, these bot/BotOS integration tests belong in praisonai wrapper tests - Preserves git history with git mv command Co-authored-by: MervinPraison <MervinPraison@users.noreply.github.com> * fix: add __test__ = False to smoke test files to prevent pytest collection - Prevents accidental pytest discovery in environments with different collection rules - Follows Python testing best practices for non-test files in test directories - Addresses Copilot's review feedback for explicit test exclusion --------- Co-authored-by: praisonai-triage-agent[bot] <272766704+praisonai-triage-agent[bot]@users.noreply.github.com> Co-authored-by: MervinPraison <MervinPraison@users.noreply.github.com>
1 parent adcfa29 commit 13d073b

4 files changed

Lines changed: 31 additions & 11 deletions

File tree

scripts/smoke_n4_real.py renamed to src/praisonai/tests/integration/smoke_n4_real.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
#!/usr/bin/env python3
2+
from __future__ import annotations
3+
24
"""N4 — Real agentic smoke test for inbound DLQ.
35
6+
This is not a pytest test file - it's an integration smoke test.
7+
48
Scenario:
59
1. Real Agent (anthropic/claude-haiku-4-5 or gemini/2.5-flash).
610
2. First chat attempt fails (we patch agent.chat to raise once).
@@ -11,11 +15,12 @@
1115
1216
Run::
1317
14-
cd /Users/praison/worktrees/n4-dlq
15-
PYTHONPATH=src/praisonai-agents:src/praisonai python scripts/smoke_n4_real.py
18+
cd /path/to/PraisonAI
19+
PYTHONPATH=src/praisonai-agents:src/praisonai python src/praisonai/tests/integration/smoke_n4_real.py
1620
"""
1721

18-
from __future__ import annotations
22+
# Prevent pytest from treating this as a test file
23+
__test__ = False
1924

2025
import asyncio
2126
import os

scripts/smoke_w1_botos_real.py renamed to src/praisonai/tests/integration/smoke_w1_botos_real.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
#!/usr/bin/env python3
2+
from __future__ import annotations
3+
24
"""W1 real agentic smoke test (high-level BotOS API, no manual mgr).
35
6+
This is not a pytest test file - it's an integration smoke test.
7+
48
Uses the user-facing API only — ``Bot``, ``BotOS``, ``InMemoryIdentityResolver`` —
59
and exercises the same cross-platform-continuity scenario as
610
``smoke_w1_real.py`` but without touching ``BotSessionManager`` directly.
@@ -13,10 +17,11 @@
1317
Run::
1418
1519
# From the repository root:
16-
PYTHONPATH=src/praisonai-agents:src/praisonai python scripts/smoke_w1_botos_real.py
20+
PYTHONPATH=src/praisonai-agents:src/praisonai python src/praisonai/tests/integration/smoke_w1_botos_real.py
1721
"""
1822

19-
from __future__ import annotations
23+
# Prevent pytest from treating this as a test file
24+
__test__ = False
2025

2126
import asyncio
2227
import os

scripts/smoke_w1_real.py renamed to src/praisonai/tests/integration/smoke_w1_real.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,18 @@
11
#!/usr/bin/env python3
2+
from __future__ import annotations
3+
24
"""W1 real agentic smoke test (bypasses pytest conftest gating).
35
6+
This is not a pytest test file - it's an integration smoke test.
7+
48
Runs::
59
6-
cd src/praisonai
7-
python ../../scripts/smoke_w1_real.py
10+
cd /path/to/PraisonAI
11+
PYTHONPATH=src/praisonai-agents:src/praisonai python src/praisonai/tests/integration/smoke_w1_real.py
812
"""
913

10-
from __future__ import annotations
14+
# Prevent pytest from treating this as a test file
15+
__test__ = False
1116

1217
import asyncio
1318
import os

scripts/smoke_w1_robust.py renamed to src/praisonai/tests/integration/smoke_w1_robust.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
#!/usr/bin/env python3
2+
from __future__ import annotations
3+
24
"""W1 deep-validation smoke test.
35
6+
This is not a pytest test file - it's an integration smoke test.
7+
48
Exercises every robustness dimension of W1 with REAL LLM calls:
59
610
T1. 3-platform unification (telegram → discord → slack, one user)
@@ -12,11 +16,12 @@
1216
1317
Run::
1418
15-
cd /Users/praison/worktrees/hermes-parity
16-
PYTHONPATH=src/praisonai-agents:src/praisonai python scripts/smoke_w1_robust.py
19+
cd /path/to/PraisonAI
20+
PYTHONPATH=src/praisonai-agents:src/praisonai python src/praisonai/tests/integration/smoke_w1_robust.py
1721
"""
1822

19-
from __future__ import annotations
23+
# Prevent pytest from treating this as a test file
24+
__test__ = False
2025

2126
import asyncio
2227
import os

0 commit comments

Comments
 (0)