mbox series

[V2,0/4] update existing cases and add new sub-cases to validate port representor action and represented port action

Message ID 20221221101252.59230-1-ke1.xu@intel.com (mailing list archive)
Headers
Series update existing cases and add new sub-cases to validate port representor action and represented port action |

Message

Ke Xu Dec. 21, 2022, 10:12 a.m. UTC
  Deprecated TO_VF action contains a 'vf original' action which is not covered in
 previous tests. This patch applys equivilant 'vf original' rule to previous
 cases for 'vf id' action to cover 'vf original' action.

This patch adds the following subcases to cover the negative scenario of 'port
 representor' action and 'represented port' action replacing 'vf original' action
 and 'vf id' action.

1. Add second PF with VFs for following cases
2. Add a subcase to validate creating 'represented port' action on a representor
3. Add a subcase to validate creating 'represented port' action on a VF
4. Add a subcase to validate creating 'represented port' action to a VF
5. Add two subcases to validate creating 'represented port' action to DCFs
6. Add a subcase to validate creating 'represented port' action to a representor of another DCF
7. Add a subcase to validate creating 'port representor' action on a representor
8. Add a subcase to validate creating 'port representor' action on a VF
9. Add a subcase to validate creating 'port representor' action to a VF
10. Add a subcase to validate creating 'port representor' action to a representor
11. Add a subcase to validate creating 'port representor' action to another DCF
12. Add a subcase to validate deprecated action
13. Add a subcase to validate using an invalid port id

Ke Xu (4):
  test_plans/ice_dcf_switch_filter: update existing negative cases to
    validate represented port action and port representor action
  test_plans/ice_dcf_switch_filter: update existing TO_VF action cases
    to cover port representor action from not covered vf original action
  tests/ice_dcf_switch_filter: update existing TO_VF action cases to
    cover port representor action from not covered vf original action
  tests/ice_dcf_switch_filter: update existing negative cases to
    validate represented port action and port representor action

 .../ice_dcf_switch_filter_test_plan.rst       | 921 ++++++++++++++----
 tests/TestSuite_ice_dcf_switch_filter.py      | 692 ++++++++-----
 2 files changed, 1209 insertions(+), 404 deletions(-)
  

Comments

Ke Xu Dec. 21, 2022, 10:38 a.m. UTC | #1
> -----Original Message-----
> From: Xu, Ke1 <ke1.xu@intel.com>
> Sent: Wednesday, December 21, 2022 6:13 PM
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>; Peng, Yuan <yuan.peng@intel.com>; Fu,
> Qi <qi.fu@intel.com>; Xu, Ke1 <ke1.xu@intel.com>
> Subject: [DTS][PATCH V2 0/4] update existing cases and add new sub-cases
> to validate port representor action and represented port action
> 
> Deprecated TO_VF action contains a 'vf original' action which is not covered
> in  previous tests. This patch applys equivilant 'vf original' rule to previous
> cases for 'vf id' action to cover 'vf original' action.
> 
> This patch adds the following subcases to cover the negative scenario of 'port
> representor' action and 'represented port' action replacing 'vf original' action
> and 'vf id' action.
> 
> 1. Add second PF with VFs for following cases 2. Add a subcase to validate
> creating 'represented port' action on a representor 3. Add a subcase to
> validate creating 'represented port' action on a VF 4. Add a subcase to
> validate creating 'represented port' action to a VF 5. Add two subcases to
> validate creating 'represented port' action to DCFs 6. Add a subcase to
> validate creating 'represented port' action to a representor of another DCF 7.
> Add a subcase to validate creating 'port representor' action on a representor
> 8. Add a subcase to validate creating 'port representor' action on a VF 9. Add
> a subcase to validate creating 'port representor' action to a VF 10. Add a
> subcase to validate creating 'port representor' action to a representor 11.
> Add a subcase to validate creating 'port representor' action to another DCF
> 12. Add a subcase to validate deprecated action 13. Add a subcase to validate
> using an invalid port id

No different modification from V1 patch. Only commit messages changed.

According to V1 patch record:
https://inbox.dpdk.org/dts/PH8PR11MB71202D1CCB1BD539F504DF849FE09@PH8PR11MB7120.namprd11.prod.outlook.com/

Tested-by: Hongbo Li < hongbox.li@intel.com>

> 
> Ke Xu (4):
>   test_plans/ice_dcf_switch_filter: update existing negative cases to
>     validate represented port action and port representor action
>   test_plans/ice_dcf_switch_filter: update existing TO_VF action cases
>     to cover port representor action from not covered vf original action
>   tests/ice_dcf_switch_filter: update existing TO_VF action cases to
>     cover port representor action from not covered vf original action
>   tests/ice_dcf_switch_filter: update existing negative cases to
>     validate represented port action and port representor action
> 
>  .../ice_dcf_switch_filter_test_plan.rst       | 921 ++++++++++++++----
>  tests/TestSuite_ice_dcf_switch_filter.py      | 692 ++++++++-----
>  2 files changed, 1209 insertions(+), 404 deletions(-)
> 
> --
> 2.25.1
  
Tu, Lijuan Dec. 22, 2022, 7:45 a.m. UTC | #2
> -----Original Message-----
> From: Xu, Ke1 <ke1.xu@intel.com>
> Sent: Wednesday, December 21, 2022 6:13 PM
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>; Peng, Yuan <yuan.peng@intel.com>;
> Fu, Qi <qi.fu@intel.com>; Xu, Ke1 <ke1.xu@intel.com>
> Subject: [DTS][PATCH V2 0/4] update existing cases and add new sub-cases
> to validate port representor action and represented port action
> 
> Deprecated TO_VF action contains a 'vf original' action which is not covered
> in  previous tests. This patch applys equivilant 'vf original' rule to previous
> cases for 'vf id' action to cover 'vf original' action.
> 
> This patch adds the following subcases to cover the negative scenario of
> 'port  representor' action and 'represented port' action replacing 'vf
> original' action  and 'vf id' action.
> 
> 1. Add second PF with VFs for following cases 2. Add a subcase to validate
> creating 'represented port' action on a representor 3. Add a subcase to
> validate creating 'represented port' action on a VF 4. Add a subcase to
> validate creating 'represented port' action to a VF 5. Add two subcases to
> validate creating 'represented port' action to DCFs 6. Add a subcase to
> validate creating 'represented port' action to a representor of another DCF
> 7. Add a subcase to validate creating 'port representor' action on a
> representor 8. Add a subcase to validate creating 'port representor' action
> on a VF 9. Add a subcase to validate creating 'port representor' action to a
> VF 10. Add a subcase to validate creating 'port representor' action to a
> representor 11. Add a subcase to validate creating 'port representor' action
> to another DCF 12. Add a subcase to validate deprecated action 13. Add a
> subcase to validate using an invalid port id
> 
> Ke Xu (4):
>   test_plans/ice_dcf_switch_filter: update existing negative cases to
>     validate represented port action and port representor action
>   test_plans/ice_dcf_switch_filter: update existing TO_VF action cases
>     to cover port representor action from not covered vf original action
>   tests/ice_dcf_switch_filter: update existing TO_VF action cases to
>     cover port representor action from not covered vf original action
>   tests/ice_dcf_switch_filter: update existing negative cases to
>     validate represented port action and port representor action

Applied, thanks