Skip to content

Commit

Permalink
minor test fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
@dak1n1
dak1n1 committed Dec 14, 2020
1 parent 990e1c3 commit 66c488b
Showing 1 changed file with 99 additions and 11 deletions.
110 changes: 99 additions & 11 deletions kubernetes/resource_kubernetes_replication_controller_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ func TestAccKubernetesReplicationController_basic(t *testing.T) {
resource.TestCheckResourceAttrSet("kubernetes_replication_controller.test", "metadata.0.resource_version"),
resource.TestCheckResourceAttrSet("kubernetes_replication_controller.test", "metadata.0.self_link"),
resource.TestCheckResourceAttrSet("kubernetes_replication_controller.test", "metadata.0.uid"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.container.0.image", nginxImageVersion1),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.container.0.image", imageName),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.container.0.name", "tf-acc-test"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.metadata.0.annotations.%", "1"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.metadata.0.annotations.TestAnnotationFive", "five"),
Expand Down Expand Up @@ -98,7 +98,7 @@ func TestAccKubernetesReplicationController_basic(t *testing.T) {
resource.TestCheckResourceAttrSet("kubernetes_replication_controller.test", "metadata.0.resource_version"),
resource.TestCheckResourceAttrSet("kubernetes_replication_controller.test", "metadata.0.self_link"),
resource.TestCheckResourceAttrSet("kubernetes_replication_controller.test", "metadata.0.uid"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.container.0.image", nginxImageVersion1),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.container.0.image", imageName),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.container.0.name", "tf-acc-test"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.metadata.0.annotations.%", "1"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.metadata.0.annotations.TestAnnotationSix", "six"),
Expand Down Expand Up @@ -168,11 +168,11 @@ func TestAccKubernetesReplicationController_regression(t *testing.T) {
CheckDestroy: testAccCheckKubernetesReplicationControllerDestroy,
Steps: []resource.TestStep{
{
Config: requiredProviders() + testAccKubernetesReplicationControllerConfig_regression("kubernetes-released", name, imageName),
ExpectNonEmptyPlan: true, // UPDATE FIXME bug has been fixed in 2.0 of the provider, but we're testing 1.13 here
Config: requiredProviders() + testAccKubernetesReplicationControllerConfig_released("kubernetes-released", name, imageName),
ExpectNonEmptyPlan: true,
Check: resource.ComposeAggregateTestCheckFunc(
testAccCheckKubernetesReplicationControllerExists("kubernetes_replication_controller.test", &conf1),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.image", busyboxImageVersion),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.image", imageName),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.name", "install"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.command.0", "wget"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.command.1", "-O"),
Expand All @@ -196,10 +196,10 @@ func TestAccKubernetesReplicationController_regression(t *testing.T) {
),
},
{
Config: requiredProviders() + testAccKubernetesReplicationControllerConfig_regression("kubernetes-local", name, imageName),
Config: requiredProviders() + testAccKubernetesReplicationControllerConfig_local("kubernetes-local", name, imageName),
Check: resource.ComposeAggregateTestCheckFunc(
testAccCheckKubernetesReplicationControllerExists("kubernetes_replication_controller.test", &conf2),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.image", busyboxImageVersion),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.image", imageName),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.name", "install"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.command.0", "wget"),
resource.TestCheckResourceAttr("kubernetes_replication_controller.test", "spec.0.template.0.spec.0.init_container.0.command.1", "-O"),
Expand Down Expand Up @@ -1297,7 +1297,7 @@ func testAccKubernetesReplicationControllerConfigWithEmptyDirVolumes(rcName, ima
`, rcName, imageName)
}

func testAccKubernetesReplicationControllerConfig_regression(provider, name, imageName string) string {
func testAccKubernetesReplicationControllerConfig_released(provider, name, imageName string) string {
return fmt.Sprintf(`resource "kubernetes_replication_controller" "test" {
provider = %s
metadata {
Expand All @@ -1316,7 +1316,7 @@ func testAccKubernetesReplicationControllerConfig_regression(provider, name, ima
}
spec {
replicas = 500 # This is intentionally high to exercise the waiter
replicas = 5
selector = {
TestLabelOne = "one"
TestLabelTwo = "two"
Expand Down Expand Up @@ -1349,7 +1349,95 @@ func testAccKubernetesReplicationControllerConfig_regression(provider, name, ima
init_container {
name = "install"
image = busyboxImageVersion
image = "%s"
command = ["wget", "-O", "/work-dir/index.html", "http://kubernetes.io"]
volume_mount {
name = "workdir"
mount_path = "/work-dir"
}
}
dns_config {
nameservers = ["1.1.1.1", "8.8.8.8", "9.9.9.9"]
searches = ["kubernetes.io"]
option {
name = "ndots"
value = 1
}
option {
name = "use-vc"
}
}
volume {
name = "workdir"
empty_dir {}
}
}
}
}
}
`, provider, name, imageName, imageName)
}

func testAccKubernetesReplicationControllerConfig_local(provider, name, imageName string) string {
return fmt.Sprintf(`resource "kubernetes_replication_controller" "test" {
provider = %s
metadata {
annotations = {
TestAnnotationOne = "one"
TestAnnotationTwo = "two"
}
labels = {
TestLabelOne = "one"
TestLabelTwo = "two"
TestLabelThree = "three"
}
name = "%s"
}
spec {
replicas = "5"
selector = {
TestLabelOne = "one"
TestLabelTwo = "two"
TestLabelThree = "three"
}
template {
metadata {
labels = {
TestLabelOne = "one"
TestLabelTwo = "two"
TestLabelThree = "three"
}
}
spec {
container {
name = "nginx"
image = "%s"
port {
container_port = 80
}
volume_mount {
name = "workdir"
mount_path = "/usr/share/nginx/html"
}
}
init_container {
name = "install"
image = "%s"
command = ["wget", "-O", "/work-dir/index.html", "http://kubernetes.io"]
volume_mount {
Expand Down Expand Up @@ -1381,7 +1469,7 @@ func testAccKubernetesReplicationControllerConfig_regression(provider, name, ima
}
}
}
`, provider, name, imageName)
`, provider, name, imageName, imageName)
}

func testAccKubernetesReplicationControllerConfigMinimal(name, imageName string) string {
Expand Down

0 comments on commit 66c488b

Please sign in to comment.