Skip to content

[platform][pytest] Create 'cli' subdirectory to house platform-related CLI tests#1964

Merged
jleveque merged 4 commits intosonic-net:masterfrom
jleveque:add_platform_cli_tests
Jul 29, 2020
Merged

[platform][pytest] Create 'cli' subdirectory to house platform-related CLI tests#1964
jleveque merged 4 commits intosonic-net:masterfrom
jleveque:add_platform_cli_tests

Conversation

@jleveque
Copy link
Contributor

  • Create cli/ subdirectory under tests/platform_tests/
  • Add test_show_platform.py file containing tests for the show platform ... commands
    • Move show platform ... tests from existing files into this file (removing test_syseeprom.py entirely in the process)
    • Add some new tests for:
      • show platform ssdhealth
      • show platform firmware status
  • Add a util.py module containing shared utility functions for working with the CLI

@lgtm-com
Copy link

lgtm-com bot commented Jul 24, 2020

This pull request fixes 1 alert when merging 58fa064 into 7ced602 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@jleveque jleveque merged commit a739355 into sonic-net:master Jul 29, 2020
@jleveque jleveque deleted the add_platform_cli_tests branch July 29, 2020 18:36
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
4236bc4 [config reload] Fixing config reload when timer based delayed services are disabled (sonic-net#1967)
d2514e4 [GCU] Different apply-patch runs should produce same sorted steps (sonic-net#1988)
2878adb [GCU] Using simulated config instead of target config when validating replace operation in NoDependencyMoveValidator (sonic-net#1987)
fb8ca98 [GCU] Loading yang-models only once (sonic-net#1981)
f88ee92 [GCU] Copying config_db before callding sonic_yang.loadData (sonic-net#1983)
9ed0e91 [GCU] Implementing DryRun by printing patch-sorter steps/imitating config_db (sonic-net#1973)
b36b5e3 [GCU] Moving PatchSorter unit-test to json file to make it easier to read/maintain (sonic-net#1977)
c0fa28b [generic-config-updater] Improving CreateOnly validator and marking /LOOPBACK_INTERFACE/LOOPBACK#/vrf_name as create-only (sonic-net#1969)
0559d04 [generic-config-updater] Adding non-strict mode (sonic-net#1929)
b07f477 [debug dump util] FDB debug dump util changes (sonic-net#1968)
6d8757a [warm/fast-reboot] Fix kexec portion to support platforms based on Device Tree (sonic-net#1966)
cc1409e [Auto Techsupport] Event driven Techsupport Bug Fixes (sonic-net#1986)
6c48bd5 Fix wrong help message for cable length setting (sonic-net#1978)
c0bbbe3 [breakout] Fix the check  when port is not present in BREAKOUT_CFG table (sonic-net#1765)
5bb8cad [doc][DPB] Update DPB related interface breakout command Info (sonic-net#1438)
e6fd990 [config] Fix 'config reload -l' command to get filename by default (sonic-net#1611)
bd8f7bb Update swss_ready check to check per namespace swss service (sonic-net#1974)
5439f94 [soft-reboot] Add support for platforms based on Device Tree (sonic-net#1963)
7c5810a [config] Add portchannel support  for static route  (sonic-net#1857)
7cb6a1b preserve old order for config reload (sonic-net#1964)
20bddbd [Auto-Techsupport] Issues related to Multiple Cores crashing handled (sonic-net#1948)
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.

2 participants