Skip to content

Improved testing to facilitate updated Iris#1363

Merged
AlyaGomaa merged 22 commits intostratosphereips:developfrom
d-strat:develop
Apr 10, 2025
Merged

Improved testing to facilitate updated Iris#1363
AlyaGomaa merged 22 commits intostratosphereips:developfrom
d-strat:develop

Conversation

@d-strat
Copy link
Copy Markdown
Contributor

@d-strat d-strat commented Mar 20, 2025

Changes proposed

Update configuration modifier in Iris Module to include IP.
Update iris_test.py to facilitate testing of soon-to-be-pushed updated Iris.
Add a bootstrapping mode.

Steps you followed to test the changes purposed in this PR:

  • running the iris_test.py multiple times with the new Iris
  • altering the integration test

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.
  • My PR is based on develop branch. (mandatory)

Screenshots

iris_test_slips_pr

Note to reviewers

The test was created to be compatible with current and previous Iris, updated Iris will follow after this PR.

Comment thread slips_files/common/parsers/config_parser.py
Comment thread config/slips.yaml Outdated
@AlyaGomaa
Copy link
Copy Markdown
Collaborator

AlyaGomaa commented Apr 8, 2025

@d-strat Also one more thing, it would be nice if we either remove this "Disabled modules" line from the text printed to cli in bootstrapping mode only. or update it with the actual disabled modules, because this takes values from slips.yaml "disable" parameter, but in the bootstrapping mode, we're disabling everything, not just from the disable param.

Screenshot_20250408_183308

@d-strat d-strat requested a review from AlyaGomaa April 9, 2025 13:57
@AlyaGomaa AlyaGomaa merged commit 79656b2 into stratosphereips:develop Apr 10, 2025
71 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Slips Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants