Added cisco config platform commands

[yucgu]

Signed-off-by: Yucai Gu yucgu@cisco.com

What I did

Add cisco sub-command option under 'config platform' command

How I did it

In config/main.py, check the platform type and import the cisco.py file under cisco platform code when it's cisco-8000.

How to verify it

Run config platform -h to see all commands. We will be able to see config platform cisco. This is only available on cisco devices.

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

[kevinskwang]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: yucgu (d87dfa1, 6995cba, bafc9e3, fadfbe6, c86e6ce, 6d15d15, 0c24820, ef9eb7c, f68795f, eb7f571, dd7b9bc, dc0889d, c9aafca, c9dbf5f, f286424, 0281112, 83aa7d2, 8d83faa, 70d86d3, 175ed3e, 8e171c7, 9a8d9e3, 53d7792, abf4c47, 0b26371, 2b66c55, 20bcc7e, 2074c35, 5b281c8, f6db8bb, 66bb737, 0021429, 54690d5, b8d5c6a, e32c64e, 2867c06, dae423f, f1de6ea, 074e0eb, d7d89cd, 3dcb1cf, 75c3775, 3bb1b7b, ba15f25, 02244bd, 69bce77, 5f17de2)

[lgtm-com]

This pull request introduces 1 alert when merging 9a8d9e3 into 14c483f - view on LGTM.com

new alerts:

• 1 for Unreachable code

[yxieca]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[yxieca]

@yucgu can you add a unit test to cover this simple change?

[yucgu]

@yucgu can you add a unit test to cover this simple change?
Hi Ying,
thanks for looking at it. I tried various changes, includes adding the test cases or calling the function directly, but still not able to make the UT coverage pass.
Probably the UT infra doesn't support this kind of coverage by now, there are two reasons:
1) the parent function where I added those 3 lines of new codes is a main entry point of a command chain, it's not called/covered, I checked other main entry points of other command chains, all are not called/covered, looks like the UT infra doesn't support that coverage.
2) even there is some way to make it called, and we use mock to make asic_type as cisco-8000, the code will still not be tested because the cisco.py is in the platform code repo, the UT of sonic-utilities doesn't have that file.