From 4236939b6c0c5002a3b2746530c7f7ee04fe81b1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miloslav=20Trma=C4=8D?= <mitr@redhat.com>
Date: Mon, 25 Nov 2024 20:41:50 +0100
Subject: [PATCH] Use testify's ErrorAs to simplify tests a bit
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Should not change (test) behavior, other than perhaps
better test failure output.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
---
 docker/errors_test.go | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/docker/errors_test.go b/docker/errors_test.go
index 38bc568b2f..fe4e2a8c1b 100644
--- a/docker/errors_test.go
+++ b/docker/errors_test.go
@@ -3,7 +3,6 @@ package docker
 import (
 	"bufio"
 	"bytes"
-	"errors"
 	"net/http"
 	"testing"
 
@@ -109,8 +108,7 @@ func TestRegistryHTTPResponseToError(t *testing.T) {
 			errorCode:         &errcode.ErrorCodeUnknown,
 			fn: func(t *testing.T, err error) {
 				var e errcode.Error
-				ok := errors.As(err, &e)
-				require.True(t, ok)
+				require.ErrorAs(t, err, &e)
 				// Note: (skopeo inspect) is checking for this errcode.Error value
 				assert.Equal(t, errcode.Error{
 					Code:    errcode.ErrorCodeUnknown, // The NOT_FOUND value is not defined, and turns into Unknown
@@ -140,8 +138,7 @@ func TestRegistryHTTPResponseToError(t *testing.T) {
 			errorCode:         &errcode.ErrorCodeUnknown,
 			fn: func(t *testing.T, err error) {
 				var e errcode.Error
-				ok := errors.As(err, &e)
-				require.True(t, ok)
+				require.ErrorAs(t, err, &e)
 				// isManifestUnknownError is checking for this
 				assert.Equal(t, errcode.Error{
 					Code:    errcode.ErrorCodeUnknown, // The 404 value is not defined, and turns into Unknown
@@ -166,8 +163,7 @@ func TestRegistryHTTPResponseToError(t *testing.T) {
 			unwrappedErrorPtr: &unwrappedUnexpectedHTTPResponseError,
 			fn: func(t *testing.T, err error) {
 				var e *unexpectedHTTPResponseError
-				ok := errors.As(err, &e)
-				require.True(t, ok)
+				require.ErrorAs(t, err, &e)
 				// isManifestUnknownError is checking for this
 				assert.Equal(t, 404, e.StatusCode)
 				assert.Equal(t, []byte("Not found\r"), e.Response)
@@ -191,8 +187,7 @@ func TestRegistryHTTPResponseToError(t *testing.T) {
 			errorCode:         &errcode.ErrorCodeUnknown,
 			fn: func(t *testing.T, err error) {
 				var e errcode.Error
-				ok := errors.As(err, &e)
-				require.True(t, ok)
+				require.ErrorAs(t, err, &e)
 				// isManifestUnknownError is checking for this
 				assert.Equal(t, errcode.Error{
 					Code:    errcode.ErrorCodeUnknown, // The NOT_FOUND value is not defined, and turns into Unknown
@@ -212,13 +207,11 @@ func TestRegistryHTTPResponseToError(t *testing.T) {
 			assert.IsType(t, c.errorType, err, c.name)
 		}
 		if c.unwrappedErrorPtr != nil {
-			found := errors.As(err, c.unwrappedErrorPtr)
-			assert.True(t, found, c.name)
+			assert.ErrorAs(t, err, c.unwrappedErrorPtr, c.name)
 		}
 		if c.errorCode != nil {
 			var ec errcode.ErrorCoder
-			ok := errors.As(err, &ec)
-			require.True(t, ok, c.name)
+			require.ErrorAs(t, err, &ec, c.name)
 			assert.Equal(t, *c.errorCode, ec.ErrorCode(), c.name)
 		}
 		if c.fn != nil {