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

Created unittest for profile_handler.py #966

Merged
merged 805 commits into from
Jan 3, 2025
Merged

Created unittest for profile_handler.py #966

merged 805 commits into from
Jan 3, 2025

Conversation

Sekhar-Kumar-Dash
Copy link
Collaborator

Fixes Issue #803

Changes proposed

this adds a unittest for profile_handler

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
update slips version to 1.1.3
beae32a
@AlyaGomaa
update slips.gif to use 1.1.3
156078e
@AlyaGomaa
CI: add test_database.py and test_symbols_handler to unit tests
300c663
@AlyaGomaa
test_database.py: use a different port for each unit test
e80c38a
@AlyaGomaa
update profiler unit tests
558a081
@AlyaGomaa
CI: cache zeek builds
ccff6dd
@AlyaGomaa
unit-tests: cache apt and pip dependencies
9c57833
@AlyaGomaa
unit-tests: hardcode cache keys to avoid cache miss
4c9e7f1
@AlyaGomaa
unit-tests:use actions/setup-python@v5 for caching python dependencies
91b34de
@AlyaGomaa
add apt_Dependencies file in install/ to avoid repetition of dependen…
a0fa9b2 
…cies in dockerfiles and github runners
@AlyaGomaa
CI: try caching /var/cache/ and /var/lib
42b1b14
@AlyaGomaa
CI: install python dependencies/restore from cache before running uni…
24be296 
…t tests
@AlyaGomaa
CI: cache pip, zeek and apt dependencies in integration tests workflo…
c73f7f6 
…w to avoid building slips for each test
@AlyaGomaa
CI: unit-tests: run unit tests without installing zeek
f35b258
@AlyaGomaa
CI: make installing slips a reusable workflow
1988fcd
@AlyaGomaa
CI: fix a typo in install-slips-dependencies.yml
cf2a88d
@AlyaGomaa
CI: make hardcoded zeek repos optional variables
2a96716
@AlyaGomaa
CI: fix installing python dependencies
4f30789
@AlyaGomaa
CI: integration-tests: fix using reusable workflow
054b7c2
@AlyaGomaa
CI: dependencies: add missing "type" property to workflow_call
f5fbfac
@AlyaGomaa
CI: make sure integration-tests can access Zeek from the cache if it …
4a06d6c 
…was successfully installed by install-dependencies-using-reusable-workflo
@AlyaGomaa
Merge pull request stratosphereips#1056 from stratosphereips/alya/reu…
24d22d6 
…sable-workflows

CI: cache zeek builds
@AlyaGomaa
move all the profiling logic from main to managers/profiler_manager.py
1c88582
@AlyaGomaa
profiler_manager.py: delete the memory_profiler_multiproc_test()
79c0816
@AlyaGomaa
profiler_manager.py: import viztracer and memray on demand
d5f4ae1
@AlyaGomaa
move the logic of stopping the profilers to process_manager's shutdow…
2bb024c 
…n_gracefully()
@AlyaGomaa
remove vistrace, memray, tqdm, setuptools and wheel from requirements…
e8f2a47 
….txt
@AlyaGomaa
update profiler unit tests
27dada8
@AlyaGomaa
Merge remote-tracking branch 'origin/alya/use-on-demand-imports' into…
f7bf2fb 
… alya/use-on-demand-imports
@AlyaGomaa
CI: integration tests: attempt to re-install zeek to be able to resto…
6971edd 
…re it from cache
AlyaGomaa and others added 28 commits December 18, 2024 13:32
@AlyaGomaa
Merge pull request stratosphereips#1127 from stratosphereips/alya/fix…
ac13666 
…_multiple_reconnection_attempts

Fix problem setting multiple reconnection attempts evidence
@AlyaGomaa
ip_info: get domain age and registrant using whois
8da18d4
@AlyaGomaa
ssl.py: detect CN and hostname mismatch
a439e10
@AlyaGomaa
set_evidence:set 2 evidence on CN and hostname mismatch
965e203
@AlyaGomaa
ip_info: get the registrant of the sld in the registrant of the main …
c0f4c7b 
…domain isnt found
@AlyaGomaa
set_evidence: better description of CN mismatch evidence
adc2c0c
@AlyaGomaa
ssl: better matching of the 2 domains orgs when checking for CN mismatch
5a82467
@AlyaGomaa
slips_utils: remove dead code
7b9b953
@AlyaGomaa
ssl: in domains_belong_to_same_org() return the common org if found
1321b5b
@AlyaGomaa
update ip info unit tests
828b5bd
@AlyaGomaa
add unit tests for CN mismatch evidence
1e0063e
@AlyaGomaa
update the docs about CN mismatch evidence
6412dab
@AlyaGomaa
update ip info unit tests
31a2c1e
@AlyaGomaa
delete redis_tests.py
11d02b9
@AlyaGomaa
Merge pull request stratosphereips#1129 from stratosphereips/alya/det…
b436cc9 
…ect_SSL_Certificate_Name_Hostname_Mismatch

Detect ssl certificate hostname mismatch
@AlyaGomaa
Merge remote-tracking branch 'origin/develop' into develop
9cc18ad
@AlyaGomaa
evidence: remove extra \n at the end of some evidence
5c54770
@dependabot
build(deps): bump psutil from 6.0.0 to 6.1.1 in /install
99e77e5 
Bumps [psutil](https://github.com/giampaolo/psutil) from 6.0.0 to 6.1.1.
- [Changelog](https://github.com/giampaolo/psutil/blob/master/HISTORY.rst)
- [Commits](giampaolo/psutil@release-6.0.0...release-6.1.1)

---
updated-dependencies:
- dependency-name: psutil
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
build(deps): bump ruff from 0.6.8 to 0.8.4 in /install
1d69cc1 
Bumps [ruff](https://github.com/astral-sh/ruff) from 0.6.8 to 0.8.4.
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@0.6.8...0.8.4)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@AlyaGomaa
Merge pull request stratosphereips#1130 from stratosphereips/dependab…
0122ad1 
…ot/pip/install/develop/psutil-6.1.1

build(deps): bump psutil from 6.0.0 to 6.1.1 in /install
@AlyaGomaa
Merge pull request stratosphereips#1131 from stratosphereips/dependab…
1904e62 
…ot/pip/install/develop/ruff-0.8.4

build(deps): bump ruff from 0.6.8 to 0.8.4 in /install
@Sekhar-Kumar-Dash @AlyaGomaa
Created unittest for profile_handler.py
6e64889
@Sekhar-Kumar-Dash @AlyaGomaa
Updated module_factory.py
7c7fd13
@AlyaGomaa
Merge remote-tracking branch 'Sekhar-Kumar-Dash/patch-59' into fork/S…
f9940d4 
…ekhar-Kumar-Dash/patch-59

# Conflicts:
#	.github/workflows/CI-production-testing.yml
#	.github/workflows/CI-staging.yml
#	tests/module_factory.py
@AlyaGomaa
update test_profile_handler.py with the current codebase
ea1e90b
@AlyaGomaa
Add profile handler tests to CI
885d3d5
@AlyaGomaa
test_profile_handler.py: improve tests and add comments
c20bd76
@AlyaGomaa
dbmanager: refactor
681554f
@AlyaGomaa AlyaGomaa merged commit d4f221e into stratosphereips:develop Jan 3, 2025
56 checks passed
@AlyaGomaa
Copy link
Collaborator

hey @Sekhar-Kumar-Dash thanks for your effort!

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
None yet
Projects
Slips
Status: Done
Milestone
No milestone
Development

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