Avoid the deprecated ioutil package.

Author: happygiraffe

roaring64/bsi64_test.go

@@ -5,7 +5,6 @@ import (
 	"encoding/binary"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"math/big"
 	"math/rand"
 	"os"
@@ -645,7 +644,7 @@ func TestMinMaxWithNilFoundSet(t *testing.T) {
 }
 
 func TestBSIWriteToReadFrom(t *testing.T) {
-	file, err := ioutil.TempFile("./testdata", "bsi-test")
+	file, err := os.CreateTemp("./testdata", "bsi-test")
 	if err != nil {
 		t.Fatal(err)
 	}

roaring64/serialization_test.go

@@ -6,7 +6,6 @@ import (
 	"bytes"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"math"
 	"os"
 	"path/filepath"
@@ -276,7 +275,7 @@ func Test_tryReadFromRoaring32WithRoaring64(t *testing.T) {
 }
 
 func Test_tryReadFromRoaring32WithRoaring64_File(t *testing.T) {
-	tempDir, err := ioutil.TempDir("./", "testdata")
+	tempDir, err := os.MkdirTemp("./", "testdata")
 	if err != nil {
 		fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 		return
@@ -290,7 +289,7 @@ func Test_tryReadFromRoaring32WithRoaring64_File(t *testing.T) {
 	}
 
 	name := filepath.Join(tempDir, "r32")
-	if err := ioutil.WriteFile(name, bs, 0o600); err != nil {
+	if err := os.WriteFile(name, bs, 0o600); err != nil {
 		t.Fatal(err)
 	}
 	file, err := os.Open(name)

serialization_frozen_test.go

@@ -6,7 +6,6 @@ package roaring
 import (
 	"bytes"
 	"fmt"
-	"io/ioutil"
 	"os"
 	"reflect"
 	"testing"
@@ -49,12 +48,12 @@ func TestFrozenFormat(t *testing.T) {
 		t.Run("view "+name, func(t *testing.T) {
 			t.Parallel()
 
-			frozenBuf, err := ioutil.ReadFile(fpath)
+			frozenBuf, err := os.ReadFile(fpath)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 				return
 			}
-			portableBuf, err := ioutil.ReadFile(ppath)
+			portableBuf, err := os.ReadFile(ppath)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 				return
@@ -75,12 +74,12 @@ func TestFrozenFormat(t *testing.T) {
 		t.Run("freeze "+name, func(t *testing.T) {
 			t.Parallel()
 
-			frozenBuf, err := ioutil.ReadFile(fpath)
+			frozenBuf, err := os.ReadFile(fpath)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 				return
 			}
-			portableBuf, err := ioutil.ReadFile(ppath)
+			portableBuf, err := os.ReadFile(ppath)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 				return
@@ -106,12 +105,12 @@ func TestFrozenFormat(t *testing.T) {
 		t.Run("freeze with writer"+name, func(t *testing.T) {
 			t.Parallel()
 
-			frozenBuf, err := ioutil.ReadFile(fpath)
+			frozenBuf, err := os.ReadFile(fpath)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 				return
 			}
-			portableBuf, err := ioutil.ReadFile(ppath)
+			portableBuf, err := os.ReadFile(ppath)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 				return

serialization_test.go

@@ -7,7 +7,6 @@ import (
 	"encoding/binary"
 	"encoding/gob"
 	"fmt"
-	"io/ioutil"
 	"os"
 	"path/filepath"
 	"runtime"
@@ -102,7 +101,7 @@ func TestSerializationToFile038(t *testing.T) {
 func TestSerializationReadRunsFromFile039(t *testing.T) {
 	fn := "testdata/bitmapwithruns.bin"
 
-	by, err := ioutil.ReadFile(fn)
+	by, err := os.ReadFile(fn)
 	if err != nil {
 		fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 		return
@@ -464,7 +463,7 @@ func TestBitmap_FromBuffer(t *testing.T) {
 	t.Run("bitmap with runs", func(t *testing.T) {
 		file := "testdata/bitmapwithruns.bin"
 
-		buf, err := ioutil.ReadFile(file)
+		buf, err := os.ReadFile(file)
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 			return
@@ -480,7 +479,7 @@ func TestBitmap_FromBuffer(t *testing.T) {
 
 	t.Run("bitmap without runs", func(t *testing.T) {
 		fn := "testdata/bitmapwithruns.bin"
-		buf, err := ioutil.ReadFile(fn)
+		buf, err := os.ReadFile(fn)
 
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
@@ -496,7 +495,7 @@ func TestBitmap_FromBuffer(t *testing.T) {
 	// all3.classic somehow created by other tests.
 	t.Run("all3.classic bitmap", func(t *testing.T) {
 		file := "testdata/all3.classic"
-		buf, err := ioutil.ReadFile(file)
+		buf, err := os.ReadFile(file)
 
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
@@ -511,7 +510,7 @@ func TestBitmap_FromBuffer(t *testing.T) {
 
 	t.Run("testdata/bitmapwithruns.bin bitmap Ops", func(t *testing.T) {
 		file := "testdata/bitmapwithruns.bin"
-		buf, err := ioutil.ReadFile(file)
+		buf, err := os.ReadFile(file)
 
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
@@ -545,7 +544,7 @@ func TestBitmap_FromBuffer(t *testing.T) {
 
 	t.Run("marking all containers as requiring COW", func(t *testing.T) {
 		file := "testdata/bitmapwithruns.bin"
-		buf, err := ioutil.ReadFile(file)
+		buf, err := os.ReadFile(file)
 
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
@@ -572,7 +571,7 @@ func TestSerializationCrashers(t *testing.T) {
 	}
 
 	for _, crasher := range crashers {
-		data, err := ioutil.ReadFile(crasher)
+		data, err := os.ReadFile(crasher)
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "\n\nIMPORTANT: For testing file IO, the roaring library requires disk access.\nWe omit some tests for now.\n\n")
 			return

smat_generate_test.go

@@ -5,7 +5,6 @@ package roaring
 
 import (
 	"fmt"
-	"io/ioutil"
 	"os"
 	"testing"
 
@@ -21,7 +20,7 @@ func TestGenerateSmatCorpus(t *testing.T) {
 				err, i, actionSeq)
 		}
 		os.MkdirAll("workdir/corpus", 0700)
-		ioutil.WriteFile(fmt.Sprintf("workdir/corpus/%d", i), byteSequence, 0600)
+		os.WriteFile(fmt.Sprintf("workdir/corpus/%d", i), byteSequence, 0600)
 	}
 }