Skip to content

test: add smoke suite (learning/framework) and CI (wave-05)#33

Merged
anhmtk merged 1 commit into
mainfrom
tests/smoke-suite-wave-05
Oct 14, 2025
Merged

test: add smoke suite (learning/framework) and CI (wave-05)#33
anhmtk merged 1 commit into
mainfrom
tests/smoke-suite-wave-05

Conversation

@anhmtk
Copy link
Copy Markdown
Owner

@anhmtk anhmtk commented Oct 14, 2025

🧪 Smoke Test Suite - Wave-05

Đã thêm:

1. 📋 pytest.ini - Cấu hình pytest chuẩn

  • Test discovery paths và python path setup
  • Custom markers (smoke, canary, learning, agentdev)
  • Output options tối ưu

2. 🧪 tests/test_learning_smoke.py - Learning System Smoke Tests

  • Test import các learning modules
  • Test components có thể instantiate
  • Test an toàn trong DRY_RUN mode

3. 🏗️ tests/test_framework_smoke.py - Framework Smoke Tests

  • Test framework import và attributes
  • Test initialization (graceful handling API key requirements)
  • Test AgentDev integration
  • Test an toàn trong DRY_RUN mode

4. 🔄 .github/workflows/ci-smoke.yml - CI Workflow

  • Chạy trên PR và push
  • Timeout 15 phút
  • Set STILLME_DRY_RUN=1 environment
  • Chỉ chạy khi có thay đổi liên quan

🎯 Kết quả test:

  • 4 tests PASSED
  • 3 tests SKIPPED (do cần API key - đúng như mong đợi trong DRY_RUN)
  • 0 tests FAILED

🚀 Lợi ích:

  • Contributor mới có thể chạy pytest -q -k "smoke" và thấy xanh trong 5 phút
  • CI/CD tự động chạy smoke tests trên mọi PR
  • DRY_RUN mode đảm bảo không gọi API thật
  • Fast feedback cho developers

Checklist:

  • pytest.ini được tạo với cấu hình chuẩn
  • Learning smoke tests pass
  • Framework smoke tests pass (skip khi cần API key)
  • CI workflow được tạo
  • DRY_RUN mode được set đúng
  • Không có network calls trong tests

@anhmtk anhmtk merged commit ad182da into main Oct 14, 2025
1 of 3 checks passed
anhmtk added a commit that referenced this pull request Nov 21, 2025
test: add smoke suite (learning/framework) and CI (wave-05)
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