Skip to content

Commit

Permalink
Run test-llm-load-plugins.sh, refs #378
Browse files Browse the repository at this point in the history
  • Loading branch information
simonw committed Jan 26, 2024
1 parent ed5a535 commit 75db4a5
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,3 +41,7 @@ jobs:
- name: Run ruff
run: |
ruff .
- name: Run test-llm-load-plugins.sh
run: |
llm install llm-cluster llm-sentence-transformers
./tests/test-llm-load-plugins.sh
29 changes: 29 additions & 0 deletions tests/test-llm-load-plugins.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
#!/bin/bash
# This should only run in environemnts where both
# llm-cluster and llm-sentence-transformers are installed

PLUGINS=$(llm plugins)
echo "$PLUGINS" | jq 'any(.[]; .name == "llm-sentence-transformers")' | \
grep -q true || ( \
echo "Test failed: llm-sentence-transformers not found" && \
exit 1 \
)
# With the LLM_LOAD_PLUGINS we should not see that
PLUGINS2=$(LLM_LOAD_PLUGINS=llm-cluster llm plugins)
echo "$PLUGINS2" | jq 'any(.[]; .name == "llm-sentence-transformers")' | \
grep -q false || ( \
echo "Test failed: llm-sentence-transformers should not have been loaded" && \
exit 1 \
)
echo "$PLUGINS2" | jq 'any(.[]; .name == "llm-cluster")' | \
grep -q true || ( \
echo "Test llm-cluster should have been loaded" && \
exit 1 \
)
# With LLM_LOAD_PLUGINS='' we should see no plugins
PLUGINS3=$(LLM_LOAD_PLUGINS='' llm plugins)
echo "$PLUGINS3"| \
grep -q '\[\]' || ( \
echo "Test failed: plugins should have returned []" && \
exit 1 \
)

0 comments on commit 75db4a5

Please sign in to comment.