-
Notifications
You must be signed in to change notification settings - Fork 291
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
Change tor_exit_nodes feed to use the Tor Onionoo API + introduce tests #1029
Conversation
tests/analytics_test.py
Outdated
@@ -13,48 +13,11 @@ | |||
from core.schemas.indicator import DiamondModel | |||
from core.schemas.observable import ObservableType | |||
from plugins.analytics.public import censys, expire_tags, shodan | |||
from core.database_arango import ArangoYetiConnector |
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.
Needs to be removed
tests/feeds.py
Outdated
defaults["name"] = "FeodoTrackerIPBlocklist" | ||
feed = feodo_tracker_ip_blocklist.FeodoTrackerIPBlockList(**defaults) | ||
feed.run() | ||
# def test_feodo_tracker_ip_blocklist(self): |
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.
Need to uncomment all these, was for debug purposes
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.
Whenever you feel ready :)
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.
Minor changes, otherwise this looks good! Thanks :)
tests/feeds.py
Outdated
defaults["name"] = "FeodoTrackerIPBlocklist" | ||
feed = feodo_tracker_ip_blocklist.FeodoTrackerIPBlockList(**defaults) | ||
feed.run() | ||
# def test_feodo_tracker_ip_blocklist(self): |
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.
Whenever you feel ready :)
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.
Thanks a lot!
and move test helper files to a separate module