Fix problems caught by staticcheck v0.3.0

This will allow us to upgrade to this version in a later commit without
causing the our build checks to fail.

internal/cloud/e2e/helper_test.go

@@ -251,6 +251,6 @@ findTfVersion:
 	}
 
 	if !hasVersion {
-		t.Skip(fmt.Sprintf("Skipping test because TFC/E does not have current Terraform version to test with (%s)", version))
+		t.Skipf("Skipping test because TFC/E does not have current Terraform version to test with (%s)", version)
 	}
 }

internal/command/logout.go

@@ -78,9 +78,6 @@ func (c *LogoutCommand) Run(args []string) int {
 		return 1
 	}
 
-	// credsCtx might not be set if we're using a mock credentials source
-	// in a test, but it should always be set in normal use.
-	if credsCtx != nil {
 	switch credsCtx.Location {
 	case cliconfig.CredentialsNotAvailable:
 		c.Ui.Output(fmt.Sprintf("No credentials for %s are stored.\n", dispHostname))
@@ -90,7 +87,6 @@ func (c *LogoutCommand) Run(args []string) int {
 	case cliconfig.CredentialsInPrimaryFile:
 		c.Ui.Output(fmt.Sprintf("Removing the stored credentials for %s from the following file:\n    %s\n", dispHostname, credsCtx.LocalFilename))
 	}
-	}
 
 	err = creds.ForgetForHost(hostname)
 	if err != nil {

internal/lang/marks/marks.go

@@ -1,18 +1,16 @@
 package marks
 
 import (
-	"strings"
-
 	"github.com/zclconf/go-cty/cty"
 )
 
 // valueMarks allow creating strictly typed values for use as cty.Value marks.
-// The variable name for new values should be the title-cased format of the
-// value to better match the GoString output for debugging.
+// Each distinct mark value must be a constant in this package whose value
+// is a valueMark whose underlying string matches the name of the variable.
 type valueMark string
 
 func (m valueMark) GoString() string {
-	return "marks." + strings.Title(string(m))
+	return "marks." + string(m)
 }
 
 // Has returns true if and only if the cty.Value has the given mark.
@@ -36,9 +34,9 @@ func Contains(val cty.Value, mark valueMark) bool {
 
 // Sensitive indicates that this value is marked as sensitive in the context of
 // Terraform.
-var Sensitive = valueMark("sensitive")
+const Sensitive = valueMark("Sensitive")
 
 // TypeType is used to indicate that the value contains a representation of
 // another value's type. This is part of the implementation of the console-only
 // `type` function.
-var TypeType = valueMark("typeType")
+const TypeType = valueMark("TypeType")

internal/terraform/evaluate.go

@@ -792,7 +792,8 @@ func (d *evaluationStateData) GetResource(addr addrs.Resource, rng tfdiags.Sourc
 		instances[key] = val
 	}
 
-	ret := cty.DynamicVal
+	// ret should be populated with a valid value in all cases below
+	var ret cty.Value
 
 	switch {
 	case config.Count != nil: