Skip to content

Commit

Permalink
Merge pull request #2520 from chenz4027/OCM-11556
Browse files Browse the repository at this point in the history
OCM-11556 | fix: remove multiple ca tests from getClusterRegistryConfig
  • Loading branch information
gdbranco authored Oct 1, 2024
2 parents c0ab25d + 42cc529 commit 3300a71
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions cmd/describe/cluster/cmd_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,6 @@ var _ = Describe("getClusterRegistryConfig", func() {
It("Should return expected output", func() {
mockCa := make(map[string]string)
mockCa["registry.io"] = "-----BEGIN CERTIFICATE-----\nlalala\n-----END CERTIFICATE-----\n"
mockCa["registry.io2"] = "-----BEGIN CERTIFICATE-----\nlalala\n-----END CERTIFICATE-----\n"
mockCluster, err := cmv1.NewCluster().RegistryConfig(cmv1.NewClusterRegistryConfig().AdditionalTrustedCa(mockCa).
RegistrySources(cmv1.NewRegistrySources().
AllowedRegistries([]string{"allow1.com", "allow2.com"}...).
Expand All @@ -164,8 +163,7 @@ var _ = Describe("getClusterRegistryConfig", func() {
" - Platform Allowlist: test-id\n" +
" - Registries: registry1.io,registry2.io\n" +
" - Additional Trusted CA: \n" +
" - registry.io: REDACTED\n" +
" - registry.io2: REDACTED\n"
" - registry.io: REDACTED\n"
Expect(output).To(Equal(expectedOutput))
})
})
Expand Down

0 comments on commit 3300a71

Please sign in to comment.