fix: evaluate integer percentage

calc/calc.py

@@ -79,7 +79,7 @@
 }
 
 SUFFIXES = {
-    '%': 0.01,
+    '%': lambda x: x * 0.01,
 }
 
 
@@ -112,24 +112,25 @@ def lower_dict(input_dict):
 
 def super_float(text):
     """
-    Like float, but with SI extensions. 1k goes to 1000.
+    Evaluate suffixes if applicable and apply float.
     """
-    if text[-1] in SUFFIXES:
-        return float(text[:-1]) * SUFFIXES[text[-1]]
-    else:
-        return float(text)
+    func = SUFFIXES[text[-1]]
+    return func(float(text[:-1]))
 
 
 def eval_number(parse_result):
     """
-    Create a float out of its string parts.
+    Returns an int or a float from string parts.
+    Calls super_float above for suffix evaluation.
 
     e.g. [ '7.13', 'e', '3' ] ->  7130
-    Calls super_float above.
     """
+    if parse_result[-1] in SUFFIXES:
+        return super_float("".join(parse_result))
+
     for item in parse_result:
         if "." in item or "e" in item or "E" in item:
-            return super_float("".join(parse_result))
+            return float("".join(parse_result))
 
     return int("".join(parse_result))
 

tests/test_calc.py

@@ -93,7 +93,11 @@ def test_si_suffix(self):
         For instance '%' stand for 1/100th so '1%' should be 0.01
         """
         test_mapping = [
-            ('4.2%', 0.042)
+            ('4.2%', 0.042),
+            ('1%', 0.01),
+            ('0.5%', 0.005),
+            ('70%', 0.7),
+            ('-50%', -0.5)
         ]
 
         for (expr, answer) in test_mapping: