Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
ff3997e7
Commit
ff3997e7
authored
May 31, 2013
by
Peter Baratta
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Add test to verify a single period is not accepted as input
parent
08783433
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
8 additions
and
1 deletions
+8
-1
common/lib/calc/calc.py
+1
-1
common/lib/calc/tests/test_calc.py
+7
-0
No files found.
common/lib/calc/calc.py
View file @
ff3997e7
...
...
@@ -182,7 +182,7 @@ def evaluator(variables, functions, string, cs=False):
number_part
=
Word
(
nums
)
# 0.33 or 7 or .34
# 0.33 or 7 or .34
or 16.
inner_number
=
(
number_part
+
Optional
(
"."
+
Optional
(
number_part
)))
|
(
"."
+
number_part
)
# 0.33k or -17
...
...
common/lib/calc/tests/test_calc.py
View file @
ff3997e7
...
...
@@ -37,6 +37,13 @@ class EvaluatorTest(unittest.TestCase):
self
.
assertEqual
(
easy_eval
(
"-3.14"
),
-
3.14
)
self
.
assertEqual
(
easy_eval
(
"-.618033989"
),
-
0.618033989
)
def
test_period
(
self
):
"""
The string '.' should not evaluate to anything.
"""
self
.
assertRaises
(
ParseException
,
calc
.
evaluator
,
{},
{},
'.'
)
self
.
assertRaises
(
ParseException
,
calc
.
evaluator
,
{},
{},
'1+.'
)
def
test_trailing_period
(
self
):
"""
Test that things like '4.' will be 4 and not throw an error
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment