Clear null length percentages without a warning (#58753)

Summary:
Pull Request resolved: #58753

Treat a null dynamic length-percentage value as the expected property-reset case instead of reporting it as an unsupported type. Preserve the existing null result while avoiding a warning whenever a percentage border radius is cleared.

Add focused coverage for point, percentage, negative, opt-in negative, and null parsing behavior.

Changelog:
[Android][Fixed] - Avoid warning when clearing percentage border radii

Differential Revision: D122278946

fbshipit-source-id: ce29b398cd3f35a798bb89bc44cf44f5e0ad7b7f

Author: Abbondanzo

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/LengthPercentage.kt

@@ -51,6 +51,7 @@ public data class LengthPercentage(
             null
           }
         }
+        ReadableType.Null -> null
         else -> {
           FLog.w(ReactConstants.TAG, "Unsupported type for radius property: ${dynamic.type}")
           null

packages/react-native/ReactAndroid/src/test/java/com/facebook/react/uimanager/LengthPercentageTest.kt

@@ -0,0 +1,47 @@
+/*
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+package com.facebook.react.uimanager
+
+import com.facebook.common.logging.FLog
+import com.facebook.react.bridge.DynamicFromObject
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.Test
+import org.mockito.Mockito.mockStatic
+
+class LengthPercentageTest {
+
+  @Test
+  fun parsesPointAndPercentageValues() {
+    assertThat(LengthPercentage.setFromDynamic(DynamicFromObject(12.0)))
+        .isEqualTo(LengthPercentage(12f, LengthPercentageType.POINT))
+    assertThat(LengthPercentage.setFromDynamic(DynamicFromObject("25%")))
+        .isEqualTo(LengthPercentage(25f, LengthPercentageType.PERCENT))
+  }
+
+  @Test
+  fun rejectsNegativeValuesByDefault() {
+    assertThat(LengthPercentage.setFromDynamic(DynamicFromObject(-1.0))).isNull()
+    assertThat(LengthPercentage.setFromDynamic(DynamicFromObject("-1%"))).isNull()
+  }
+
+  @Test
+  fun acceptsNegativeValuesWhenAllowed() {
+    assertThat(LengthPercentage.setFromDynamic(DynamicFromObject(-1.0), allowNegative = true))
+        .isEqualTo(LengthPercentage(-1f, LengthPercentageType.POINT))
+    assertThat(LengthPercentage.setFromDynamic(DynamicFromObject("-1%"), allowNegative = true))
+        .isEqualTo(LengthPercentage(-1f, LengthPercentageType.PERCENT))
+  }
+
+  @Test
+  fun clearsNullWithoutWarning() {
+    mockStatic(FLog::class.java).use { flog ->
+      assertThat(LengthPercentage.setFromDynamic(DynamicFromObject(null))).isNull()
+      flog.verifyNoInteractions()
+    }
+  }
+}