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
Add connected devices traffgen support to testing framework #2754
Conversation
402b3c7
to
8a5a969
Compare
819409e
to
d44de56
Compare
15db525
to
e999e81
Compare
e999e81
to
33189ef
Compare
ef058b7
to
385d3d9
Compare
Note the target branch. Merge before merging |
0d5d761
to
070c234
Compare
...ests/src/test/groovy/org/openkilda/functionaltests/spec/flows/FlowConnectedDeviceSpec.groovy
Outdated
Show resolved
Hide resolved
...ests/src/test/groovy/org/openkilda/functionaltests/spec/flows/FlowConnectedDeviceSpec.groovy
Show resolved
Hide resolved
...ests/src/test/groovy/org/openkilda/functionaltests/spec/flows/FlowConnectedDeviceSpec.groovy
Outdated
Show resolved
Hide resolved
...ests/src/test/groovy/org/openkilda/functionaltests/spec/flows/FlowConnectedDeviceSpec.groovy
Show resolved
Hide resolved
...ests/src/test/groovy/org/openkilda/functionaltests/spec/switches/SwitchValidationSpec.groovy
Outdated
Show resolved
Hide resolved
...ests/src/test/groovy/org/openkilda/functionaltests/spec/switches/SwitchValidationSpec.groovy
Outdated
Show resolved
Hide resolved
...s/src/test-library/src/main/java/org/openkilda/testing/service/traffexam/model/LldpData.java
Show resolved
Hide resolved
070c234
to
4da8265
Compare
6cd5bba
to
ba7bd7f
Compare
4da8265
to
f29db12
Compare
f29db12
to
83095bc
Compare
1ac3236
to
ed9ce39
Compare
ed9ce39
to
e47081d
Compare
involvedSwitches[1..-1].each { switchId -> | ||
northbound.deleteSwitchRules(switchId, DeleteRulesAction.IGNORE_DEFAULTS) | ||
when: "Synchronize the switch" | ||
verifyAll(northbound.synchronizeSwitch(switchPair.src.dpId, false)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
northbound.synchronizeSwitch(switchPair.src.dpId, false).rules.installed == [egressCookie]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done both
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a mistake. with
and verifyAll
are forcing the verification in 'when' block. So it was correct. Your option is missing an assert
keyword
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All related comments are reverted to this style:
when: "Synchronize the switch"
with(northbound.synchronizeSwitch(switchPair.src.dpId, false)) {
rules.installed == [egressCookie]
}
...ests/src/test/groovy/org/openkilda/functionaltests/spec/switches/SwitchValidationSpec.groovy
Outdated
Show resolved
Hide resolved
8dc5687
to
df8ff73
Compare
-Also major refactoring of all switch validation tests
df8ff73
to
96f0b5c
Compare
No description provided.