Closed sweep-ai[bot] closed 10 months ago
The command:
Run pipenv run black --check .
yielded the following error:
`##[error]Process completed with exit code 1.
[36;1mdocker build . -t pulling_ace:test[0m shell: /usr/bin/bash -e {0}
[python-deps 4/7] COPY pulling_ace /app/pulling_ace:
Dockerfile:19
17 |
18 | # Copy the pulling_ace directory into the Docker image 19 | >>> COPY pulling_ace /app/pulling_ace 20 |
21 | # Install pipenv and compilation dependenciesERROR: failed to solve: failed to compute cache key: failed to calculate checksum of ref 5ee0a90d-9ba7-4120-a52c-7d8898402fd7::bjiwm5fgpc3vvj2h9w6817fzk: "/pulling_ace": not found
[error]Process completed with exit code 1.`
Here are the logs:
would reformat pulling_ace/cli.py would reformat test/test_subprocessor.py would reformat pulling_ace/utils/subprocessor.py
Oh no! 💥 💔 💥 3 files would be reformatted, 18 files would be left unchanged.
❌ Failed to make changes
Please join our Discord to report this issue.
PR Feedback (click)
I created this PR to address this rule:
"Code should always include unit tests."
Description
This PR adds unit tests for the functions in the file
pulling_ace/utils/subprocessor.py
. The functionstoxicity
,promptinjection
,riskcards
,riskcard_wrapper
,promptinjection_wrapper
,toxicity_wrapper
, andrun_injections
now have corresponding unit tests to ensure their correctness.Summary of Changes
pulling_ace/utils/subprocessor.py
.test/test_subprocessor.py
to contain the unit tests.pulling_ace/utils/subprocessor.py
has a corresponding test function intest/test_subprocessor.py
.Please review and merge this PR to ensure the reliability and correctness of the code in
pulling_ace/utils/subprocessor.py
.