Minimize cross-module dependencies in test_vnet_vxlan.py
.
#14675
+302
−285
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of PR
In the
test_vnet_vxlan.py
script, theTestWrArp
class is imported fromarp/test_wr_arp.py
. To minimize cross-module dependencies, we have refactored this class and moved the shared functions to a common location.Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
In the
test_vnet_vxlan.py
script, theTestWrArp
class is imported fromarp/test_wr_arp.py
. To minimize cross-module dependencies, we have refactored this class and moved the shared functions to a common location.How did you do it?
How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation