ci: Fix flaky float printing

Signed-off-by: Johannes Zottele <[email protected]>

Author: Jozott00

.github/workflows/protobuf-conformance.yml

@@ -22,7 +22,6 @@ jobs:
       - name: Check if protobuf-conformance test is up-to-date
         run: |
           if [[ -n "$(git status --porcelain | grep tests/protobuf-conformance/)" ]]; then
-            git diff -- tests/protobuf-conformance/ || true
             echo "Protobuf conformance test is not up to date. Please run './gradlew tests:protobuf-conformance:bufGenerateMain' and commit changes"
             exit 1
           fi

protoc-gen/protobuf/src/main/kotlin/kotlinx/rpc/protoc/gen/ModelToProtobufKotlinCommonGenerator.kt

@@ -933,22 +933,18 @@ class ModelToProtobufKotlinCommonGenerator(
                 java.lang.Long.toUnsignedString(value) + "uL"
             }
 
-            value is Float -> {
-                when (value.toString()) {
-                    "Infinity" -> "Float.POSITIVE_INFINITY"
-                    "-Infinity" -> "Float.NEGATIVE_INFINITY"
-                    "NaN" -> "Float.NaN"
-                    else -> value.toString() + "f"
-                }
+            value is Float -> when {
+                value.isNaN() -> "Float.NaN"
+                value == Float.POSITIVE_INFINITY -> "Float.POSITIVE_INFINITY"
+                value == Float.NEGATIVE_INFINITY -> "Float.NEGATIVE_INFINITY"
+                else -> "Float.fromBits(0x%08X)".format(java.lang.Float.floatToRawIntBits(value))
             }
 
-            value is Double -> {
-                when (value.toString()) {
-                    "Infinity" -> "Double.POSITIVE_INFINITY"
-                    "-Infinity" -> "Double.NEGATIVE_INFINITY"
-                    "NaN" -> "Double.NaN"
-                    else -> value.toString()
-                }
+            value is Double -> when {
+                value.isNaN() -> "Double.NaN"
+                value == Double.POSITIVE_INFINITY -> "Double.POSITIVE_INFINITY"
+                value == Double.NEGATIVE_INFINITY -> "Double.NEGATIVE_INFINITY"
+                else -> "Double.fromBits(0x%016XL)".format(java.lang.Double.doubleToRawLongBits(value))
             }
 
             else -> {

tests/protobuf-conformance/src/main/generated-code/kotlin-multiplatform/com/google/protobuf_test_messages/editions/proto2/_rpc_internal/TestMessagesProto2Editions.kt

@@ -178,8 +178,8 @@ class TestAllTypesProto2Internal: com.google.protobuf_test_messages.editions.pro
     override var defaultFixed64: ULong by MsgFieldDelegate(PresenceIndices.defaultFixed64) { 10123456789123456789uL }
     override var defaultSfixed32: Int by MsgFieldDelegate(PresenceIndices.defaultSfixed32) { -123456789 }
     override var defaultSfixed64: Long by MsgFieldDelegate(PresenceIndices.defaultSfixed64) { -9123456789123456789L }
-    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { 9.0E9f }
-    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { 7.0E22 }
+    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { Float.fromBits(0x50061C46) }
+    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { Double.fromBits(0x44ADA56A4B0835C0L) }
     override var defaultBool: Boolean by MsgFieldDelegate(PresenceIndices.defaultBool) { true }
     override var defaultString: String by MsgFieldDelegate(PresenceIndices.defaultString) { "Rosebud" }
     override var defaultBytes: ByteArray by MsgFieldDelegate(PresenceIndices.defaultBytes) { BytesDefaults.defaultBytes }
@@ -1148,8 +1148,8 @@ class TestAllRequiredTypesProto2Internal: com.google.protobuf_test_messages.edit
     override var defaultFixed64: ULong by MsgFieldDelegate(PresenceIndices.defaultFixed64) { 10123456789123456789uL }
     override var defaultSfixed32: Int by MsgFieldDelegate(PresenceIndices.defaultSfixed32) { -123456789 }
     override var defaultSfixed64: Long by MsgFieldDelegate(PresenceIndices.defaultSfixed64) { -9123456789123456789L }
-    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { 9.0E9f }
-    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { 7.0E22 }
+    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { Float.fromBits(0x50061C46) }
+    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { Double.fromBits(0x44ADA56A4B0835C0L) }
     override var defaultBool: Boolean by MsgFieldDelegate(PresenceIndices.defaultBool) { true }
     override var defaultString: String by MsgFieldDelegate(PresenceIndices.defaultString) { "Rosebud" }
     override var defaultBytes: ByteArray by MsgFieldDelegate(PresenceIndices.defaultBytes) { BytesDefaults.defaultBytes }

tests/protobuf-conformance/src/main/generated-code/kotlin-multiplatform/com/google/protobuf_test_messages/proto2/_rpc_internal/TestMessagesProto2.kt

@@ -178,8 +178,8 @@ class TestAllTypesProto2Internal: com.google.protobuf_test_messages.proto2.TestA
     override var defaultFixed64: ULong by MsgFieldDelegate(PresenceIndices.defaultFixed64) { 10123456789123456789uL }
     override var defaultSfixed32: Int by MsgFieldDelegate(PresenceIndices.defaultSfixed32) { -123456789 }
     override var defaultSfixed64: Long by MsgFieldDelegate(PresenceIndices.defaultSfixed64) { -9123456789123456789L }
-    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { 9.0E9f }
-    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { 7.0E22 }
+    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { Float.fromBits(0x50061C46) }
+    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { Double.fromBits(0x44ADA56A4B0835C0L) }
     override var defaultBool: Boolean by MsgFieldDelegate(PresenceIndices.defaultBool) { true }
     override var defaultString: String by MsgFieldDelegate(PresenceIndices.defaultString) { "Rosebud" }
     override var defaultBytes: ByteArray by MsgFieldDelegate(PresenceIndices.defaultBytes) { BytesDefaults.defaultBytes }
@@ -1148,8 +1148,8 @@ class TestAllRequiredTypesProto2Internal: com.google.protobuf_test_messages.prot
     override var defaultFixed64: ULong by MsgFieldDelegate(PresenceIndices.defaultFixed64) { 10123456789123456789uL }
     override var defaultSfixed32: Int by MsgFieldDelegate(PresenceIndices.defaultSfixed32) { -123456789 }
     override var defaultSfixed64: Long by MsgFieldDelegate(PresenceIndices.defaultSfixed64) { -9123456789123456789L }
-    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { 9.0E9f }
-    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { 7.0E22 }
+    override var defaultFloat: Float by MsgFieldDelegate(PresenceIndices.defaultFloat) { Float.fromBits(0x50061C46) }
+    override var defaultDouble: Double by MsgFieldDelegate(PresenceIndices.defaultDouble) { Double.fromBits(0x44ADA56A4B0835C0L) }
     override var defaultBool: Boolean by MsgFieldDelegate(PresenceIndices.defaultBool) { true }
     override var defaultString: String by MsgFieldDelegate(PresenceIndices.defaultString) { "Rosebud" }
     override var defaultBytes: ByteArray by MsgFieldDelegate(PresenceIndices.defaultBytes) { BytesDefaults.defaultBytes }