Skip to content
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

✨ Improved testing + local development tools #323

Closed
wants to merge 2 commits into from

Conversation

Gary-H9
Copy link
Contributor

@Gary-H9 Gary-H9 commented Oct 30, 2024

This is related to this bug.

🚧 WIP πŸ‘·β€β™‚οΈ 🚧 WIP πŸ‘·β€β™‚οΈ 🚧 WIP πŸ‘·β€β™‚οΈ 🚧 WIP πŸ‘·β€β™‚οΈ 🚧 WIP πŸ‘·β€β™‚οΈ

Currently the new test_handler.py function downloads the ClamAV definitions and scans test files which are created in the container upon running the python file.

It returns:

ClamAV Definitions downloaded successfully.
Scan result for /var/task/test.txt: Clean
Scan result for /var/task/eicar.txt: Infected
Scan result for /var/task/test space.txt: Infected
/var/task/test: No such file or directory
WARNING: /var/task/test: Can't access file
space.txt: No such file or directory
WARNING: space.txt: Can't access file

I need to rectify this error to allow for testing in the event of a file with a space in the name.

Edit 31st Oct. The approach here is wrong at the moment. Pivoting.

@Gary-H9 Gary-H9 changed the title Feat/improved testing ✨ Improved handler testing + local development tools Oct 30, 2024
@Gary-H9 Gary-H9 changed the title ✨ Improved handler testing + local development tools ✨ Improved testing + local development tools Oct 30, 2024
@Gary-H9 Gary-H9 closed this Jan 7, 2025
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