Skip to content

Commit f39cd6d

Browse files
authored
Add test for instanceIDFromProviderID (#133)
Signed-off-by: Niclas Schad <[email protected]>
1 parent 9b48982 commit f39cd6d

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

pkg/ccm/instances_test.go

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,18 @@ var _ = Describe("Node Controller", func() {
5555
Expect(err).NotTo(HaveOccurred())
5656
})
5757

58+
DescribeTable("instanceIDFromProviderID",
59+
func(providerID string, expectedInstanceID string, expectedRegion string) {
60+
instance, region, err := instanceIDFromProviderID(providerID)
61+
Expect(err).NotTo(HaveOccurred())
62+
Expect(instance).To(Equal(expectedInstanceID))
63+
Expect(region).To(Equal(expectedRegion))
64+
},
65+
Entry("new providerID", "stackit://hello-server", "hello-server", ""),
66+
Entry("old providerID", "openstack:///hello-server", "hello-server", ""),
67+
Entry("old regional providerID", "openstack://eu01/hello-server", "hello-server", "eu01"),
68+
)
69+
5870
Describe("InstanceExists", func() {
5971
It("does not error if instance not found", func() {
6072
nodeMockClient.EXPECT().ListServers(gomock.Any(), projectID, region).Return(&[]iaas.Server{}, nil)

0 commit comments

Comments
 (0)