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.

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

Refactor storm_control tests #321

Merged

Conversation

stavrukPLV
Copy link
Contributor

✓ Refactored code and test steps by using the Python style guidelines laid out in PEP 8
✓ Add function verify_expected_rx_rate to storm_control_utils.py
✓ Removing extra spaces
✓ Change traffic_duration

Signed-off-by: Kostiantyn Stavruk <kostiantyn.stavruk@plvision.eu>
@yuriilisovskyi yuriilisovskyi added the phase2 Tests from Phase 2 label May 4, 2023
stavrukPLV and others added 2 commits May 5, 2023 09:57
Signed-off-by: Kostiantyn Stavruk <119867993+stavrukPLV@users.noreply.github.com>
Signed-off-by: Kostiantyn Stavruk <kostiantyn.stavruk@plvision.eu>
@stavrukPLV stavrukPLV force-pushed the functional_refactor_storm_control branch from 43e04f7 to 31370ba Compare May 5, 2023 10:35
@mgheorghe mgheorghe merged commit c2ffdcd into dentproject:main May 8, 2023
@stavrukPLV stavrukPLV deleted the functional_refactor_storm_control branch May 25, 2023 21:26
@taraschornyiplv taraschornyiplv linked an issue May 30, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
phase2 Tests from Phase 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Storm Control tests
4 participants