-
Notifications
You must be signed in to change notification settings - Fork 11
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
Sweep: Add Unit Tests for the project #85
Comments
Here's the PR! #87.⚡ Sweep Basic Tier: I'm using GPT-4. You have 5 GPT-4 tickets left for the month and 3 for the day.
For more GPT-4 tickets, visit our payment portal. For a one week free trial, try Sweep Pro (unlimited GPT-4 tickets). Actions (click)
Install Sweep Configs: Pull Request
Step 1: 🔎 SearchingI found the following snippets in your repository. I will now analyze these snippets and come up with a plan. Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.Step 2: ⌨️ Coding
Create tests/test_zap_scanner.py with contents:
Modify requirements.txt with contents: Remove the line 'pprint==0.1' from the file. This is a built-in Python module and does not need to be included in the requirements file.
Create tests/test_nexpose_scanner.py with contents:
Create tests/test_openvas_scanner.py with contents:
Create tests/test_storage_service.py with contents:
Modify scanners/zap_scanner.py with contents:
Modify scanners/nexpose_scanner.py with contents:
Modify scanners/openvas_scanner.py with contents:
Modify core/storage_service.py with contents:
Run core/storage_service.py through the sandbox. Step 3: 🔁 Code ReviewI have finished reviewing the code for completeness. I did not find errors for 🎉 Latest improvements to Sweep:
💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. |
Hello can you be able to help me here? I'm trying also to install but I'm receiving the error that Could not build wheels for cffi, lxml, which is required to install pyproject.toml-based projects I get the error when it is now building wheels Building wheels for collected packages: cffi, lxml × python setup.py bdist_wheel did not run successfully. note: This error originates from a subprocess, and is likely not a problem with pip. × python setup.py bdist_wheel did not run successfully. note: This error originates from a subprocess, and is likely not a problem with pip. |
Checklist
tests/test_zap_scanner.py
✓ c858ad4tests/test_zap_scanner.py
✗requirements.txt
✓ 2c720dcrequirements.txt
✗tests/test_zap_scanner.py
✗tests/test_nexpose_scanner.py
✓ 3223bf6tests/test_nexpose_scanner.py
✗tests/test_openvas_scanner.py
✓ b999729tests/test_openvas_scanner.py
✗tests/test_storage_service.py
✓ 5a379datests/test_storage_service.py
✗scanners/zap_scanner.py
✓ 0ffce44scanners/zap_scanner.py
✗scanners/nexpose_scanner.py
✓ 250c51bscanners/nexpose_scanner.py
✗scanners/openvas_scanner.py
✓ 1f754f5scanners/openvas_scanner.py
✗core/storage_service.py
! No changes madeThe text was updated successfully, but these errors were encountered: