Skip to content

Commit

Permalink
Add the parse_float builtin function
Browse files Browse the repository at this point in the history
  • Loading branch information
zeroSteiner committed Jul 8, 2023
1 parent 2b511e3 commit 320cb0d
Show file tree
Hide file tree
Showing 6 changed files with 43 additions and 22 deletions.
14 changes: 14 additions & 0 deletions lib/rule_engine/_utils.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
import ast as pyast
import datetime
import decimal
import re

from . import errors
Expand Down Expand Up @@ -26,6 +28,18 @@ def parse_datetime(string, default_timezone):
dt = dt.replace(tzinfo=default_timezone)
return dt

def parse_float(string):
if re.match('^0[0-9]', string):
raise errors.RuleSyntaxError('invalid floating point literal: ' + string + ' (leading zeros in decimal literals are not permitted)')
try:
if re.match('^0[box]', string):
val = decimal.Decimal(pyast.literal_eval(string))
else:
val = decimal.Decimal(string)
except Exception:
raise errors.RuleSyntaxError('invalid floating point literal: ' + string) from None
return val

def parse_timedelta(periodstring):
if periodstring == "P":
raise errors.TimedeltaSyntaxError('empty timedelta string', periodstring)
Expand Down
9 changes: 6 additions & 3 deletions lib/rule_engine/ast.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
import re

from . import errors
from ._utils import parse_datetime, parse_timedelta
from ._utils import parse_datetime, parse_float, parse_timedelta
from .suggestions import suggest_symbol
from .types import *

Expand Down Expand Up @@ -251,8 +251,7 @@ class TimedeltaExpression(LiteralExpressionBase):
result_type = DataType.TIMEDELTA
@classmethod
def from_string(cls, context, string):
dt = parse_timedelta(string)
return cls(context, dt)
return cls(context, parse_timedelta(string))

class FloatExpression(LiteralExpressionBase):
"""Literal float expressions representing numerical values."""
Expand All @@ -261,6 +260,10 @@ def __init__(self, context, value, **kwargs):
value = coerce_value(value)
super(FloatExpression, self).__init__(context, value, **kwargs)

@classmethod
def from_string(cls, context, string):
return cls(context, parse_float(string))

class MappingExpression(LiteralExpressionBase):
"""Literal mapping expression representing a set of associations between keys and values."""
result_type = DataType.MAPPING
Expand Down
4 changes: 3 additions & 1 deletion lib/rule_engine/builtins.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
import math
import random

from ._utils import parse_datetime, parse_timedelta
from ._utils import parse_datetime, parse_float, parse_timedelta
from . import ast
from . import errors
from . import types
Expand Down Expand Up @@ -161,6 +161,7 @@ def from_defaults(cls, values=None, **kwargs):
'min': min,
'filter': _builtin_filter,
'parse_datetime': BuiltinValueGenerator(lambda builtins: functools.partial(_builtin_parse_datetime, builtins)),
'parse_float': parse_float,
'parse_timedelta': parse_timedelta,
'random': _builtin_random,
'split': _builtins_split
Expand All @@ -182,6 +183,7 @@ def from_defaults(cls, values=None, **kwargs):
'min': ast.DataType.FUNCTION('min', return_type=ast.DataType.FLOAT, argument_types=(ast.DataType.ARRAY(ast.DataType.FLOAT),)),
'filter': ast.DataType.FUNCTION('filter', argument_types=(ast.DataType.FUNCTION, ast.DataType.ARRAY)),
'parse_datetime': ast.DataType.FUNCTION('parse_datetime', return_type=ast.DataType.DATETIME, argument_types=(ast.DataType.STRING,)),
'parse_float': ast.DataType.FUNCTION('parse_float', return_type=ast.DataType.FLOAT, argument_types=(ast.DataType.STRING,)),
'parse_timedelta': ast.DataType.FUNCTION('parse_timedelta', return_type=ast.DataType.TIMEDELTA, argument_types=(ast.DataType.STRING,)),
'random': ast.DataType.FUNCTION('random', return_type=ast.DataType.FLOAT, argument_types=(ast.DataType.FLOAT,), minimum_arguments=0),
'split': ast.DataType.FUNCTION(
Expand Down
2 changes: 1 addition & 1 deletion lib/rule_engine/errors.py
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ def __init__(self, message, error, value):
"""The regular expression value which contains the syntax error which caused this exception to be raised."""

class RuleSyntaxError(SyntaxError):
"""An error raised for issues identified in while parsing the grammar of the rule text."""
"""An error raised for issues identified while parsing the grammar of the rule text."""
def __init__(self, message, token=None):
"""
:param str message: A text description of what error occurred.
Expand Down
23 changes: 6 additions & 17 deletions lib/rule_engine/parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@

import ast as pyast
import collections
import re
import threading
import types as pytypes

Expand Down Expand Up @@ -461,23 +460,13 @@ def p_expression_timedelta(self, p):
p[0] = _DeferredAstNode(ast.TimedeltaExpression, args=(self.context, p[1]), method='from_string')

def p_expression_float(self, p):
'expression : FLOAT'
"""
expression : FLOAT
| FLOAT_NAN
| FLOAT_INF
"""
str_val = p[1]
if re.match('^0[0-9]', str_val):
raise errors.RuleSyntaxError('invalid floating point literal: ' + str_val + ' (leading zeros in decimal literals are not permitted)')
try:
val = literal_eval(str_val)
except SyntaxError:
raise errors.RuleSyntaxError('invalid floating point literal: ' + str_val)
p[0] = _DeferredAstNode(ast.FloatExpression, args=(self.context, float(val)))

def p_expression_float_nan(self, p):
'expression : FLOAT_NAN'
p[0] = _DeferredAstNode(ast.FloatExpression, args=(self.context, float('nan')))

def p_expression_float_inf(self, p):
'expression : FLOAT_INF'
p[0] = _DeferredAstNode(ast.FloatExpression, args=(self.context, float('inf')))
p[0] = _DeferredAstNode(ast.FloatExpression, args=(self.context, str_val), method='from_string')

def p_expression_null(self, p):
'object : NULL'
Expand Down
13 changes: 13 additions & 0 deletions tests/builtins.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@

import contextlib
import datetime
import decimal
import random
import string
import unittest
Expand Down Expand Up @@ -158,6 +159,18 @@ def test_engine_builtins_function_parse_datetime(self):
with self.assertRaises(errors.DatetimeSyntaxError):
self.assertBuiltinFunction('parse_datetime', now, '')

def test_engine_builtins_function_parse_float(self):
self.assertBuiltinFunction('parse_float', 1, '1')
self.assertBuiltinFunction('parse_float', 0b10, '0b10')
self.assertBuiltinFunction('parse_float', 0o10, '0o10')
self.assertBuiltinFunction('parse_float', 0x10, '0x10')
self.assertBuiltinFunction('parse_float', decimal.Decimal('1.1'), '1.1')
self.assertBuiltinFunction('parse_float', 1e1, '1e1')
self.assertBuiltinFunction('parse_float', float('inf'), 'inf')
self.assertBuiltinFunction('parse_float', -1, '-1')
with self.assertRaises(errors.RuleSyntaxError):
self.assertBuiltinFunction('parse_float', 1, 'f00d')

def test_engine_builtins_function_parse_timedelta(self):
self.assertBuiltinFunction('parse_timedelta', datetime.timedelta(days=1), 'P1D')
with self.assertRaises(errors.TimedeltaSyntaxError):
Expand Down

0 comments on commit 320cb0d

Please sign in to comment.