Do not throw error if alterFunc does not change the original key (#249

)

* Do not throw error if alterfunc does not change the key

* test case explanation

* Update mapstr/mapstr_test.go

Co-authored-by: Denis <[email protected]>

* update

---------

Co-authored-by: Denis <[email protected]>

mapstr/mapstr.go

@@ -218,6 +218,12 @@ func (m M) AlterPath(path string, mode TraversalMode, alterFunc AlterFunc) (err
 		if newKey == "" {
 			return fmt.Errorf("replacement key for %q cannot be empty", key)
 		}
+
+		// if altered key is equal to the original key, skip below delete/put func
+		if newKey == key {
+			return nil
+		}
+
 		_, exists := level[newKey]
 		if exists {
 			return fmt.Errorf("replacement key %q already exists: %w", newKey, ErrKeyCollision)
@@ -271,7 +277,7 @@ func (m M) Traverse(path string, mode TraversalMode, visitor TraversalVisitor) (
 
 	for i, segment := range segments {
 		if !found {
-			return ErrKeyNotFound
+			return fmt.Errorf("could not fetch value for key: %s, Error: %w ", path, ErrKeyNotFound)
 		}
 		found = false
 
@@ -316,7 +322,7 @@ func (m M) Traverse(path string, mode TraversalMode, visitor TraversalVisitor) (
 	}
 
 	if !found {
-		return ErrKeyNotFound
+		return fmt.Errorf("could not fetch value for key: %s, Error: %w", path, ErrKeyNotFound)
 	}
 
 	return nil

mapstr/mapstr_test.go

@@ -1264,6 +1264,22 @@ func TestAlterPath(t *testing.T) {
 				},
 			},
 		},
+		{
+			name:      "does not return an error if alterFunc returns the key unchanged",
+			from:      "level1_field1.key",
+			mode:      CaseInsensitiveMode,
+			alterFunc: lower,
+			m: M{
+				"level1_field1": M{
+					"Key": "value1",
+				},
+			},
+			exp: M{
+				"level1_field1": M{ //first level is unchanged - already in lowercase
+					"key": "value1",
+				},
+			},
+		},
 		{
 			name:      "alters keys on all nested levels with case-insensitive matching",
 			from:      "level1_field1.level2_field1.level3_field1",