Fix wrong complex repr() for -0.0

dieterplex · dieterplex · commit 63b91a42c3b6 · 2022-03-13T13:50:03.000+08:00
Test: assert repr(complex('-0.0')) == '(-0+0j)'
diff --git a/Lib/test/test_complex.py b/Lib/test/test_complex.py
@@ -578,8 +578,6 @@ def test(v, expected, test_fn=self.assertEqual):
         self.assertEqual(-6j,complex(repr(-6j)))
         self.assertEqual(6j,complex(repr(6j)))
 
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
     @support.requires_IEEE_754
     def test_negative_zero_repr_str(self):
         def test(v, expected, test_fn=self.assertEqual):
@@ -636,8 +634,6 @@ def test_overflow(self):
         self.assertEqual(complex("-1e500j"), complex(0.0, -INF))
         self.assertEqual(complex("-1e500+1.8e308j"), complex(-INF, INF))
 
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
     @support.requires_IEEE_754
     def test_repr_roundtrip(self):
         vals = [0.0, 1e-500, 1e-315, 1e-200, 0.0123, 3.1415, 1e50, INF, NAN]
diff --git a/vm/src/builtins/complex.rs b/vm/src/builtins/complex.rs
@@ -322,9 +322,13 @@ impl PyComplex {
         };
         im_part.push('j');
 
-        // empty => return im_part, integer => drop ., fractional => float_ops
+        // positive empty => return im_part, integer => drop ., fractional => float_ops
         let re_part = if re == 0.0 {
-            return im_part;
+            if re.is_sign_positive() {
+                return im_part;
+            } else {
+                re.to_string()
+            }
         } else if re.fract() == 0.0 {
             re.to_string()
         } else {
