Skip to content

Commit 744cd8b

Browse files
committed
fix unit test and linting
1 parent 55dedf6 commit 744cd8b

File tree

2 files changed

+2
-18
lines changed

2 files changed

+2
-18
lines changed

internal/controller/netbox_testdata_test.go

-1
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ var restorationHash = "6f6c67651f0b43b2969ba2ae35c74fc91815513b"
5656
var customFieldsCR = map[string]string{"example_field": "example value"}
5757
var customFieldsWithHashCR = map[string]string{"example_field": "example value", "netboxOperatorRestorationHash": restorationHash}
5858

59-
var customFields = map[string]interface{}{"example_field": "example value"}
6059
var customFieldsWithHash = map[string]interface{}{"example_field": "example value", "netboxOperatorRestorationHash": restorationHash}
6160
var customFieldsWithHashMissmatch = map[string]interface{}{"example_field": "example value", "netboxOperatorRestorationHash": "different hash"}
6261

pkg/netbox/api/prefix_test.go

+2-17
Original file line numberDiff line numberDiff line change
@@ -383,22 +383,6 @@ func TestPrefix_ReserveOrUpdate(t *testing.T) {
383383
mockTenancy := mock_interfaces.NewMockTenancyInterface(ctrl)
384384
mockDcim := mock_interfaces.NewMockDcimInterface(ctrl)
385385

386-
//prefix mock input
387-
prefixToCreate := &netboxModels.WritablePrefix{
388-
Comments: comments + warningComment,
389-
Description: description + warningComment,
390-
CustomFields: make(map[string]interface{}),
391-
Prefix: prefixPtr,
392-
Site: &siteOutputId,
393-
Tenant: &tenantOutputId,
394-
Status: "active",
395-
}
396-
397-
createPrefixInput := ipam.
398-
NewIpamPrefixesCreateParams().
399-
WithDefaults().
400-
WithData(prefixToCreate)
401-
402386
//prefix mock output
403387
createPrefixOutput := &ipam.IpamPrefixesCreateCreated{
404388
Payload: &netboxModels.Prefix{
@@ -419,7 +403,8 @@ func TestPrefix_ReserveOrUpdate(t *testing.T) {
419403
mockTenancy.EXPECT().TenancyTenantsList(tenantListRequestInput, nil).Return(tenantListRequestOutput, nil).AnyTimes()
420404
mockDcim.EXPECT().DcimSitesList(siteListRequestInput, nil).Return(siteListRequestOutput, nil).AnyTimes()
421405
mockIpam.EXPECT().IpamPrefixesList(prefixListRequestInput, nil).Return(emptyPrefixListOutput, nil)
422-
mockIpam.EXPECT().IpamPrefixesCreate(createPrefixInput, nil).Return(createPrefixOutput, nil)
406+
// use go mock Any as the input parameter contains pointers
407+
mockIpam.EXPECT().IpamPrefixesCreate(gomock.Any(), nil).Return(createPrefixOutput, nil)
423408

424409
netboxClient := &NetboxClient{
425410
Ipam: mockIpam,

0 commit comments

Comments
 (0)