From 2c8060d05d9f0ce2994bce5a2998502571f6d758 Mon Sep 17 00:00:00 2001
From: nickolasdaniel <nickolas.filip@itsyscom.com>
Date: Fri, 14 May 2021 13:51:13 +0300
Subject: [PATCH] Added unit tests for utils

---
 utils/reflect.go      |  8 +---
 utils/reflect_test.go | 94 +++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 96 insertions(+), 6 deletions(-)

diff --git a/utils/reflect.go b/utils/reflect.go
index 46e7e0893..61ed199b9 100644
--- a/utils/reflect.go
+++ b/utils/reflect.go
@@ -488,14 +488,10 @@ func Difference(tm string, items ...interface{}) (diff interface{}, err error) {
 		diff = d
 	case time.Time:
 		d := dt
-		for i, item := range items[1:] {
+		for _, item := range items[1:] {
 			if itmVal, err := IfaceAsTime(item, tm); err == nil {
 				diff = d.Sub(itmVal)
-				if len(items) == i+1 {
-					return diff, nil
-				}
-				items[i] = diff
-				return Difference(tm, items[i:]...)
+				return diff, nil
 			}
 
 			if itmVal, err := IfaceAsDuration(item); err != nil {
diff --git a/utils/reflect_test.go b/utils/reflect_test.go
index 36fc667c5..9b5535fe0 100644
--- a/utils/reflect_test.go
+++ b/utils/reflect_test.go
@@ -528,6 +528,16 @@ func TestDifference(t *testing.T) {
 
 }
 
+func TestDifferenceTime(t *testing.T) {
+	if diff, err := Difference("", time.Date(2009, 11, 10, 23, 10, 0, 0, time.UTC),
+		time.Date(2009, 11, 10, 23, 13, 0, 0, time.UTC), 10*time.Second); err != nil {
+		t.Error(err)
+	} else if diff != -3*time.Minute+0*time.Second {
+		t.Errorf("Expected %v but received %v %T", "-3m0s", diff, diff)
+	}
+
+}
+
 func TestMultiply(t *testing.T) {
 	if _, err := Multiply(10); err == nil || err != ErrNotEnoughParameters {
 		t.Error(err)
@@ -1385,3 +1395,87 @@ func TestReflectFieldMethodInterfaceInvalidField(t *testing.T) {
 		t.Errorf("Expected %v but received %v", ErrNotFound, err)
 	}
 }
+
+type Object struct{}
+
+func (obj *Object) TestFunc1() string {
+	return "test"
+}
+
+func (obj *Object) TestFunc2() (string, string, error) {
+	return "test1", "test2", nil
+}
+
+func (obj *Object) TestFunc3(str string) {}
+
+func (obj *Object) TestFunc4() (string, error) {
+	return "test1", nil
+}
+
+func (obj *Object) TestFunc5() (error, string) {
+	return nil, "test1"
+}
+func (obj *Object) TestFunc6() (string, error) {
+	return "test1", ErrNotFound
+}
+
+func TestReflectFieldMethodInterfaceInvalidFieldMethod1(t *testing.T) {
+	myObj := new(Object)
+	rcv, err := ReflectFieldMethodInterface(myObj, "TestFunc1")
+	if err != nil {
+		t.Error(err)
+	}
+	exp := "test"
+	if rcv != exp {
+		t.Errorf("Expected %v but received %v", exp, rcv)
+	}
+}
+
+func TestReflectFieldMethodInterfaceInvalidFieldMethod2(t *testing.T) {
+	myObj := new(Object)
+	_, err := ReflectFieldMethodInterface(myObj, "TestFunc2")
+	exp := "invalid function called"
+	if err == nil || err.Error() != exp {
+		t.Errorf("Expected %v but received %v", exp, err)
+	}
+
+}
+
+func TestReflectFieldMethodInterfaceInvalidFieldMethod3(t *testing.T) {
+	myObj := new(Object)
+	_, err := ReflectFieldMethodInterface(myObj, "TestFunc3")
+	exp := "invalid function called"
+	if err == nil || err.Error() != exp {
+		t.Errorf("Expected %v but received %v", exp, err)
+	}
+}
+
+func TestReflectFieldMethodInterfaceInvalidFieldMethod4(t *testing.T) {
+	myObj := new(Object)
+	rcv, err := ReflectFieldMethodInterface(myObj, "TestFunc4")
+	if err != nil {
+		t.Error(err)
+	}
+	exp := "test1"
+	if rcv != exp {
+		t.Errorf("Expected %v but received %v", exp, rcv)
+	}
+}
+
+func TestReflectFieldMethodInterfaceInvalidFieldMethod5(t *testing.T) {
+	myObj := new(Object)
+	_, err := ReflectFieldMethodInterface(myObj, "TestFunc5")
+	exp := "invalid function called"
+	if err == nil || err.Error() != exp {
+		t.Errorf("Expected %v but received %v", exp, err)
+	}
+}
+
+func TestReflectFieldMethodInterfaceInvalidFieldMethod6(t *testing.T) {
+	myObj := new(Object)
+	_, err := ReflectFieldMethodInterface(myObj, "TestFunc6")
+	exp := ErrNotFound
+	if err == nil || err != exp {
+		t.Errorf("Expected %v but received %v", exp, err)
+	}
+}