adding key lookup with value helper

Mzack9999 · Mzack9999 · commit 8b924a4a5dc0 · 2023-02-28T10:07:21.000+01:00
diff --git a/maps/generic_map.go b/maps/generic_map.go
@@ -1,7 +1,7 @@
 package mapsutil
 
 // Map wraps a generic map type
-type Map[K comparable, V any] map[K]V
+type Map[K, V comparable] map[K]V
 
 // Has checks if the current map has the provided key
 func (m Map[K, V]) Has(key K) bool {
@@ -32,3 +32,15 @@ func (m Map[K, V]) Merge(n map[K]V) {
 		m[k] = v
 	}
 }
+
+// GetKeyWithValue returns the first key having value
+func (m Map[K, V]) GetKeyWithValue(value V) (K, bool) {
+	var zero K
+	for k, v := range m {
+		if v == value {
+			return k, true
+		}
+	}
+
+	return zero, false
+}
diff --git a/maps/generic_map_test.go b/maps/generic_map_test.go
@@ -3,6 +3,8 @@ package mapsutil
 import (
 	"reflect"
 	"testing"
+
+	"github.com/stretchr/testify/require"
 )
 
 func TestMapHas(t *testing.T) {
@@ -66,3 +68,53 @@ func TestMapMerge(t *testing.T) {
 		t.Errorf("Merge(%v) = %v, expected %v", n, m, expected)
 	}
 }
+
+func TestMap_GetKeyWithValue(t *testing.T) {
+	type testCase[K, V comparable] struct {
+		InputMap    Map[K, V]
+		Value       V
+		ExpectedKey K
+		ExpectedOk  bool
+	}
+
+	genericMap := Map[string, string]{"a": "a", "b": "b", "c": "c"}
+
+	testCases := []testCase[string, string]{
+		{
+			InputMap:    genericMap,
+			Value:       "b",
+			ExpectedKey: "b",
+			ExpectedOk:  true,
+		},
+		{
+			InputMap:    genericMap,
+			Value:       "d",
+			ExpectedKey: "",
+			ExpectedOk:  false,
+		},
+		{
+			InputMap:    genericMap,
+			Value:       "b",
+			ExpectedKey: "b",
+			ExpectedOk:  true,
+		},
+		{
+			InputMap:    genericMap,
+			Value:       "d",
+			ExpectedKey: "",
+			ExpectedOk:  false,
+		},
+		{
+			InputMap:    genericMap,
+			Value:       "value",
+			ExpectedKey: "",
+			ExpectedOk:  false,
+		},
+	}
+
+	for _, tc := range testCases {
+		key, ok := tc.InputMap.GetKeyWithValue(tc.Value)
+		require.Equal(t, tc.ExpectedKey, key)
+		require.Equal(t, tc.ExpectedOk, ok)
+	}
+}
diff --git a/maps/synclock_map.go b/maps/synclock_map.go
@@ -11,7 +11,7 @@ var (
 )
 
 // SyncLock adds sync and lock capabilities to generic map
-type SyncLockMap[K comparable, V any] struct {
+type SyncLockMap[K, V comparable] struct {
 	ReadOnly atomic.Bool
 	mu       sync.RWMutex
 	Map      Map[K, V]

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ var (`
`11`	`11`	`)`
`12`	`12`
`13`	`13`	`// SyncLock adds sync and lock capabilities to generic map`
`14`		`-type SyncLockMap[K comparable, V any] struct {`
	`14`	`+type SyncLockMap[K, V comparable] struct {`
`15`	`15`	`ReadOnly atomic.Bool`
`16`	`16`	`mu sync.RWMutex`
`17`	`17`	`Map Map[K, V]`