Fix tests for updated sync integration data #1766

Merged
mfreeman451 merged 1 commit from refs/pull/1766/head into 849-update-devices_mv-materialized-view-to-include-relationship-metadata 2025-06-20 03:02:56 +00:00
mfreeman451 commented 2025-06-20 03:01:09 +00:00 (Migrated from github.com)
Owner

Imported from GitHub pull request.

Original GitHub pull request: #992
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/992
Original created: 2025-06-20T03:01:09Z
Original updated: 2025-06-20T03:02:59Z
Original head: carverauto/serviceradar:codex/fix-lint-errors-in-tests
Original base: 849-update-devices_mv-materialized-view-to-include-relationship-metadata
Original merged: 2025-06-20T03:02:56Z by @mfreeman451

Summary

  • update sync tests for new New signature
  • adjust Netbox processDevices test for sweep result output
  • update Armis integration tests for updated data structures

Testing

  • make lint (fails: unsupported configuration version)
  • make test (fails: some sweeper tests fail)
  • go test ./pkg/sync/integrations/netbox -run TestProcessDevices_UsesIDs -count=1 -v
  • go test ./pkg/sync/integrations/armis -run TestArmisIntegration_Fetch_NoUpdater -count=1 -v

https://chatgpt.com/codex/tasks/task_e_6854c9676f348320bd9d765e1ab4d68c

Imported from GitHub pull request. Original GitHub pull request: #992 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/992 Original created: 2025-06-20T03:01:09Z Original updated: 2025-06-20T03:02:59Z Original head: carverauto/serviceradar:codex/fix-lint-errors-in-tests Original base: 849-update-devices_mv-materialized-view-to-include-relationship-metadata Original merged: 2025-06-20T03:02:56Z by @mfreeman451 --- ## Summary - update sync tests for new `New` signature - adjust Netbox processDevices test for sweep result output - update Armis integration tests for updated data structures ## Testing - `make lint` *(fails: unsupported configuration version)* - `make test` *(fails: some sweeper tests fail)* - `go test ./pkg/sync/integrations/netbox -run TestProcessDevices_UsesIDs -count=1 -v` - `go test ./pkg/sync/integrations/armis -run TestArmisIntegration_Fetch_NoUpdater -count=1 -v` ------ https://chatgpt.com/codex/tasks/task_e_6854c9676f348320bd9d765e1ab4d68c
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
carverauto/serviceradar!1766
No description provided.