cmd/govim: move unimported completion tests to scenario_default

The default for the unimported completions config option in gopls (and govim) switched to be "on" some time ago. Hence we do not need a special scenario for the corresponding tests, and can move them to scenario_default.
govim · Feb 22, 2020 · f3e7127 · f3e7127
1 parent 60ac92c
commit f3e7127
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 3 deletions.
diff --git a/cmd/govim/config/config.go b/cmd/govim/config/config.go
@@ -99,11 +99,15 @@ type Config struct {
 	CompletionMatcher *CompletionMatcher `json:",omitempty"`
 
 	// Staticcheck enables staticcheck analyses in gopls
+	//
+	// Default: false
 	Staticcheck *bool `json:",omitempty"`
 
 	// CompleteUnimported configures gopls to attempt completions for unimported
 	// standard library packages. e.g. when a user completes rand.<>, propose
 	// rand.Seed (from math/rand) and rand.Prime (from crypto/rand), etc.
+	//
+	// Default: true
 	CompleteUnimported *bool `json:",omitempty"`
 
 	// GoImportsLocalPrefix is used to specify goimports's -local behavior. When

diff --git a/cmd/govim/testdata/scenario_completeunimported/user_config.json b/cmd/govim/testdata/scenario_completeunimported/user_config.json
diff --git a/.../scenario_completeunimported/complete.txt → .../scenario_default/complete_unimported.txt b/.../scenario_completeunimported/complete.txt → .../scenario_default/complete_unimported.txt
diff --git a/...ompleteunimported/complete_empty_file.txt → ...efault/complete_unimported_empty_file.txt b/...ompleteunimported/complete_empty_file.txt → ...efault/complete_unimported_empty_file.txt
diff --git a/..._completeunimported/complete_new_file.txt → ..._default/complete_unimported_new_file.txt b/..._completeunimported/complete_new_file.txt → ..._default/complete_unimported_new_file.txt