add time.Sleep to cross-module tests

hashicorp · Nov 18, 2021 · 8431c2d · 8431c2d
1 parent 2aed806
commit 8431c2d
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/internal/langserver/handlers/code_lens_test.go b/internal/langserver/handlers/code_lens_test.go
@@ -262,6 +262,9 @@ variable "instances" {
   type = number
 }
 `, submodUri.URI())})
+	// TODO remove once we support synchronous dependent tasks
+	// See https://github.com/hashicorp/terraform-ls/issues/719
+	time.Sleep(2 * time.Second)
 	ls.CallAndExpectResponse(t, &langserver.CallRequest{
 		Method: "textDocument/codeLens",
 		ReqParams: fmt.Sprintf(`{

diff --git a/internal/langserver/handlers/complete_test.go b/internal/langserver/handlers/complete_test.go
@@ -717,11 +717,9 @@ output "test" {
 		}
 	}`, mainCfg, tmpDir.URI())})
 
-	// module manifest-dependent tasks are scheduled & executed
-	// asynchronously and we currently have no way of waiting
-	// for them to complete.
 	// TODO remove once we support synchronous dependent tasks
-	time.Sleep(1500 * time.Millisecond)
+	// See https://github.com/hashicorp/terraform-ls/issues/719
+	time.Sleep(2 * time.Second)
 
 	ls.CallAndExpectResponse(t, &langserver.CallRequest{
 		Method: "textDocument/completion",

diff --git a/internal/langserver/handlers/go_to_ref_target_test.go b/internal/langserver/handlers/go_to_ref_target_test.go
@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"path/filepath"
 	"testing"
+	"time"
 
 	"github.com/hashicorp/go-version"
 	"github.com/hashicorp/terraform-ls/internal/langserver"
@@ -154,6 +155,9 @@ func TestDefinition_moduleInputToVariable(t *testing.T) {
 			"uri": "%s/main.tf"
 		}
 	}`, modUri.URI())})
+	// TODO remove once we support synchronous dependent tasks
+	// See https://github.com/hashicorp/terraform-ls/issues/719
+	time.Sleep(2 * time.Second)
 	ls.CallAndExpectResponse(t, &langserver.CallRequest{
 		Method: "textDocument/definition",
 		ReqParams: fmt.Sprintf(`{

diff --git a/internal/langserver/handlers/references_test.go b/internal/langserver/handlers/references_test.go
@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"path/filepath"
 	"testing"
+	"time"
 
 	"github.com/hashicorp/go-version"
 	"github.com/hashicorp/terraform-ls/internal/langserver"
@@ -178,6 +179,9 @@ variable "instances" {
 			"uri": "%s/main.tf"
 		}
 	}`, submodUri.URI())})
+	// TODO remove once we support synchronous dependent tasks
+	// See https://github.com/hashicorp/terraform-ls/issues/719
+	time.Sleep(2 * time.Second)
 	ls.CallAndExpectResponse(t, &langserver.CallRequest{
 		Method: "textDocument/references",
 		ReqParams: fmt.Sprintf(`{