Skip to content

Only copy source code to test notebook if install URI not supplied#197

Merged
iansrobinson merged 1 commit intomainfrom
int-test-install-options
Apr 16, 2026
Merged

Only copy source code to test notebook if install URI not supplied#197
iansrobinson merged 1 commit intomainfrom
int-test-install-options

Conversation

@iansrobinson
Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

By default, the integration tests copy the lexical graph source to the test notebook. When the tests run, they use this local version of the toolkit.

With this change, if a --lexical-graph-install parameter is supplied, the source will not be copied to the notebook. This ensures the tests run against the install supplied in the parameter.

For example:

sh build-tests.sh --test-file lexical.short

will run tests against the source code copied to the notebook, whereas

sh build-tests.sh --test-file lexical.short --lexical-graph-install graphrag-lexical-graph

will run the tests against the latest version of the toolkit from PyPi.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@iansrobinson iansrobinson merged commit 9699b35 into main Apr 16, 2026
3 checks passed
@iansrobinson iansrobinson deleted the int-test-install-options branch April 16, 2026 14:44
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