-
Notifications
You must be signed in to change notification settings - Fork 164
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
Added more tests to test_whitelist.py #565
base: develop
Are you sure you want to change the base?
Added more tests to test_whitelist.py #565
Conversation
…cts with tensorflow
…unning, either using -S or -D
… current daemon to run it
…y cant be used with the daemon
…aemon's or the Main() class'
Hey @AlyaGomaa , it is saying the process is canceled. I could not get it. Did it happen because of errors in the code? |
…k_why_redis_config_file_is_modified_each_run only overwrite redis config file when the dafault options are modified
Updated and added more tests to test_flow_handler.py
@Sekhar-Kumar-Dash i don't know that was weird, but no this is not coming from your code. i restarted CI and we'll see |
…t_the_used_cmd_in_metadata Add the cmd used to metadata/info
…k_exporting_of_sensitive_evidence Fix exporting of sensitive evidence
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @Sekhar-Kumar-Dash thanks for all the hard work!!
i left some comments, and i will wait for you to fix them and re-review the PR.
again thanks for making Slips better :)!
hey @AlyaGomaa Just updated the test file please check |
Fixes Issue #564
This pull request adds more tests for whitelist.py