Added alias for mode option as switchport_mode for nxos_l2_interfaces (#1009)

* Added alias for switchport_mode

* chore: auto fixes from pre-commit.com hooks

* Added changelog and documentation

* Added changelog and documentation

* chore: auto fixes from pre-commit.com hooks

* Make integration test for alias non voting

* Update vars

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sagar Paul <sagpaul@redhat.com>
Co-authored-by: KB-perByte <paul.sagar@yahoo.com>

Author: 3 people

changelogs/fragments/l2_inerfaces_mode_alias.yaml

@@ -0,0 +1,3 @@
+---
+minor_changes:
+  - Added alias for mode option as switchport_mode for nxos_l2_interfaces

docs/cisco.nxos.nxos_l2_interfaces_module.rst

@@ -216,6 +216,7 @@ Parameters
                 <td>
                         <div>Mode in which interface needs to be configured.</div>
                         <div>Access mode is not shown in interface facts, so idempotency will not be maintained for switchport mode access and every time the output will come as changed=True.</div>
+                        <div style="font-size: small; color: darkgreen"><br/>aliases: switchport_mode</div>
                 </td>
             </tr>
             <tr>

plugins/module_utils/network/nxos/argspec/l2_interfaces/l2_interfaces.py

@@ -55,6 +55,7 @@ class L2_interfacesArgs(object):  # pylint: disable=R0903
                         "fex-fabric",
                         "fabricpath",
                     ],
+                    "aliases": ["switchport_mode"],
                 },
                 "cdp_enable": {"type": "bool"},
                 "link_flap": {

plugins/modules/nxos_l2_interfaces.py

@@ -72,6 +72,8 @@
         - trunk
         - fex-fabric
         - fabricpath
+        aliases:
+          - switchport_mode
       cdp_enable:
         type: bool
         description: Enable/disable CDP on the interface

tests/integration/targets/nxos_l2_interfaces/tests/common/merged.yaml

@@ -95,6 +95,26 @@
         that:
           - result.changed == false
           - result.commands|length == 0
+
+    - name: Using switchport_mode alias with dot1q-tunnel (alias running with check_mode)
+      register: result_check
+      check_mode: true
+      cisco.nxos.nxos_l2_interfaces:
+        config:
+          - name: "{{ nxos_int2 }}"
+            switchport_mode: dot1q-tunnel
+            access:
+              vlan: 30
+        state: merged
+
+    - ansible.builtin.assert:
+        that:
+          - result_check.changed == true
+          - "'interface {{ nxos_int2 }}' in result_check.commands"
+          - "'switchport mode dot1q-tunnel' in result_check.commands"
+          - "'switchport access vlan 30' in result_check.commands"
+          - result_check.commands|length == 3
+
   always:
     - name: Teardown
       ignore_errors: true