Allow an empty tariff to be set

Author: mbardelmeijer

CHANGELOG.md

@@ -2,9 +2,13 @@
 
 All notable changes to `CMSMS` will be documented in this file
 
+## [0.0.5] - 2016-09-29
+#### Fixed
+- Allow an empty `tariff` to be set.
+
 ## [0.0.4] - 2016-09-13
 #### Fixed
-- Moved the `tariff` field under the correct XML child
+- Moved the `tariff` field under the correct XML child.
 
 ## [0.0.3] - 2016-09-12
 #### Added

src/CmsmsMessage.php

@@ -82,7 +82,7 @@ public function reference($reference)
      */
     public function tariff($tariff)
     {
-        if (empty($tariff) || ! is_int($tariff)) {
+        if (! is_int($tariff)) {
             throw InvalidMessage::invalidTariff($tariff);
         }
 

tests/CmsmsMessageTest.php

@@ -106,11 +106,11 @@ public function it_can_set_tariff()
     }
 
     /** @test */
-    public function it_cannot_set_an_empty_tariff()
+    public function it_can_set_an_empty_tariff()
     {
-        $this->setExpectedException(InvalidMessage::class);
+        $message = (new CmsmsMessage)->tariff(0);
 
-        (new CmsmsMessage)->tariff(0);
+        $this->assertEquals(0, $message->getTariff());
     }
 
     /** @test */