-
Notifications
You must be signed in to change notification settings - Fork 8
Open
Description
According to
pysqa/src/pysqa/base/validate.py
Lines 34 to 37 in 153d293
| if isinstance(memory_max, (int, float)): | |
| memory_max = value_in_range( | |
| value=memory_max, value_max=active_queue["memory_max"] | |
| ) |
the value of the memory is not checked if it is a string, however,
pysqa/src/pysqa/base/validate.py
Lines 75 to 83 in 153d293
| if value is not None: | |
| value_, value_min_, value_max_ = [ | |
| ( | |
| _memory_spec_string_to_value(v) | |
| if v is not None and isinstance(v, str) | |
| else v | |
| ) | |
| for v in (value, value_min, value_max) | |
| ] |
looks like it capable of checking this!
Indeed, I would like any string for a memory not passing the _is_memory_string check to be rejected by pysqa.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels