Skip to content

test: increase test coverage #742

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ImBIOS
Copy link

@ImBIOS ImBIOS commented May 3, 2025

This PR adds unit tests to increase test coverage for the BYOB project, focusing on core functionality and modules.

Changes

  • Added tests for the security module (AES encryption/decryption, CBC mode, and Diffie-Hellman key exchange)
  • Added tests for the portscanner module with proper mocking to avoid actual network connections
  • Added a Makefile to simplify running tests and generating coverage reports
  • Added requirements-dev.txt for development dependencies

Benefits

  • Ensures critical security components work as expected
  • Validates module functionality without requiring actual network connections
  • Sets up a foundation for more comprehensive testing coverage
  • Makes it easier for contributors to run tests and keep code quality high

Testing

The tests can be run with:

# Install development dependencies
pip install -r requirements-dev.txt

# Run tests
make test

# Run tests with coverage
make coverage

Future Improvements

Additional test coverage could be added for:

  • The web GUI
  • Other modules like keylogger, persistence, etc.
  • Client-server communication
  • Integration tests for end-to-end functionality

This PR is a first step toward comprehensive test coverage for the entire project.

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