Skip to content

[tagged_arp] fix issue "fixture 'ports_list' not found"#6773

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
AntonHryshchuk:fixture_ports_list
Nov 9, 2022
Merged

[tagged_arp] fix issue "fixture 'ports_list' not found"#6773
liat-grozovik merged 1 commit intosonic-net:masterfrom
AntonHryshchuk:fixture_ports_list

Conversation

@AntonHryshchuk
Copy link
Contributor

@AntonHryshchuk AntonHryshchuk commented Nov 8, 2022

Signed-off-by: Anton antonh@nvidia.com

Description of PR

Summary:
Fixed test issue:

  def test_tagged_arp_pkt(ptfadapter, utils_vlan_ports_list, duthosts, rand_one_dut_hostname):    # noqa F811
file /root/mars/workspace/sonic-mgmt/tests/common/fixtures/duthost_utils.py, line 243
  @pytest.fixture(scope="module")
  def utils_vlan_ports_list(duthosts, rand_one_dut_hostname, rand_selected_dut, tbinfo, ports_list):
E       fixture 'ports_list' not found

Fixes # (issue)

Type of change

  • [] Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

Stabilize the test

How did you do it?

Added back the removed import

How did you verify/test it?

Test executed and passed

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Signed-off-by: Anton <antonh@nvidia.com>
@liat-grozovik liat-grozovik merged commit ebeadcd into sonic-net:master Nov 9, 2022
@AntonHryshchuk AntonHryshchuk deleted the fixture_ports_list branch November 16, 2022 08:42
wangxin pushed a commit that referenced this pull request Nov 23, 2022
- What is the motivation for this PR?
Stabilize the test

- How did you do it?
Added back the removed import

- How did you verify/test it?
Test executed and passed

Signed-off-by: Anton <antonh@nvidia.com>
bingwang-ms pushed a commit to bingwang-ms/sonic-mgmt that referenced this pull request Jul 27, 2023
…ic-mgmt into internal-202205

Fix merge conflicts.

- Fix verify_no_packet_any call in fib_test (sonic-net#6461)
- Fix the test case test_TSA failure when check the routes on the eos host (sonic-net#6483)
- Use conditional mark to skip testcase instead of required_mocked_dualtor (sonic-net#6766)
- [tagged_arp] fix issue 'fixture ports_list not found' (sonic-net#6773)
- [QoS] fixes after moving to python3 (sonic-net#6786)
- update parse funciton for image url (sonic-net#6848)
- Fix typo in get_queue_counter (sonic-net#6852)
- Revert "Fix loganalyzer.py UnicodeDecodeError (sonic-net#6524)" (sonic-net#6858)
- Enhancing core_dump_and_config_check to be multi-asic aware (sonic-net#6527)
- Adding support for calculating balancing in multi-lc/multi-asic case (Test_fib.py) (sonic-net#6391)
- Support different RC in case of pre or post sanity check failed (sonic-net#6860)
- Update getbuild.py to support pass an empty access_token
- [202205] Fixing auto_techsupport (sonic-net#6882)
- Merge branch 'azure-202205' into dev/yaqiangzhu/202205_manually_merge
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.

5 participants