Add MutablePair.ofNonNull(Map.Entry)

A variation of PR #1461

Author: garydgregory

src/changes/changes.xml

@@ -55,6 +55,7 @@ The <action> type attribute can be add,update,fix,remove.
     <action                   type="add" dev="ggregory" due-to="Gary Gregory">Add SystemProperties.getPath(String, Supplier&lt;Path&gt;).</action>
     <action                   type="add" dev="ggregory" due-to="Gary Gregory">Add JavaVersion.JAVA_25.</action>
     <action                   type="add" dev="ggregory" due-to="Gary Gregory">Add SystemUtils.IS_JAVA_25.</action>
+    <action                   type="add" dev="ggregory" due-to="jack5505, Gary Gregory">Add MutablePair.ofNonNull(Map.Entry).</action>
     <!-- UPDATE -->
     <action                   type="update" dev="ggregory" due-to="Gary Gregory">Bump org.apache.commons:commons-parent from 88 to 89.</action>
   </release>

src/main/java/org/apache/commons/lang3/tuple/MutablePair.java

@@ -114,6 +114,20 @@ public static <L, R> MutablePair<L, R> ofNonNull(final L left, final R right) {
         return of(Objects.requireNonNull(left, "left"), Objects.requireNonNull(right, "right"));
     }
 
+    /**
+     * Creates a mutable pair from a map entry.
+     *
+     * @param <L> the left element type
+     * @param <R> the right element type
+     * @param pair the existing map entry.
+     * @return a mutable pair formed from the map entry
+     * @throws NullPointerException if the pair is null.
+     * @since 3.20
+     */
+    public static <L, R> MutablePair<L, R> ofNonNull(final Map.Entry<L, R> pair) {
+        return of(Objects.requireNonNull(pair, "pair"));
+    }
+
     /** Left object. */
     public L left;
 

src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java

@@ -114,6 +114,15 @@ void testOfNonNull() {
         assertEquals("y", pair.right);
     }
 
+    @Test
+    void testOfNonNullMapEntry() {
+        assertNullPointerException(() -> MutablePair.ofNonNull(null));
+        final Pair<Integer, String> pair = Pair.of(0, "foo");
+        final MutablePair<Integer, String> mutablePair = MutablePair.ofNonNull(pair);
+        assertEquals(pair.getLeft(), mutablePair.getLeft());
+        assertEquals(pair.getRight(), mutablePair.getRight());
+    }
+
     @Test
     void testPairOfMapEntry() {
         assertNull(MutablePair.of(null).getLeft());