-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add test file for AI VPN CLI: aivpn.py #59
base: develop
Are you sure you want to change the base?
Conversation
@verovaleros Kindly review this new PR please :) |
Can you write one sentence for each function test with what you aim to test? For every component of the CLI we can test for a wide variety of things, so I want to make sure we both understand exactly what functionality your tests aim to cover. Tested locally, and all tests passed, but I want to double-check we agree on the logic. Thanks! |
I will do one thing then @verovaleros. |
That sounds good. I love small atomic commits with clear messages so I can track the changes as well. Thanks! |
These are some info related to each function test as you asked test_manage_info_active_profile(self):: This is a test method for the manage_info function when the profile is active. test_manage_info_expired_profile(self):: This is a test method for the manage_info function when the profile is expired. test_manage_expire(self):: This is a test method for the manage_expire function. Inside this method, I mocked various functions related to profile expiration to simulate their behavior during testing. |
…n provisioning queue
Pull request type
Please check the type of change your PR introduces:
What is the current behavior?
Closes #52
What is the new behavior?
The project is not having any sort of unit testing right now so this is a sample unit test to start the process of "code enhancement using unit testing" project.
Other information
This unit test is related to AIVPN file.