[PR #293] [MERGED] add llama_index integration tests #296

Closed
opened 2026-02-16 02:17:10 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/workflows-py/pull/293
Author: @adrianlyjak
Created: 1/22/2026
Status: Merged
Merged: 1/23/2026
Merged by: @adrianlyjak

Base: mainHead: claude/clone-llama-index-repo-yoQgc


📝 Commits (10+)

  • cfb985a chore: add llama_index to gitignore
  • 9a9878a feat: add llama-index-integration-tests package
  • 1a10c31 refactor: use llama-index-core MockFunctionCallingLLM
  • fd9a307 fix: improve integration tests package
  • 34d62a1 refactor: reorganize integration tests by workflow feature
  • c09e633 fix ci
  • e6fea9e better types
  • 36c2238 type fixes/ignores
  • f6d7150 Update packages/llama-index-integration-tests/src/llama_index_integration_tests/helpers.py
  • 5b9c0ab fix

📊 Changes

17 files changed (+972 additions, -21 deletions)

View changed files

📝 .github/workflows/test.yml (+10 -1)
📝 .gitignore (+3 -0)
📝 .pre-commit-config.yaml (+1 -0)
packages/llama-index-integration-tests/README.md (+34 -0)
packages/llama-index-integration-tests/pyproject.toml (+30 -0)
packages/llama-index-integration-tests/src/llama_index_integration_tests/__init__.py (+3 -0)
packages/llama-index-integration-tests/src/llama_index_integration_tests/helpers.py (+50 -0)
packages/llama-index-integration-tests/tests/conftest.py (+134 -0)
packages/llama-index-integration-tests/tests/test_context_store.py (+177 -0)
packages/llama-index-integration-tests/tests/test_error_handling.py (+116 -0)
packages/llama-index-integration-tests/tests/test_event_streaming.py (+163 -0)
packages/llama-index-integration-tests/tests/test_human_in_the_loop.py (+196 -0)
📝 packages/llama-index-workflows/tests/test_decorator.py (+1 -1)
📝 packages/llama-index-workflows/tests/test_resources.py (+4 -4)
📝 packages/llama-index-workflows/tests/test_utils.py (+8 -8)
📝 pyproject.toml (+6 -3)
📝 uv.lock (+36 -4)

📄 Description

Copied adapted llama index core workflows tests to ensure that they run against any core workflows updates.

Also updated ANN checks to run, to enforce better typing in tests. Adds a bunch of check ignores for existing issues that were being ignored


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/workflows-py/pull/293 **Author:** [@adrianlyjak](https://github.com/adrianlyjak) **Created:** 1/22/2026 **Status:** ✅ Merged **Merged:** 1/23/2026 **Merged by:** [@adrianlyjak](https://github.com/adrianlyjak) **Base:** `main` ← **Head:** `claude/clone-llama-index-repo-yoQgc` --- ### 📝 Commits (10+) - [`cfb985a`](https://github.com/run-llama/workflows-py/commit/cfb985ac2da6e7759be6c1ff7213d768708c8fc9) chore: add llama_index to gitignore - [`9a9878a`](https://github.com/run-llama/workflows-py/commit/9a9878aaafbad86e76ea8c127dadc2b2b5c265db) feat: add llama-index-integration-tests package - [`1a10c31`](https://github.com/run-llama/workflows-py/commit/1a10c311b0466d7c25e8fef7110c63f41cea96a6) refactor: use llama-index-core MockFunctionCallingLLM - [`fd9a307`](https://github.com/run-llama/workflows-py/commit/fd9a3074505319d2e439e0ec6abb9aa64b9f6bd7) fix: improve integration tests package - [`34d62a1`](https://github.com/run-llama/workflows-py/commit/34d62a15afae03dbcfe9bcc7b533c922c98ad4c4) refactor: reorganize integration tests by workflow feature - [`c09e633`](https://github.com/run-llama/workflows-py/commit/c09e6331938f58c1497d1bbf3b78f6c6b2eb6ad8) fix ci - [`e6fea9e`](https://github.com/run-llama/workflows-py/commit/e6fea9e7496e1b7f9054843e948fa55d3e5ac60f) better types - [`36c2238`](https://github.com/run-llama/workflows-py/commit/36c2238e0c6e7a4c0b6a0195698ad9e410082e93) type fixes/ignores - [`f6d7150`](https://github.com/run-llama/workflows-py/commit/f6d71505e82b0c2cd0c5ead94ee3fd39a95381a8) Update packages/llama-index-integration-tests/src/llama_index_integration_tests/helpers.py - [`5b9c0ab`](https://github.com/run-llama/workflows-py/commit/5b9c0abe9d3fb1e5435469bb0800f986908c4df5) fix ### 📊 Changes **17 files changed** (+972 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test.yml` (+10 -1) 📝 `.gitignore` (+3 -0) 📝 `.pre-commit-config.yaml` (+1 -0) ➕ `packages/llama-index-integration-tests/README.md` (+34 -0) ➕ `packages/llama-index-integration-tests/pyproject.toml` (+30 -0) ➕ `packages/llama-index-integration-tests/src/llama_index_integration_tests/__init__.py` (+3 -0) ➕ `packages/llama-index-integration-tests/src/llama_index_integration_tests/helpers.py` (+50 -0) ➕ `packages/llama-index-integration-tests/tests/conftest.py` (+134 -0) ➕ `packages/llama-index-integration-tests/tests/test_context_store.py` (+177 -0) ➕ `packages/llama-index-integration-tests/tests/test_error_handling.py` (+116 -0) ➕ `packages/llama-index-integration-tests/tests/test_event_streaming.py` (+163 -0) ➕ `packages/llama-index-integration-tests/tests/test_human_in_the_loop.py` (+196 -0) 📝 `packages/llama-index-workflows/tests/test_decorator.py` (+1 -1) 📝 `packages/llama-index-workflows/tests/test_resources.py` (+4 -4) 📝 `packages/llama-index-workflows/tests/test_utils.py` (+8 -8) 📝 `pyproject.toml` (+6 -3) 📝 `uv.lock` (+36 -4) </details> ### 📄 Description Copied adapted llama index core workflows tests to ensure that they run against any core workflows updates. Also updated ANN checks to run, to enforce better typing in tests. Adds a bunch of check ignores for existing issues that were being ignored --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 02:17:10 -05:00
yindo closed this issue 2026-02-16 02:17:10 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/workflows-py#296