Replace deprecated ioutil

Author: tristanmorgan

api/api_test.go

@@ -11,9 +11,9 @@ import (
 	"errors"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"net/http"
 	"net/http/httptest"
+	"os"
 	"path/filepath"
 	"regexp"
 	"testing"
@@ -354,7 +354,7 @@ func TestServeWithTLS(t *testing.T) {
 			tlsConf := &tls.Config{}
 			if tc.clientCACert != "" {
 				caCertPool := x509.NewCertPool()
-				caCert, err := ioutil.ReadFile(tc.clientCACert)
+				caCert, err := os.ReadFile(tc.clientCACert)
 				require.NoError(t, err)
 				caCertPool.AppendCertsFromPEM(caCert)
 				tlsConf = &tls.Config{
@@ -549,11 +549,11 @@ func TestServeWithMutualTLS_MultipleCA(t *testing.T) {
 	cleanup := testutils.MakeTempDir(t, tmpDir)
 	defer cleanup()
 	for _, src := range caFiles {
-		input, err := ioutil.ReadFile(src)
+		input, err := os.ReadFile(src)
 		file := filepath.Base(src)
 		dest := filepath.Join(tmpDir, file)
 		require.NoError(t, err)
-		err = ioutil.WriteFile(dest, input, 0644)
+		err = os.WriteFile(dest, input, 0644)
 		require.NoError(t, err)
 	}
 

api/oapigen/client.go

@@ -9,7 +9,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"testing"
 	"time"
@@ -82,7 +82,7 @@ func TestRequest(t *testing.T) {
 			require.NoError(t, err)
 			bytesR := bytes.NewBuffer(b)
 			mockResp := &http.Response{
-				Body:       ioutil.NopCloser(bytesR),
+				Body:       io.NopCloser(bytesR),
 				StatusCode: tc.httpStatus,
 			}
 			hc.On("Do", mock.Anything).Return(mockResp, tc.httpError).Once()

api/server_test.go

@@ -6,7 +6,7 @@ package api
 import (
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"sort"
 	"strings"
@@ -101,7 +101,7 @@ func (h *taskHandler) updateTask(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	body, err := ioutil.ReadAll(r.Body)
+	body, err := io.ReadAll(r.Body)
 	if err != nil {
 		logger.Trace("unable to read request body from update", "error", err)
 		jsonErrorResponse(ctx, w, http.StatusInternalServerError, err)

api/task.go

@@ -4,15 +4,15 @@
 package command
 
 import (
-	"io/ioutil"
+	"io"
 	"testing"
 
 	"github.com/mitchellh/cli"
 	"github.com/stretchr/testify/assert"
 )
 
 func Test_Commands(t *testing.T) {
-	cf := Commands(ioutil.Discard, ioutil.Discard)
+	cf := Commands(io.Discard, io.Discard)
 
 	// map of commands to synopsis
 	expectedCommands := map[string]cli.Command{

command/commands_test.go

@@ -7,7 +7,6 @@ import (
 	"flag"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"net/url"
 	"strings"
 
@@ -82,7 +81,7 @@ func (m *meta) defaultFlagSet(name string) *flag.FlagSet {
 	m.tls.sslVerify = m.flags.Bool(FlagSSLVerify, true, fmt.Sprintf("Boolean to verify SSL or not. Set to true to verify SSL. "+
 		"\n\t\tThis can also be specified using the %s environment variable.", api.EnvTLSSSLVerify))
 
-	m.flags.SetOutput(ioutil.Discard)
+	m.flags.SetOutput(io.Discard)
 
 	return m.flags
 }

command/meta.go

@@ -6,7 +6,6 @@ package config
 import (
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
 	"os"
 	"path/filepath"
 	"strconv"
@@ -489,7 +488,7 @@ func fromFile(path string) (*Config, error) {
 	}
 
 	logger := logging.Global().Named(logSystemName)
-	content, err := ioutil.ReadFile(path)
+	content, err := os.ReadFile(path)
 	if err != nil {
 		logger.Error("failed reading config file from disk", filePathLogKey, path)
 		return nil, err
@@ -533,7 +532,7 @@ func fromPath(path string) (*Config, error) {
 	}
 
 	// Ensure the given filepath has at least one config file
-	files, err := ioutil.ReadDir(path)
+	files, err := os.ReadDir(path)
 	if err != nil {
 		logger.Error("failed listing directory", filePathLogKey, path)
 		return nil, err
@@ -591,7 +590,7 @@ func stringFromEnv(list []string, def string) *string {
 
 func stringFromFile(list []string, def string) *string {
 	for _, s := range list {
-		c, err := ioutil.ReadFile(s)
+		c, err := os.ReadFile(s)
 		if err == nil {
 			return String(strings.TrimSpace(string(c)))
 		}

config/config.go

@@ -7,7 +7,7 @@ import (
 	"bytes"
 	"context"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"log"
 	"os"
 	"path/filepath"
@@ -145,7 +145,7 @@ func NewTerraform(config *TerraformConfig) (*Terraform, error) {
 		postApply:         h,
 		resolver:          hcat.NewResolver(),
 		watcher:           config.Watcher,
-		fileReader:        ioutil.ReadFile,
+		fileReader:        os.ReadFile,
 		logger:            logger,
 	}, nil
 }
@@ -496,7 +496,7 @@ func (tf *Terraform) inspectTask(ctx context.Context, returnPlan bool) (InspectP
 		if tf.logClient {
 			tfLogger = log.New(log.Writer(), "", log.Flags())
 		} else {
-			tfLogger = log.New(ioutil.Discard, "", 0)
+			tfLogger = log.New(io.Discard, "", 0)
 		}
 		defer tf.client.SetStdout(tfLogger.Writer())
 	}

driver/terraform.go

@@ -9,7 +9,6 @@ package e2e
 
 import (
 	"fmt"
-	"io/ioutil"
 	"os"
 	"path/filepath"
 	"regexp"
@@ -631,7 +630,7 @@ task {
 			// If required by test case, verify contents of generated variables file
 			for _, v := range tc.tfVarsFiles {
 				expectedFilePath := filepath.Join(tempDir, taskName, "variables.auto.tfvars")
-				b, err := ioutil.ReadFile(expectedFilePath)
+				b, err := os.ReadFile(expectedFilePath)
 				require.NoError(t, err)
 				assert.Contains(t, string(b), v)
 			}

e2e/command_test.go

@@ -8,7 +8,7 @@ package hcltmpl
 
 import (
 	"context"
-	"io/ioutil"
+	"io"
 	"testing"
 	"time"
 
@@ -77,8 +77,8 @@ func TestLoadDynamicConfig_ConsulKV(t *testing.T) {
 	srv, err := testutil.NewTestServerConfigT(tb,
 		func(c *testutil.TestServerConfig) {
 			c.LogLevel = "warn"
-			c.Stdout = ioutil.Discard
-			c.Stderr = ioutil.Discard
+			c.Stdout = io.Discard
+			c.Stderr = io.Discard
 		})
 	require.NoError(t, err)
 	clients := hcat.NewClientSet()