added support for general cases where the amount is 0 or negative

- there are use cases (CAPI) where we need this library to return a 0 or
  negative byte amount
- this will allow other consumers of Palm Civet to handle these
  scenarios on a case-by-case basis

Signed-off-by: Tim Downey <[email protected]>

lib/palm_civet.rb

@@ -65,9 +65,6 @@ def self.to_bytes(bytes)
     end
 
     value = Float(matches[1])
-    if value <= 0
-      raise InvalidByteQuantityError
-    end
 
     case matches[2][0].capitalize
     when "T"

spec/palm_civet_spec.rb

@@ -60,33 +60,33 @@
       expect(PalmCivet.to_bytes("\t\n\r 5MB ")).to eq(5 * PalmCivet::MEGABYTE)
     end
 
-    context "raises an error" do
-      it "when the unit is missing" do
-	expect {
-	  PalmCivet.to_bytes("5")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+    context 'when the byte amount is 0' do
+      it 'returns 0 bytes' do
+        expect(PalmCivet.to_bytes("0TB")).to eq(0)
       end
+    end
 
-      it "when the unit is unrecognized" do
-	expect {
-	  PalmCivet.to_bytes("5MBB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
-
-	expect {
-	  PalmCivet.to_bytes("5BB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+    context 'when the byte amount is negative' do
+      it 'returns a negative amount of bytes' do
+        expect(PalmCivet.to_bytes('-200B')).to eq(-200)
       end
+    end
 
-      it "for negative values" do
-	expect {
-	  PalmCivet.to_bytes("-5MB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+    context "raises an error" do
+      it "when the unit is missing" do
+        expect {
+          PalmCivet.to_bytes("5")
+        }.to raise_error(PalmCivet::InvalidByteQuantityError)
       end
 
-      it "for zero values" do
-	expect {
-	  PalmCivet.to_bytes("0TB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+      it "when the unit is unrecognized" do
+        expect {
+          PalmCivet.to_bytes("5MBB")
+        }.to raise_error(PalmCivet::InvalidByteQuantityError)
+
+        expect {
+          PalmCivet.to_bytes("5BB")
+        }.to raise_error(PalmCivet::InvalidByteQuantityError)
       end
     end
   end
@@ -123,33 +123,33 @@
       expect(PalmCivet.to_megabytes("\t\n\r 5MB ")).to eq(5)
     end
 
-    context "raises an error" do
-      it "when the unit is missing" do
-	expect {
-	  PalmCivet.to_megabytes("5")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+    context 'when the byte amount is 0' do
+      it 'returns 0 megabytes' do
+        expect(PalmCivet.to_megabytes('0TB')).to eq(0)
       end
+    end
 
-      it "when the unit is unrecognized" do
-	expect {
-	  PalmCivet.to_megabytes("5MBB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
-
-	expect {
-	  PalmCivet.to_megabytes("5BB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+    context 'when the byte amount is negative' do
+      it 'returns a negative amount of megabytes' do
+        expect(PalmCivet.to_megabytes('-200MB')).to eq(-200)
       end
+    end
 
-      it "for negative values" do
-	expect {
-	  PalmCivet.to_megabytes("-5MB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+    context "raises an error" do
+      it "when the unit is missing" do
+        expect {
+          PalmCivet.to_megabytes("5")
+        }.to raise_error(PalmCivet::InvalidByteQuantityError)
       end
 
-      it "for zero values" do
-	expect {
-	  PalmCivet.to_megabytes("0TB")
-	}.to raise_error(PalmCivet::InvalidByteQuantityError)
+      it "when the unit is unrecognized" do
+        expect {
+          PalmCivet.to_megabytes("5MBB")
+        }.to raise_error(PalmCivet::InvalidByteQuantityError)
+
+        expect {
+          PalmCivet.to_megabytes("5BB")
+        }.to raise_error(PalmCivet::InvalidByteQuantityError)
       end
     end
   end