add module_type aswell as tests

David Vølker · David Vølker · commit 5c72ceadb5b8 · 2026-01-20T15:35:53.000+01:00
diff --git a/changelogs/fragments/module_type_template.yml b/changelogs/fragments/module_type_template.yml
@@ -0,0 +1,4 @@
+---
+minor_changes:
+  - Add `module_type` as supported to `netbox_rear_port_template`
+  - Add `module_type` as supported to `netbox_front_port_template`
diff --git a/plugins/inventory/nb_inventory.py b/plugins/inventory/nb_inventory.py
@@ -408,7 +408,6 @@
 from ansible.module_utils.six.moves.urllib.parse import urlencode
 from ansible.module_utils.six.moves.urllib.parse import urlparse
 
-
 try:
     from packaging import specifiers, version
 except ImportError as imp_exc:
diff --git a/plugins/module_utils/netbox_circuits.py b/plugins/module_utils/netbox_circuits.py
@@ -11,7 +11,6 @@
     SLUG_REQUIRED,
 )
 
-
 NB_PROVIDERS = "providers"
 NB_PROVIDER_NETWORKS = "provider_networks"
 NB_CIRCUIT_TYPES = "circuit_types"
diff --git a/plugins/module_utils/netbox_dcim.py b/plugins/module_utils/netbox_dcim.py
@@ -12,7 +12,6 @@
     SLUG_REQUIRED,
 )
 
-
 NB_CABLES = "cables"
 NB_CONSOLE_PORTS = "console_ports"
 NB_CONSOLE_PORT_TEMPLATES = "console_port_templates"
diff --git a/plugins/module_utils/netbox_ipam.py b/plugins/module_utils/netbox_ipam.py
@@ -15,7 +15,6 @@
     SLUG_REQUIRED,
 )
 
-
 NB_AGGREGATES = "aggregates"
 NB_ASNS = "asns"
 NB_FHRP_GROUPS = "fhrp_groups"
diff --git a/plugins/module_utils/netbox_tenancy.py b/plugins/module_utils/netbox_tenancy.py
@@ -11,7 +11,6 @@
     SLUG_REQUIRED,
 )
 
-
 NB_TENANTS = "tenants"
 NB_TENANT_GROUPS = "tenant_groups"
 NB_CONTACTS = "contacts"
diff --git a/plugins/module_utils/netbox_virtualization.py b/plugins/module_utils/netbox_virtualization.py
@@ -11,7 +11,6 @@
     SLUG_REQUIRED,
 )
 
-
 NB_VIRTUAL_MACHINES = "virtual_machines"
 NB_CLUSTERS = "clusters"
 NB_CLUSTER_GROUP = "cluster_groups"
diff --git a/plugins/module_utils/netbox_vpn.py b/plugins/module_utils/netbox_vpn.py
@@ -14,7 +14,6 @@
     SLUG_REQUIRED,
 )
 
-
 NB_L2VPNS = "l2vpns"
 NB_L2VPN_TERMINATIONS = "l2vpn_terminations"
 NB_TUNNELS = "tunnels"
diff --git a/plugins/module_utils/netbox_wireless.py b/plugins/module_utils/netbox_wireless.py
@@ -11,7 +11,6 @@
     SLUG_REQUIRED,
 )
 
-
 NB_WIRELESS_LANS = "wireless_lans"
 NB_WIRELESS_LAN_GROUPS = "wireless_lan_groups"
 NB_WIRELESS_LINKS = "wireless_links"
diff --git a/plugins/modules/netbox_front_port_template.py b/plugins/modules/netbox_front_port_template.py
@@ -34,7 +34,12 @@
       device_type:
         description:
           - The device type the front port template is attached to
-        required: true
+          - Either I(device_type) or I(module_type) are required
+        type: raw
+      module_type:
+        description:
+          - The module type the front port template is attached to
+          - Either I(device_type) or I(module_type) are required
         type: raw
       name:
         description:
@@ -103,6 +108,17 @@
           rear_port_template: Test Rear Port Template
         state: present
 
+    - name: Create front port template for a module type within NetBox
+      netbox.netbox.netbox_front_port_template:
+        netbox_url: http://netbox.local
+        netbox_token: thisIsMyToken
+        data:
+          name: Test Front Port Template
+          module_type: Test Module Type
+          type: bnc
+          rear_port_template: Test Rear Port Template
+        state: present
+
     - name: Update front port template with other fields
       netbox.netbox.netbox_front_port_template:
         netbox_url: http://netbox.local
@@ -160,7 +176,8 @@ def main():
                 type="dict",
                 required=True,
                 options=dict(
-                    device_type=dict(required=True, type="raw"),
+                    device_type=dict(required=False, type="raw"),
+                    module_type=dict(required=False, type="raw"),
                     name=dict(required=True, type="str"),
                     type=dict(
                         required=False,
@@ -192,12 +209,19 @@ def main():
     )
 
     required_if = [
-        ("state", "present", ["device_type", "name", "type", "rear_port_template"]),
-        ("state", "absent", ["device_type", "name", "type", "rear_port_template"]),
+        ("state", "present", ["name", "type", "rear_port_template"]),
+        ("state", "absent", ["name", "type", "rear_port_template"]),
+    ]
+
+    required_one_of = [
+        ("device_type", "module_type"),
     ]
 
     module = NetboxAnsibleModule(
-        argument_spec=argument_spec, supports_check_mode=True, required_if=required_if
+        argument_spec=argument_spec,
+        supports_check_mode=True,
+        required_if=required_if,
+        required_one_of=required_one_of,
     )
 
     netbox_front_port_template = NetboxDcimModule(module, NB_FRONT_PORT_TEMPLATES)
diff --git a/plugins/modules/netbox_rear_port_template.py b/plugins/modules/netbox_rear_port_template.py
@@ -34,7 +34,12 @@
       device_type:
         description:
           - The device type the rear port template is attached to
-        required: true
+          - Either I(device_type) or I(module_type) are required
+        type: raw
+      module_type:
+        description:
+          - The module type the rear port template is attached to
+          - Either I(device_type) or I(module_type) are required
         type: raw
       name:
         description:
@@ -97,6 +102,16 @@
           type: bnc
         state: present
 
+    - name: Create rear port template for a module type within NetBox
+      netbox.netbox.netbox_rear_port_template:
+        netbox_url: http://netbox.local
+        netbox_token: thisIsMyToken
+        data:
+          name: Test Rear Port Template
+          module_type: Test Module Type
+          type: bnc
+        state: present
+
     - name: Update rear port template with other fields
       netbox.netbox.netbox_rear_port_template:
         netbox_url: http://netbox.local
@@ -152,7 +167,8 @@ def main():
                 type="dict",
                 required=True,
                 options=dict(
-                    device_type=dict(required=True, type="raw"),
+                    device_type=dict(required=False, type="raw"),
+                    module_type=dict(required=False, type="raw"),
                     name=dict(required=True, type="str"),
                     type=dict(
                         required=False,
@@ -183,12 +199,19 @@ def main():
     )
 
     required_if = [
-        ("state", "present", ["device_type", "name", "type"]),
-        ("state", "absent", ["device_type", "name", "type"]),
+        ("state", "present", ["name", "type"]),
+        ("state", "absent", ["name", "type"]),
+    ]
+
+    required_one_of = [
+        ("device_type", "module_type"),
     ]
 
     module = NetboxAnsibleModule(
-        argument_spec=argument_spec, supports_check_mode=True, required_if=required_if
+        argument_spec=argument_spec,
+        supports_check_mode=True,
+        required_if=required_if,
+        required_one_of=required_one_of,
     )
 
     netbox_rear_port_template = NetboxDcimModule(module, NB_REAR_PORT_TEMPLATES)
diff --git a/tests/integration/targets/v4.3/tasks/netbox_front_port_template.yml b/tests/integration/targets/v4.3/tasks/netbox_front_port_template.yml
@@ -7,6 +7,15 @@
 ### NETBOX_FRONT_PORT_TEMPLATE
 ##
 ##
+- name: "NETBOX_FRONT_PORT_TEMPLATE 0.1: Create module type for testing power ports on module types"
+  netbox.netbox.netbox_module_type:
+    netbox_url: http://localhost:32768
+    netbox_token: "0123456789abcdef0123456789abcdef01234567"
+    data:
+      model: Module Type Front And Rear Port Tests
+      manufacturer: Test Manufacturer
+    state: present
+
 - name: "FRONT_PORT_TEMPLATE 1: Necessary info creation"
   netbox.netbox.netbox_front_port_template:
     netbox_url: http://localhost:32768
@@ -145,3 +154,142 @@
       - test_six['front_port_template']['type'] == "bnc"
       - test_six['front_port_template']['rear_port'] == 1
       - test_six['msg'] == "front_port_template Front Port Template already exists"
+
+- name: "FRONT_PORT_TEMPLATE 7: Necessary info creation"
+  netbox.netbox.netbox_front_port_template:
+    netbox_url: http://localhost:32768
+    netbox_token: "0123456789abcdef0123456789abcdef01234567"
+    data:
+      name: Module Type Front Port Template
+      module_type: Module Type Front And Rear Port Tests
+      type: bnc
+      rear_port_template: Module Type Rear Port Template
+    state: present
+  register: test_seven
+
+- name: "FRONT_PORT_TEMPLATE 7: ASSERT - Necessary info creation"
+  ansible.builtin.assert:
+    that:
+      - test_seven is changed
+      - test_seven['diff']['before']['state'] == "absent"
+      - test_seven['diff']['after']['state'] == "present"
+      - test_seven['front_port_template']['name'] == "Module Type Front Port Template"
+      - test_seven['front_port_template']['module_type'] == 1
+      - test_seven['front_port_template']['type'] == "bnc"
+      - test_seven['front_port_template']['rear_port'] == 4
+      - test_seven['msg'] == "front_port_template Module Type Front Port Template created"
+
+- name: "FRONT_PORT_TEMPLATE 8: Create duplicate"
+  netbox.netbox.netbox_front_port_template:
+    netbox_url: http://localhost:32768
+    netbox_token: "0123456789abcdef0123456789abcdef01234567"
+    data:
+      name: Module Type Front Port Template
+      module_type: Module Type Front And Rear Port Tests
+      type: bnc
+      rear_port_template: Module Type Rear Port Template
+    state: present
+  register: test_eight
+
+- name: "FRONT_PORT_TEMPLATE 8: ASSERT - Create duplicate"
+  ansible.builtin.assert:
+    that:
+      - not test_eight['changed']
+      - test_eight['front_port_template']['name'] == "Module Type Front Port Template"
+      - test_eight['front_port_template']['module_type'] == 1
+      - test_eight['front_port_template']['type'] == "bnc"
+      - test_eight['front_port_template']['rear_port'] == 4
+      - test_eight['msg'] == "front_port_template Module Type Front Port Template already exists"
+
+- name: "FRONT_PORT_TEMPLATE 9: Update Front Port Template with other fields"
+  netbox.netbox.netbox_front_port_template:
+    netbox_url: http://localhost:32768
+    netbox_token: "0123456789abcdef0123456789abcdef01234567"
+    data:
+      name: Module Type Front Port Template
+      module_type: Module Type Front And Rear Port Tests
+      type: bnc
+      rear_port_template: Module Type Rear Port Template
+      rear_port_template_position: 5
+    state: present
+  register: test_nine
+
+- name: "FRONT_PORT_TEMPLATE 9: ASSERT - Update Front Port Template with other fields"
+  ansible.builtin.assert:
+    that:
+      - test_nine is changed
+      - test_nine['diff']['after']['rear_port_position'] == 5
+      - test_nine['front_port_template']['name'] == "Module Type Front Port Template"
+      - test_nine['front_port_template']['module_type'] == 1
+      - test_nine['front_port_template']['type'] == "bnc"
+      - test_nine['front_port_template']['rear_port_position'] == 5
+      - test_nine['front_port_template']['rear_port'] == 4
+      - test_nine['msg'] == "front_port_template Module Type Front Port Template updated"
+
+- name: "FRONT_PORT_TEMPLATE 10: Create Front Port Template for Delete Test"
+  netbox.netbox.netbox_front_port_template:
+    netbox_url: http://localhost:32768
+    netbox_token: "0123456789abcdef0123456789abcdef01234567"
+    data:
+      name: Module Type Front Port Template 2
+      module_type: Module Type Front And Rear Port Tests
+      type: bnc
+      rear_port_template: Module Type Rear Port Template
+    state: present
+  register: test_ten
+
+- name: "FRONT_PORT_TEMPLATE 10: ASSERT - Create Front Port Template for Delete Test"
+  ansible.builtin.assert:
+    that:
+      - test_ten is changed
+      - test_ten['diff']['before']['state'] == "absent"
+      - test_ten['diff']['after']['state'] == "present"
+      - test_ten['front_port_template']['name'] == "Module Type Front Port Template 2"
+      - test_ten['front_port_template']['module_type'] == 1
+      - test_ten['front_port_template']['type'] == "bnc"
+      - test_ten['front_port_template']['rear_port'] == 4
+      - test_ten['msg'] == "front_port_template Module Type Front Port Template 2 created"
+
+- name: "FRONT_PORT_TEMPLATE 11: Delete Front Port Template"
+  netbox.netbox.netbox_front_port_template:
+    netbox_url: http://localhost:32768
+    netbox_token: "0123456789abcdef0123456789abcdef01234567"
+    data:
+      name: Module Type Front Port Template 2
+      module_type: Module Type Front And Rear Port Tests
+      type: bnc
+      rear_port_template: Module Type Rear Port Template
+    state: absent
+  register: test_eleven
+
+- name: "FRONT_PORT_TEMPLATE 11: ASSERT - Delete Front Port Template"
+  ansible.builtin.assert:
+    that:
+      - test_eleven is changed
+      - test_eleven['diff']['before']['state'] == "present"
+      - test_eleven['diff']['after']['state'] == "absent"
+      - test_eleven['msg'] == "front_port_template Module Type Front Port Template 2 deleted"
+
+- name: "FRONT_PORT_TEMPLATE 12: Create duplicate with rear_port_template dictionary"
+  netbox.netbox.netbox_front_port_template:
+    netbox_url: http://localhost:32768
+    netbox_token: "0123456789abcdef0123456789abcdef01234567"
+    data:
+      name: Module Type Front Port Template
+      module_type: Module Type Front And Rear Port Tests
+      type: bnc
+      rear_port_template:
+        module: Module Type Front And Rear Port Tests
+        name: Module Type Rear Port Template
+    state: present
+  register: test_twelve
+
+- name: "FRONT_PORT_TEMPLATE 12: ASSERT - Create duplicate with rear_port_template dictionary"
+  ansible.builtin.assert:
+    that:
+      - not test_twelve['changed']
+      - test_twelve['front_port_template']['name'] == "Module Type Front Port Template"
+      - test_twelve['front_port_template']['module_type'] == 1
+      - test_twelve['front_port_template']['type'] == "bnc"
+      - test_twelve['front_port_template']['rear_port'] == 4
+      - test_twelve['msg'] == "front_port_template Module Type Front Port Template already exists"
diff --git a/tests/integration/targets/v4.3/tasks/netbox_power_port_template.yml b/tests/integration/targets/v4.3/tasks/netbox_power_port_template.yml
@@ -145,7 +145,7 @@
       - test_six['diff']['before']['state'] == "absent"
       - test_six['diff']['after']['state'] == "present"
       - test_six['power_port_template']['name'] == "Module Power Port Template"
-      - test_six['power_port_template']['module_type'] == 1
+      - test_six['power_port_template']['module_type'] == 2
       - test_six['msg'] == "power_port_template Module Power Port Template created"
 
 - name: "POWER_PORT_TEMPLATE 7: Create duplicate"
@@ -163,7 +163,7 @@
     that:
       - not test_seven['changed']
       - test_seven['power_port_template']['name'] == "Module Power Port Template"
-      - test_seven['power_port_template']['module_type'] == 1
+      - test_seven['power_port_template']['module_type'] == 2
       - test_seven['msg'] == "power_port_template Module Power Port Template already exists"
 
 - name: "POWER_PORT_TEMPLATE 8: Update power_port_template with other fields"
@@ -187,7 +187,7 @@
       - test_eight['diff']['after']['allocated_draw'] == 10
       - test_eight['diff']['after']['maximum_draw'] == 20
       - test_eight['power_port_template']['name'] == "Module Power Port Template"
-      - test_eight['power_port_template']['module_type'] == 1
+      - test_eight['power_port_template']['module_type'] == 2
       - test_eight['power_port_template']['type'] == "ita-e"
       - test_eight['power_port_template']['allocated_draw'] == 10
       - test_eight['power_port_template']['maximum_draw'] == 20
diff --git a/tests/integration/targets/v4.3/tasks/netbox_rear_port_template.yml b/tests/integration/targets/v4.3/tasks/netbox_rear_port_template.yml

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,6 @@`
`11`	`11`	`SLUG_REQUIRED,`
`12`	`12`	`)`
`13`	`13`
`14`		`-`
`15`	`14`	`NB_PROVIDERS = "providers"`
`16`	`15`	`NB_PROVIDER_NETWORKS = "provider_networks"`
`17`	`16`	`NB_CIRCUIT_TYPES = "circuit_types"`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,6 @@`
`12`	`12`	`SLUG_REQUIRED,`
`13`	`13`	`)`
`14`	`14`
`15`		`-`
`16`	`15`	`NB_CABLES = "cables"`
`17`	`16`	`NB_CONSOLE_PORTS = "console_ports"`
`18`	`17`	`NB_CONSOLE_PORT_TEMPLATES = "console_port_templates"`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,6 @@`
`15`	`15`	`SLUG_REQUIRED,`
`16`	`16`	`)`
`17`	`17`
`18`		`-`
`19`	`18`	`NB_AGGREGATES = "aggregates"`
`20`	`19`	`NB_ASNS = "asns"`
`21`	`20`	`NB_FHRP_GROUPS = "fhrp_groups"`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@`
`14`	`14`	`SLUG_REQUIRED,`
`15`	`15`	`)`
`16`	`16`
`17`		`-`
`18`	`17`	`NB_L2VPNS = "l2vpns"`
`19`	`18`	`NB_L2VPN_TERMINATIONS = "l2vpn_terminations"`
`20`	`19`	`NB_TUNNELS = "tunnels"`