Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

resource - fix various tests #27544

Merged
merged 6 commits into from
Nov 14, 2024
Merged

resource - fix various tests #27544

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
d3f3a0b
resource - fix various tests
mbfrahry Sep 30, 2024
243f9e1
Address review
mbfrahry Oct 1, 2024
c2f1539
Lint
mbfrahry Oct 1, 2024
3f3c056
merge with main
mbfrahry Oct 17, 2024
15cd211
lint
mbfrahry Oct 17, 2024
0700624
remove anonymous function from requiresImport test
stephybun Oct 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
remove anonymous function from requiresImport test
  • Loading branch information
@stephybun
stephybun committed Oct 23, 2024
commit 070062409cf16c2af3fc4cd5b4e87dd09e86554e
4 changes: 1 addition & 3 deletions internal/services/resource/resource_management_private_link_association_resource_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,7 @@ func TestAccResourceManagementPrivateLinkAssociation_requiresImport(t *testing.T
check.That(data.ResourceName).ExistsInAzure(r),
),
},
data.RequiresImportErrorStep(func(data acceptance.TestData) string {
return r.requiresImport(data)
}),
data.RequiresImportErrorStep(r.requiresImport),
})
}

Expand Down
Loading