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.

Sign up for GitHub

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

Jump to bottom

added unittests for redis_manager.py #944

Merged
merged 489 commits into from
Oct 21, 2024
Merged

added unittests for redis_manager.py #944

merged 489 commits into from
Oct 21, 2024

Conversation

Sekhar-Kumar-Dash
Copy link
Collaborator

Fixes Issue #803

Changes proposed

added unittests for redis_manager.py

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

@AlyaGomaa
Remove duplicate print functions from all modules and undo making all…
eadd7da 
… slips modules observables, instead, use the printer, which is an observable
@AlyaGomaa
rename evidence_structure/ to structures/
28820de
@AlyaGomaa
rename evidence_structure/ to structures/
352bef0
@AlyaGomaa
force that the start_time and end_time of every timewindow be in iso …
f2157ad 
…format
@AlyaGomaa
unify how modules check if slips is running non stop
7405b7b
@AlyaGomaa
flowalerts: Handle problem setting conn without dns in interface mode
4e16e44
@AlyaGomaa
change how slips stores and retrieves the current host ip
f7e9c5d
@AlyaGomaa
idmef: use the current host ip when running on an interface, and the …
ea2e512 
…default ip otherwise
@AlyaGomaa
Add a flow classifier to turn the msg received in redis channels to t…
868dfbc 
…heir corresponding flow object
@AlyaGomaa
flowalerts: use flow objects instead of dicts
0ee2469
@AlyaGomaa
evidence.py: Add an optional src and dst port field to each Evidence
8cb81b8
@AlyaGomaa
suricata.py: add a flow_source field for the classifier to differenti…
c1e7b62 
…ate between suricata flows and conn.log flows
@AlyaGomaa
profiler_handler: pass the flows ad dicts instead of unpacking them
9be55b9
@AlyaGomaa
idmef: use the version as the model and add "Network" to the "Data" f…
e336595 
…ield
@AlyaGomaa
arp: use flow obj instead of dict
9ee4254
@AlyaGomaa
http: use flow obj instead of dict
d76550c
@AlyaGomaa
ip info: use flow obj instead of dict
19f14c6
@AlyaGomaa
network_discovery: use flow obj instead of dict
382ac0b
@AlyaGomaa
virustotal: use flow obj instead of dict
d12c3b1
@AlyaGomaa
timeline: use flow obj instead of dict
8aeeb2a
@AlyaGomaa
vt: use flow obj instead of dict
589fdba
@AlyaGomaa
move the function that converts dicts to flow objects to the classifi…
b576c71 
…er instead of slips utils to avoid circular imports
@AlyaGomaa
use flow.starttime instead of flow.timestamp
3e7fdd4
@AlyaGomaa
arp: fix problem accessing ARP flows' src and dst macs
486cdb4
@AlyaGomaa
flowmldetection.py: adapt to the new flow format
c27321b
@AlyaGomaa
handle each evidence having src and dst port for idmef format instead…
cd9f402 
… of having one "port" field
@AlyaGomaa
add a "state" and an "interpreted state" for each flow
b10efbc
@AlyaGomaa
idmef: use the evidence sport and dport
b0dc7b4
@AlyaGomaa
flowalerts: remove all occurances of flow.allbytes it's no longer sup…
b973a40 
…ported
@AlyaGomaa
flowalerts: fix refactoring errors
1f45d2e
AlyaGomaa and others added 27 commits October 9, 2024 14:49
@AlyaGomaa
Merge remote-tracking branch 'origin/develop' into fork/Sekhar-Kumar-…
87da435 
…Dash/patch-49

# Conflicts:
#	.github/workflows/unit-tests.yml
@AlyaGomaa
test_evidence_handler.py: fix test_format_evidence_for_printing() dep…
22863b6 
…ending on the timezone
@AlyaGomaa
Merge pull request stratosphereips#915 from Sekhar-Kumar-Dash/patch-49
98cf310 
added test for evidence handler
@AlyaGomaa
update contributing FAQ
1994ccf
@AlyaGomaa
update contributing FAQ
c2fb60a
@AlyaGomaa
Print more info in the CLI like, host ip, used network if recognized,…
ec90f96 
… used slips threshold, used twid width, etc.
@AlyaGomaa
Merge pull request stratosphereips#1032 from stratosphereips/alya/pri…
77dbc63 
…nt_more_info

Print more info
@AlyaGomaa
better logging of idmef errors
f51fbe8
@AlyaGomaa
db: normalize the confidence of each evidence, they all should range …
7e0129e 
…from 0 to 1
@AlyaGomaa
Merge pull request stratosphereips#1027 from stratosphereips/dependab…
53898d0 
…ot/pip/install/develop/redis-5.1.1

build(deps): bump redis from 5.0.8 to 5.1.1 in /install
@Sekhar-Kumar-Dash @AlyaGomaa
Created test_alert_handler.py
0e6e467
@Sekhar-Kumar-Dash @AlyaGomaa
Updated module_factory.py
908c14d
@Sekhar-Kumar-Dash @AlyaGomaa
Update module_factory.py
7b1b5e8
@AlyaGomaa
fix indentation error
a988137
@AlyaGomaa
Merge pull request stratosphereips#1034 from stratosphereips/alya/fix…
2608095 
…-evidence-handler

Better logging of IDMEF errors and normalize the confidence of all evidence to range from 0 to 1
@AlyaGomaa
fix test_set_evidence_causing_alert()
aa8c940
@AlyaGomaa
unit_tests: add test_alert_handler.py to CI
eafe9ed
@AlyaGomaa
db: make set_accumulated_threat_level() a private function
e41c0e6
@AlyaGomaa
update unit tests
90bb450
@AlyaGomaa
Merge pull request stratosphereips#943 from Sekhar-Kumar-Dash/patch-52
0730b50 
added unit tests for alert handler
@dependabot
build(deps): bump pre-commit from 3.8.0 to 4.0.1 in /install
aa4b512 
Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 3.8.0 to 4.0.1.
- [Release notes](https://github.com/pre-commit/pre-commit/releases)
- [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md)
- [Commits](pre-commit/pre-commit@v3.8.0...v4.0.1)

---
updated-dependencies:
- dependency-name: pre-commit
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@AlyaGomaa
Merge pull request stratosphereips#1035 from stratosphereips/dependab…
dcfe707 
…ot/pip/install/develop/pre-commit-4.0.1

build(deps): bump pre-commit from 3.8.0 to 4.0.1 in /install
@Sekhar-Kumar-Dash @AlyaGomaa
Create test_redis_manager.py
39ed6cb
@AlyaGomaa
CI: add etst_Redis_manager to unit tests
ecf6f50
@AlyaGomaa
Merge remote-tracking branch 'Sekhar-Kumar-Dash/patch-53' into fork/S…
f6ca6d8 
…ekhar-Kumar-Dash/patch-53

# Conflicts:
#	.github/workflows/CI-production-testing.yml
#	.github/workflows/CI-staging.yml
@AlyaGomaa
redis_manager: refactor
4acd583
@AlyaGomaa
test_redis_manager.py: update unit tests
2872725
@AlyaGomaa AlyaGomaa merged commit 56a7842 into stratosphereips:develop Oct 21, 2024
48 checks passed
@AlyaGomaa
Copy link
Collaborator

Thanks for the tests! @Sekhar-Kumar-Dash 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Waiting for CI to pass
Projects
Slips
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Sekhar-Kumar-Dash @AlyaGomaa