chore: remove user sleeps

humanitec · Sep 19, 2023 · b36e039 · b36e039
1 parent 05264ff
commit b36e039
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/internal/provider/resource_application_user_test.go b/internal/provider/resource_application_user_test.go
@@ -12,7 +12,6 @@ import (
 func TestAccResourceApplicationUser(t *testing.T) {
 	id := fmt.Sprintf("app-user-test-%d", time.Now().UnixNano())
 	testUserID := "1b305f15-f18f-4357-8311-01f88ed99d1b"
-	eventualConsistentUserAPITimeout := 30 * time.Second
 
 	resource.Test(t, resource.TestCase{
 		PreCheck:                 func() { testAccPreCheck(t) },
@@ -28,9 +27,6 @@ func TestAccResourceApplicationUser(t *testing.T) {
 			},
 			// ImportState testing
 			{
-				PreConfig: func() {
-					time.Sleep(eventualConsistentUserAPITimeout)
-				},
 				ResourceName:      "humanitec_resource_application_user.another_user",
 				ImportState:       true,
 				ImportStateVerify: true,
@@ -42,10 +38,6 @@ func TestAccResourceApplicationUser(t *testing.T) {
 			{
 				Config: testAccResourceApplicationUser(id, testUserID, "developer"),
 				Check: resource.ComposeAggregateTestCheckFunc(
-					func(s *terraform.State) error {
-						time.Sleep(eventualConsistentUserAPITimeout)
-						return nil
-					},
 					resource.TestCheckResourceAttr("humanitec_resource_application_user.another_user", "role", "developer"),
 				),
 			},

diff --git a/internal/provider/resource_environment_type_user_test.go b/internal/provider/resource_environment_type_user_test.go
@@ -12,7 +12,6 @@ import (
 func TestAccResourceEnvironmentTypeUser(t *testing.T) {
 	id := fmt.Sprintf("env-type-user-test-%d", time.Now().UnixNano())
 	testUserID := "c0725726-0613-43d4-8398-907d07fba2e4"
-	eventualConsistentUserAPITimeout := 30 * time.Second
 
 	resource.Test(t, resource.TestCase{
 		PreCheck:                 func() { testAccPreCheck(t) },
@@ -28,9 +27,6 @@ func TestAccResourceEnvironmentTypeUser(t *testing.T) {
 			},
 			// ImportState testing
 			{
-				PreConfig: func() {
-					time.Sleep(eventualConsistentUserAPITimeout)
-				},
 				ResourceName:      "humanitec_resource_environment_type_user.another_user",
 				ImportState:       true,
 				ImportStateVerify: true,
@@ -43,10 +39,6 @@ func TestAccResourceEnvironmentTypeUser(t *testing.T) {
 				// At the moment there is nothing we can update :-/
 				Config: testAccResourceEnvironmentTypeUser(id, testUserID, "deployer"),
 				Check: resource.ComposeAggregateTestCheckFunc(
-					func(s *terraform.State) error {
-						time.Sleep(eventualConsistentUserAPITimeout)
-						return nil
-					},
 					resource.TestCheckResourceAttr("humanitec_resource_environment_type_user.another_user", "role", "deployer"),
 				),
 			},