test case for skip dns validation

Author: kart2bc

helpers/config/config.go

@@ -39,6 +39,7 @@ type CatsConfig interface {
 	GetIncludeVolumeServices() bool
 	GetShouldKeepUser() bool
 	GetSkipSSLValidation() bool
+	GetSkipDNSValidation() bool
 	GetUseExistingUser() bool
 
 	GetAddExistingUserToExistingSpace() bool

helpers/config/config_test.go

@@ -27,13 +27,14 @@ type testConfig struct {
 	requiredConfig
 
 	// timeouts
-	DefaultTimeout               *int `json:"default_timeout,omitempty"`
-	CfPushTimeout                *int `json:"cf_push_timeout,omitempty"`
-	LongCurlTimeout              *int `json:"long_curl_timeout,omitempty"`
-	BrokerStartTimeout           *int `json:"broker_start_timeout,omitempty"`
-	AsyncServiceOperationTimeout *int `json:"async_service_operation_timeout,omitempty"`
-	DetectTimeout                *int `json:"detect_timeout,omitempty"`
-	SleepTimeout                 *int `json:"sleep_timeout,omitempty"`
+	DefaultTimeout               *int  `json:"default_timeout,omitempty"`
+	CfPushTimeout                *int  `json:"cf_push_timeout,omitempty"`
+	LongCurlTimeout              *int  `json:"long_curl_timeout,omitempty"`
+	BrokerStartTimeout           *int  `json:"broker_start_timeout,omitempty"`
+	AsyncServiceOperationTimeout *int  `json:"async_service_operation_timeout,omitempty"`
+	DetectTimeout                *int  `json:"detect_timeout,omitempty"`
+	SleepTimeout                 *int  `json:"sleep_timeout,omitempty"`
+	SkipDNSValidation            *bool `json:"skip_dns_validation"`
 
 	TimeoutScale *float64 `json:"timeout_scale,omitempty"`
 
@@ -124,6 +125,7 @@ type nullConfig struct {
 	IsolationSegmentDomain *string `json:"isolation_segment_domain"`
 
 	SkipSSLValidation *bool `json:"skip_ssl_validation"`
+	SkipDNSValidation *bool `json:"skip_dns_validation"`
 
 	ArtifactsDirectory *string `json:"artifacts_directory"`
 
@@ -948,6 +950,54 @@ var _ = Describe("Config", func() {
 		})
 	})
 
+	Describe("GetSkipDNSValidation", func() {
+		Context("when skip_dns_validation is not set", func() {
+			It("returns false (default)", func() {
+				config, err := cfg.NewCatsConfig(tmpFilePath)
+				Expect(err).NotTo(HaveOccurred())
+				Expect(config.GetSkipDNSValidation()).To(BeFalse())
+			})
+		})
+
+		Context("when skip_dns_validation is set to true", func() {
+			BeforeEach(func() {
+				testCfg.SkipDNSValidation = ptrToBool(true)
+			})
+
+			It("returns true", func() {
+				config, err := cfg.NewCatsConfig(tmpFilePath)
+				Expect(err).NotTo(HaveOccurred())
+				Expect(config.GetSkipDNSValidation()).To(BeTrue())
+			})
+
+			It("skips DNS lookup for apps domain", func() {
+				testCfg.AppsDomain = ptrToString("apps.domain.example.com")
+				config, err := cfg.NewCatsConfig(tmpFilePath)
+				Expect(err).NotTo(HaveOccurred())
+				Expect(config.GetAppsDomain()).To(Equal("apps.domain.example.com"))
+			})
+		})
+
+		Context("when skip_dns_validation is set to false", func() {
+			BeforeEach(func() {
+				testCfg.SkipDNSValidation = ptrToBool(false)
+			})
+
+			It("returns false", func() {
+				config, err := cfg.NewCatsConfig(tmpFilePath)
+				Expect(err).NotTo(HaveOccurred())
+				Expect(config.GetSkipDNSValidation()).To(BeFalse())
+			})
+
+			It("performs DNS lookup for apps domain", func() {
+				testCfg.AppsDomain = ptrToString("domain.example.com.does-not-exist")
+				_, err := cfg.NewCatsConfig(tmpFilePath)
+				Expect(err).To(HaveOccurred())
+				Expect(err.Error()).To(ContainSubstring("no such host"))
+			})
+		})
+	})
+
 	Describe("GetAdminUser", func() {
 		It("returns the admin user", func() {
 			c, err := cfg.NewCatsConfig(tmpFilePath)