-
Notifications
You must be signed in to change notification settings - Fork 14
chore: Add path filters for integrations #122
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
93d5abf
Add fixtures for detecting if langchain, langgraph and deepagents are…
dagardner-nv fcb8880
Move fixtures
dagardner-nv 106da78
Merge branch 'release/0.2' of github.com:NVIDIA/NeMo-Flow into david-…
dagardner-nv 8a2021b
Rename 'langchain' dir to prevent 'import langchain' from accidentall…
dagardner-nv 231d150
Remove unneeded fixture dependency
dagardner-nv f129828
Merge branch 'main' of github.com:NVIDIA/NeMo-Flow into david-path-fi…
dagardner-nv d2ceaed
Make langchain tests skippable when not installed
dagardner-nv 1aa9087
Instruct the agent how to write fixtures
dagardner-nv b119ece
Refactor _model_request and _tool_call_request as fixtures
dagardner-nv f7a7b32
WIP
dagardner-nv 96110f9
Merge branch 'main' of github.com:NVIDIA/NeMo-Flow into david-path-fi…
dagardner-nv 535040c
Rename tool_request fixture to tool_call_request
dagardner-nv 2ffa713
Rename langgraph test dir to not conflict with the langgraph package
dagardner-nv 9c6350c
Rename deepagents test dir to not conflict with the deepagents package
dagardner-nv df7f356
Perform langgraph imports lazily [skip ci]
dagardner-nv b915df0
Remove constructor, avoids warning about not being able to collect th…
dagardner-nv 59332a3
Perform lazy imports for deepagents tests
dagardner-nv 317c902
WIP
dagardner-nv 39e77fa
Exlude integration tests from coverage reports
dagardner-nv 93670c9
Seperate the just recipes for python [skip ci]
dagardner-nv 661d505
Optionally run Python LC integration tests
dagardner-nv 58decb5
Formatting/linting
dagardner-nv 6b22798
Use return to match return type hint
dagardner-nv 0341d2c
Linting fixes
dagardner-nv 9add9b5
Formatting
dagardner-nv c044f9d
Avoid installing LC deps if we don't need to
dagardner-nv 4e48df5
Linting fix
dagardner-nv 8d6c142
Linting fix
dagardner-nv d4737b6
Linting
dagardner-nv 12793ee
Fix handling of uv sync flags
dagardner-nv 005b767
Fix recipe name
dagardner-nv cc72e20
Formatting
dagardner-nv File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,45 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| import types | ||
|
|
||
| import pytest | ||
|
|
||
|
|
||
| @pytest.fixture(name="integration_langchain", scope="session") | ||
| def integration_langchain_fixture() -> types.ModuleType: | ||
| """ | ||
| Use for integration tests that require LangChain to be installed. | ||
| """ | ||
| try: | ||
| import langchain | ||
|
|
||
| return langchain | ||
| except Exception: | ||
| pytest.skip(reason="langchain must be installed to run LangChain based tests") | ||
|
dagardner-nv marked this conversation as resolved.
|
||
|
|
||
|
|
||
| @pytest.fixture(name="integration_langgraph", scope="session") | ||
| def integration_langgraph_fixture(integration_langchain: types.ModuleType) -> types.ModuleType: | ||
| """ | ||
| Use for integration tests that require LangGraph to be installed. | ||
| """ | ||
| try: | ||
| import langgraph | ||
|
|
||
| return langgraph | ||
| except Exception: | ||
| pytest.skip(reason="langgraph must be installed to run LangGraph based tests") | ||
|
|
||
|
|
||
| @pytest.fixture(name="integration_deepagents", scope="session") | ||
| def integration_deepagents_fixture(integration_langgraph: types.ModuleType) -> types.ModuleType: | ||
| """ | ||
| Use for integration tests that require Deep Agents to be installed. | ||
| """ | ||
| try: | ||
| import deepagents | ||
|
|
||
| return deepagents | ||
| except Exception: | ||
| pytest.skip(reason="deepagents must be installed to run Deep Agents based tests") | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| import types | ||
|
|
||
| import pytest | ||
|
|
||
|
|
||
| @pytest.fixture(name="integration_deepagents", scope="session", autouse=True) | ||
| def integration_deepagents_fixture(integration_deepagents: types.ModuleType) -> types.ModuleType: | ||
| """ | ||
| Override the integration_deepagents fixture to make it autouse | ||
| """ | ||
| return integration_deepagents |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.