Skip to content

TEST: smoke-test own-data workflow examples#399

Open
biru-codeastromer wants to merge 1 commit intosktime:mainfrom
biru-codeastromer:fix/own-data-example-smoke-tests
Open

TEST: smoke-test own-data workflow examples#399
biru-codeastromer wants to merge 1 commit intosktime:mainfrom
biru-codeastromer:fix/own-data-example-smoke-tests

Conversation

@biru-codeastromer
Copy link
Copy Markdown
Contributor

Summary

This PR adds smoke coverage for the repository's runnable own-data examples and makes the SQL example degrade cleanly when its optional dependency is not installed.

Closes #398.

Why

The repo documents MCP workflows on user data (pandas, csv, sql), but those example scripts were not covered by tests. In a standard dev environment without sqlalchemy, examples/sql_example.py currently attempts the runnable SQLite workflow anyway and emits noisy executor error logs before reaching its template-only guidance.

Changes

  • make examples/sql_example.py check for SQL support up front and print a clean install/skip message when sqlalchemy is unavailable
  • add smoke tests for:
    • examples/03_pipeline_instantiation.py
    • examples/pandas_example.py
    • examples/csv_example.py
    • examples/sql_example.py

Verification

  • .venv/bin/ruff check examples/sql_example.py tests/test_example_data_workflows.py
  • .venv/bin/ruff format --check examples/sql_example.py tests/test_example_data_workflows.py
  • .venv/bin/python examples/sql_example.py
  • .venv/bin/pytest tests/test_example_data_workflows.py -q
  • .venv/bin/pytest -q

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.

[BUG] own-data examples are not smoke-tested and SQL example fails noisily without optional dependencies

1 participant