Bugfix error stringify

irods/common/error_code.go

@@ -1377,23 +1377,27 @@ func init() {
 
 func SplitIRODSErrorCode(code ErrorCode) (ErrorCode, LinuxErrorCode) {
 	mainErrCode := (code / 1000) * 1000
-	subErrCode := (code / 1000)
+	subErrCode := (code % 1000)
 
 	return mainErrCode, LinuxErrorCode(subErrCode)
 }
 
 // GetIRODSErrorString returns string representation of error code
 func GetIRODSErrorString(code ErrorCode) string {
-	if code < 0 {
-		code = -1 * code
+	if code == 0 {
+		return ""
 	}
 
 	mainErrCode, subErrCode := SplitIRODSErrorCode(code)
 
-	if mainErrCode > 0 {
+	if mainErrCode != 0 {
+		if mainErrCode > 0 {
+			mainErrCode *= -1
+		}
+
 		mainErrString, ok := errorCodeDescriptionTable[mainErrCode]
 		if ok {
-			if subErrCode > 0 {
+			if subErrCode != 0 {
 				subErrString := GetLinuxErrorString(subErrCode)
 				return fmt.Sprintf("%s (sub %s)", mainErrString, subErrString)
 			}

irods/common/linux_error_code.go

@@ -272,6 +272,10 @@ func init() {
 
 // GetLinuxErrorString returns string representation of error code
 func GetLinuxErrorString(code LinuxErrorCode) string {
+	if code == 0 {
+		return ""
+	}
+
 	if code < 0 {
 		code = -1 * code
 	}

test/testcases/error_test.go

@@ -0,0 +1,37 @@
+package testcases
+
+import (
+	"testing"
+
+	"github.com/cyverse/go-irodsclient/irods/common"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestError(t *testing.T) {
+	t.Run("test ErrorString", testErrorString)
+}
+
+func testErrorString(t *testing.T) {
+	errcode := common.REMOTE_SERVER_AUTHENTICATION_FAILURE
+
+	// test - value
+	errstr := common.GetIRODSErrorString(errcode)
+	assert.Contains(t, errstr, "REMOTE_SERVER_AUTHENTICATION_FAILURE")
+
+	// test + value
+	errstr = common.GetIRODSErrorString(common.ErrorCode(-1 * int(errcode)))
+	assert.Contains(t, errstr, "REMOTE_SERVER_AUTHENTICATION_FAILURE")
+
+	// test sub value
+	errcode = common.ErrorCode(int(common.REMOTE_SERVER_AUTHENTICATION_FAILURE) - int(common.EIO))
+	assert.Equal(t, int(errcode), -910005)
+
+	mainErrcode, subErrcode := common.SplitIRODSErrorCode(errcode)
+	assert.Equal(t, common.REMOTE_SERVER_AUTHENTICATION_FAILURE, mainErrcode)
+	assert.Equal(t, -1*common.EIO, subErrcode)
+
+	errstr = common.GetIRODSErrorString(errcode)
+	assert.Contains(t, errstr, "REMOTE_SERVER_AUTHENTICATION_FAILURE")
+	assert.Contains(t, errstr, "I/O error")
+
+}