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

[vlan] Add pytest cases to validate the behavior about add LAG member to vlan. #875

Merged
merged 5 commits into from
Aug 8, 2019

Conversation

linemma
Copy link
Contributor

@linemma linemma commented May 3, 2019

What I did
[vlan] Add pytest cases to validate the behavior about add LAG member to vlan.

Why I did it

How I verified it
Add test case:

  1. Add LAG member to vlan and remove it. => succeed

Details if related
==================================================================== test session starts =====================================================================
platform linux2 -- Python 2.7.15rc1, pytest-3.3.0, py-1.8.0, pluggy-0.6.0 -- /usr/bin/python
cachedir: .cache
rootdir: /home/ubuntu/sonic-buildimage/src/sonic-swss/tests, inifile:
collected 11 items

test_vlan.py::TestVlan::test_VlanAddRemove PASSED [ 9%]
test_vlan.py::TestVlan::test_MultipleVlan PASSED [ 18%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectKeyPrefix[test_input0-0] PASSED [ 27%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectKeyPrefix[test_input1-0] PASSED [ 36%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectKeyPrefix[test_input2-0] PASSED [ 45%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectKeyPrefix[test_input3-1] PASSED [ 54%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectValueType[test_input0-0] PASSED [ 63%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectValueType[test_input1-0] PASSED [ 72%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectValueType[test_input2-0] PASSED [ 81%]
test_vlan.py::TestVlan::test_AddVlanWithIncorrectValueType[test_input3-1] PASSED [ 90%]
test_vlan.py::TestVlan::test_AddPortChannelToVlan PASSED [100%]

================================================================= 11 passed in 62.15 seconds =================================================================

Signed-off-by: Emma Lin [email protected]

@linemma
Copy link
Contributor Author

linemma commented May 10, 2019

retest this please

@lguohan
Copy link
Contributor

lguohan commented Jun 19, 2019

@linemma , can you resolve the conflict?

@linemma
Copy link
Contributor Author

linemma commented Jun 20, 2019

@lguohan, resolved, please review again. Thanks.

@linemma
Copy link
Contributor Author

linemma commented Jun 28, 2019

retest this please

@lguohan
Copy link
Contributor

lguohan commented Aug 1, 2019

need to resolve the conflict again due to previous vlan test contribution from you.

@linemma
Copy link
Contributor Author

linemma commented Aug 2, 2019

@lguohan, resolved, please review again. Thanks!

@lguohan
Copy link
Contributor

lguohan commented Aug 2, 2019

retest this please

3 similar comments
@lguohan
Copy link
Contributor

lguohan commented Aug 3, 2019

retest this please

@lguohan
Copy link
Contributor

lguohan commented Aug 3, 2019

retest this please

@linemma
Copy link
Contributor Author

linemma commented Aug 5, 2019

retest this please

@stcheng
Copy link
Contributor

stcheng commented Aug 5, 2019

retest this please

1 similar comment
@linemma
Copy link
Contributor Author

linemma commented Aug 6, 2019

retest this please

@lguohan lguohan merged commit bb0bcf8 into sonic-net:master Aug 8, 2019
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
…the MACsec port was disabled in runtime (sonic-net#875)

* Add MACsec filter state guard

Signed-off-by: Ze Gan <[email protected]>

* Add volatile qualifer for state

Signed-off-by: Ze Gan <[email protected]>

* Polish code format

Signed-off-by: Ze Gan <[email protected]>

* Fix the macsec device was shutdown by ouutside

Signed-off-by: Ze Gan <[email protected]>

* fix typo

Signed-off-by: Ze Gan <[email protected]>

* Remove overkill qualifier

Signed-off-by: Ze Gan <[email protected]>

* fix typo

Signed-off-by: Ze Gan <[email protected]>

* Fix spell warning

Signed-off-by: Ze Gan <[email protected]>

* Fix bug

Signed-off-by: Ze Gan <[email protected]>

* Format code

Signed-off-by: Ze Gan <[email protected]>

* Fix unittest

Signed-off-by: Ze Gan <[email protected]>

* Add unittest for State guard

Signed-off-by: Ze Gan <[email protected]>

* Update Makefile.am for testing MACsec Filter State Guard

Signed-off-by: Ze Gan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants