Merge pull request #39 from zopefoundation/compile_tests

Compile tests of invalid options

.coveragerc

@@ -1,6 +1,11 @@
 [run]
 branch = True
 source = RestrictedPython
+omit =
+    src/RestrictedPython/tests
+    src/RestrictedPython/tests/*.py
+    tests/*.py
+    bootstrap.py
 
 [report]
 precision = 3

.gitignore

@@ -5,6 +5,7 @@ pip-selfcheck.json
 pyvenv.cfg
 /.python-version
 /*.egg-info
+/.eggs/
 /.Python
 /.cache
 /.installed.cfg
@@ -20,6 +21,7 @@ pyvenv.cfg
 /eggs
 /fake-eggs
 /htmlcov
+/report-*.html
 /include
 /lib
 /share

docs/conf.py

@@ -49,7 +49,7 @@
 
 # General information about the project.
 project = u'RestrictedPython'
-copyright = u'2016, Zope Foundation'
+copyright = u'2017, Zope Foundation and Contributors'
 author = u'Alexander Loechel'
 
 # The version info for the project you're documenting, acts as replacement for

setup.cfg

@@ -10,15 +10,31 @@ upload-dir = docs/html
 ignore =
     .travis.yml
     bootstrap-buildout.py
+    bootstrap.py
     buildout.cfg
     jenkins.cfg
     travis.cfg
 
+[aliases]
+test = pytest
+
+[tool:pytest]
+addopts =
+testpaths =
+    tests
+    src/RestrictedPython/tests
+norecursedirs = fixures
+
+isort_ignore =
+    bootstrap.py
+
+
 [isort]
 force_alphabetical_sort = True
 force_single_line = True
 lines_after_imports = 2
 line_length = 200
+skip = bootstrap.py
 not_skip = __init__.py
 
 [flake8]

setup.py

@@ -49,6 +49,13 @@ def read(*rnames):
       package_dir={'': 'src'},
       install_requires=[
           'setuptools',
+
+      ],
+      setup_requires=[
+          'pytest-runner',
+      ],
+      test_requires=[
+          'pytest',
       ],
       extras_require={
           'docs': [
@@ -57,9 +64,6 @@ def read(*rnames):
           'release': [
               'zest.releaser',
           ],
-          'test': [
-              'pytest',
-          ],
           'develop': [
               'pdbpp',
               'isort',

src/RestrictedPython/compile.py

@@ -107,10 +107,11 @@ def compile_restricted_single(
 
 
 def compile_restricted_function(
-        source,
+        p,
+        body,
+        name,
         filename='<string>',
-        flags=0,
-        dont_inherit=0,
+        globalize=None,
         policy=RestrictingNodeTransformer):
     """Compile a restricted code object for a function.
 
@@ -121,17 +122,9 @@ def compile_restricted_function(
     The globalize argument, if specified, is a list of variable names to be
     treated as globals (code is generated as if each name in the list
     appeared in a global statement at the top of the function).
-
-    TODO: Special function not comparable with the other restricted_compile_*
-    functions.
     """
-    return _compile_restricted_mode(
-        source,
-        filename=filename,
-        mode='function',
-        flags=flags,
-        dont_inherit=dont_inherit,
-        policy=policy)
+    # TODO: Special function not comparable with the other restricted_compile_* functions.  # NOQA
+    return None
 
 
 def compile_restricted(

src/RestrictedPython/transformer.py

@@ -627,9 +627,12 @@ def visit_Expr(self, node):
 
     def visit_UnaryOp(self, node):
         """
-
+        UnaryOp (Unary Operations) is the overall element for:
+        * Not --> which should be allowed
+        * UAdd
+        * USub
         """
-        self.not_allowed(node)
+        return self.node_contents_visit(node)
 
     def visit_UAdd(self, node):
         """
@@ -645,9 +648,9 @@ def visit_USub(self, node):
 
     def visit_Not(self, node):
         """
-
+        The Not Operator should be allowed.
         """
-        self.not_allowed(node)
+        return self.node_contents_visit(node)
 
     def visit_Invert(self, node):
         """

tests/builtins/test_limits.py

@@ -0,0 +1,72 @@
+from RestrictedPython.Limits import _limited_list
+from RestrictedPython.Limits import _limited_range
+from RestrictedPython.Limits import _limited_tuple
+
+import pytest
+
+
+def test__limited_range_length_1():
+    result = _limited_range(1)
+    assert result == range(0, 1)
+
+
+def test__limited_range_length_10():
+    result = _limited_range(10)
+    assert result == range(0, 10)
+
+
+def test__limited_range_5_10():
+    result = _limited_range(5, 10)
+    assert result == range(5, 10)
+
+
+def test__limited_range_5_10_sm1():
+    result = _limited_range(5, 10, -1)
+    assert result == range(5, 10, -1)
+
+
+def test__limited_range_15_10_s2():
+    result = _limited_range(15, 10, 2)
+    assert result == range(15, 10, 2)
+
+
+def test__limited_range_no_input():
+    with pytest.raises(TypeError):
+        _limited_range()
+
+
+def test__limited_range_more_steps():
+    with pytest.raises(AttributeError):
+        _limited_range(0, 0, 0, 0)
+
+
+def test__limited_range_zero_step():
+    with pytest.raises(ValueError):
+        _limited_range(0, 10, 0)
+
+
+def test__limited_range_range_overflow():
+    with pytest.raises(ValueError):
+        _limited_range(0, 5000, 1)
+
+
+def test__limited_list_valid_list_input():
+    input = [1, 2, 3]
+    result = _limited_list(input)
+    assert result == input
+
+
+def test__limited_list_invalid_string_input():
+    with pytest.raises(TypeError):
+        _limited_list('input')
+
+
+def test__limited_tuple_valid_list_input():
+    input = [1, 2, 3]
+    result = _limited_tuple(input)
+    assert result == tuple(input)
+
+
+def test__limited_tuple_invalid_string_input():
+    with pytest.raises(TypeError):
+        _limited_tuple('input')