Skip to content

test: added test for api/services/rag_pipeline folder#33222

Merged
QuantumGhost merged 115 commits intolanggenius:mainfrom
infocusp-fullstack:test/api-services-ragPipeline
Apr 2, 2026
Merged

test: added test for api/services/rag_pipeline folder#33222
QuantumGhost merged 115 commits intolanggenius:mainfrom
infocusp-fullstack:test/api-services-ragPipeline

Conversation

@akashseth-ifp
Copy link
Copy Markdown
Contributor

@akashseth-ifp akashseth-ifp commented Mar 10, 2026

Checklist

  • This change requires a documentation update.
  • I added unit tests for api/services/rag_pipeline and its sub-folders.
  • I validated the new test files with lint and test runs.
  • Type-check for the added test files is clean in this environment.

Unit Test Added Summary

Added test files

  • api/tests/unit_tests/services/rag_pipeline/test_pipeline_generate_service.py

  • api/tests/unit_tests/services/rag_pipeline/test_pipeline_service_api_entities.py

  • api/tests/unit_tests/services/rag_pipeline/test_rag_pipeline_dsl_service.py

  • api/tests/unit_tests/services/rag_pipeline/test_rag_pipeline_manage_service.py

  • api/tests/unit_tests/services/rag_pipeline/test_rag_pipeline_service.py

  • api/tests/unit_tests/services/rag_pipeline/test_rag_pipeline_transform_service.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_built_in_retrieval.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_customized_retrieval.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_database_retrieval.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_package_imports.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_pipeline_template_base.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_pipeline_template_factory.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_pipeline_template_type.py

  • api/tests/unit_tests/services/rag_pipeline/pipeline_template/test_remote_retrieval.py

  • Result: ✅ Passed (80/80 tests)

Validation Commands

  • uv run --project api ruff check api/tests/unit_tests/services/rag_pipeline api/tests/unit_tests/services/test_rag_pipeline_task_proxy.py
  • uv run --project api mypy --follow-imports=skip api/tests/unit_tests/services/rag_pipeline
  • uv run --project api pytest -q api/tests/unit_tests/services/rag_pipeline api/tests/unit_tests/services/test_rag_pipeline_task_proxy.py

Coverage Report (Changed Source Files)

File Lines (%) Branch (%)
api/services/entities/knowledge_entities/rag_pipeline_entities.py 95.29 87.50
api/services/rag_pipeline/entity/pipeline_service_api_entities.py 100.00 100.00
api/services/rag_pipeline/pipeline_generate_service.py 94.59 90.91
api/services/rag_pipeline/pipeline_template/built_in/built_in_retrieval.py 100.00 100.00
api/services/rag_pipeline/pipeline_template/customized/customized_retrieval.py 100.00 100.00
api/services/rag_pipeline/pipeline_template/database/database_retrieval.py 100.00 100.00
api/services/rag_pipeline/pipeline_template/pipeline_template_base.py 100.00 100.00
api/services/rag_pipeline/pipeline_template/pipeline_template_factory.py 100.00 100.00
api/services/rag_pipeline/pipeline_template/pipeline_template_type.py 100.00 100.00
api/services/rag_pipeline/pipeline_template/remote/remote_retrieval.py 100.00 100.00
api/services/rag_pipeline/rag_pipeline.py 92.82 86.21
api/services/rag_pipeline/rag_pipeline_dsl_service.py 93.60 88.04
api/services/rag_pipeline/rag_pipeline_manage_service.py 100.00 100.00
api/services/rag_pipeline/rag_pipeline_transform_service.py 88.62 87.27

Fixes Issue - #32582

@github-actions
Copy link
Copy Markdown
Contributor

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Pyrefly Diff

No changes detected.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Pyrefly Diff

No changes detected.

@QuantumGhost QuantumGhost added this pull request to the merge queue Apr 2, 2026
Merged via the queue into langgenius:main with commit e54383d Apr 2, 2026
27 checks passed
volcano303 pushed a commit to volcano303/dify that referenced this pull request Apr 7, 2026
Co-authored-by: sahil-infocusp <73810410+sahil-infocusp@users.noreply.github.com>
HanqingZ pushed a commit to HanqingZ/dify that referenced this pull request Apr 23, 2026
Co-authored-by: sahil-infocusp <73810410+sahil-infocusp@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👻 feat:rag Embedding related issue, like qdrant, weaviate, milvus, vector database. lgtm This PR has been approved by a maintainer needs revision for anti-slop needs-revision size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants