Fix bug when syncing interfaces installed in a module

[gsnider2195]

Closes: NAPPS-799

Bug

When onboarding a Device that uses a DeviceType with InterfaceTemplates attached to ModuleTypes, the Device will be created with interfaces that have a null device field. On subsequent Sync Network Data jobs, the job will try to recreate the interface attached directly to the device instead of attached to the module.

What's Changed

What this PR does: This PR makes the diffsync Nautobot adapters aware of existing interfaces connected to devices through modules (Interface.device is null but Interface.module is not null).

What this PR does not do: Change any of the diffsync Network adapters to parse module information or onboard Device Modules.

To Do

• Explanation of Change(s)
• Added change log fragment(s) (for more information see the documentation)
• Attached Screenshots, Payload Example
• Unit, Integration Tests
• Documentation Updates (when adding/changing features)
• Outline Remaining Work, Constraints from Design

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
nautobot_device_onboarding
constants.py
nautobot_device_onboarding/diffsync/adapters
sync_devices_adapters.py
sync_network_data_adapters.py					199-200, 256, 358, 363
nautobot_device_onboarding/diffsync/models
sync_devices_models.py
sync_network_data_models.py					593
Project Total

This report was generated by python-coverage-comment-action

[smk4664]

This looks great! I see some improvements in here that should lower the repetitive database hits by using data we already received from a previous query instead of doing a new filter.

[jeffkala]

looks good to me. As long as all the testing is still working I'm satisfied.