Improve pretty-print formatting of negative amount of money (#4)

* Improve pretty-print formatting of negative amount of money

.php_cs

@@ -72,6 +72,7 @@ $config = PhpCsFixer\Config::create()
         'standardize_not_equals' => true,
         'strict_comparison' => true,
         'strict_param' => true,
+        'ternary_operator_spaces' => true,
         'ternary_to_null_coalescing' => true,
         'trailing_comma_in_multiline_array' => true,
         'trim_array_spaces' => true,

CHANGELOG.md

@@ -18,6 +18,11 @@ Security - in case of vulnerabilities.
 
 _TBD_
 
+## 1.0.1 (2019-03-05)
+
+### Fixed
++ Fixed pretty-print formatting of negative amount of money
+
 ## 1.0.0 (2018-12-08)
 
 Initial Release

src/Money.php

@@ -165,7 +165,11 @@ public function getFormattedAmount(string $decimalPoint = '.', string $thousands
      */
     public function getPrettyPrint(string $decimalPoint = '.', string $thousandsSeparator = ','): string
     {
-        return $this->getCurrency()->getSign() . $this->getFormattedAmount($decimalPoint, $thousandsSeparator);
+        $currencySign = $this->getCurrency()->getSign();
+        $formattedAmount = ltrim($this->getFormattedAmount($decimalPoint, $thousandsSeparator), '-');
+        $amountSign = $this->isNegative() ? '-' : '';
+
+        return "{$amountSign}{$currencySign}{$formattedAmount}";
     }
 
     /**

tests/Exchange/RateTest.php

@@ -15,13 +15,13 @@ public function testExceptionIsRaisedForInvalidConstructorArgument(): void
     {
         $cp = new CurrencyPair(new Currency('USD'), new Currency('EUR'));
         $this->expectException(InvalidArgumentException::class);
-        new Rate($cp, new DateTime(), 0);
+        new Rate($cp, $this->getRateDate(), 0);
     }
 
     public function testCanBeConstructed()
     {
         $cp = new CurrencyPair(new Currency('USD'), new Currency('EUR'));
-        $rate = new Rate($cp, new DateTime(), 0.92);
+        $rate = new Rate($cp, $this->getRateDate(), 0.92);
 
         $this->assertInstanceOf(Rate::class, $rate);
 
@@ -36,7 +36,7 @@ public function testCanBeConstructed()
     public function testCanBeSerialized(Rate $rate): void
     {
         $this->assertSame(
-            '{"baseCurrency":"USD","quoteCurrency":"EUR","date":"' . date('c') . '","ratio":0.92}',
+            '{"baseCurrency":"USD","quoteCurrency":"EUR","date":"' . $this->getRateDate()->format('c') . '","ratio":0.92}',
             json_encode($rate)
         );
     }
@@ -112,4 +112,9 @@ public function testCanConvertMoneyWithMarkup(Rate $rate): void
         $this->assertSame(966, $newMoney->getAmount());
         $this->assertSame('EUR', $newMoney->getCurrency()->getCurrencyCode());
     }
+
+    private function getRateDate(): DateTime
+    {
+        return new DateTime('2019-03-05 13:32');
+    }
 }

tests/MoneyTest.php

@@ -113,6 +113,13 @@ public function testFormattedAmountCanBeRetrieved(): void
         $this->assertSame('€1,200.34', $m->getPrettyPrint('.', ','));
     }
 
+    public function testFormattedNegativeAmountCanBeRetrieved(): void
+    {
+        $m = new Money(-120034, new Currency('EUR'));
+        $this->assertSame('-1 200.34', $m->getFormattedAmount('.', ' '));
+        $this->assertSame('-€1,200.34', $m->getPrettyPrint('.', ','));
+    }
+
     /**
      * @depends testObjectCanBeConstructedFromIntegerValueAndCurrencyObject
      *