KalleOlaviNiemitalo commented on code in PR #3423:
URL: https://github.com/apache/avro/pull/3423#discussion_r2188160570


##########
lang/csharp/src/apache/test/AvroDecimalTest.cs:
##########
@@ -38,7 +38,19 @@ public void TestAvroDecimalToString(decimal value)
         {
             var valueString = value.ToString();
 
-            var avroDecimal = new AvroDecimal(value);            
+            var avroDecimal = new AvroDecimal(value);
+            var avroDecimalString = avroDecimal.ToString();
+
+            Assert.AreEqual(valueString, avroDecimalString);
+        }
+
+        [Test]
+        public void TestHighPrecisionAvroDecimalToString()
+        {
+            var value = 4.1748330066797328106875724512m; // High precision 
decimal value

Review Comment:
   Can you add a test of converting a negative `decimal` to AvroDecimal?  Even 
though this PR doesn't change how that works.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to