Skip to content

Conversation

@dhavlik
Copy link

@dhavlik dhavlik commented Sep 14, 2017

Increase code coverage to nearly 100%.

@dhavlik dhavlik requested a review from icemac September 14, 2017 11:20
Copy link
Member

@icemac icemac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM besides a too long line which makes Flake8 unhappy.



def test_Eval__RestictionCapableEval__prepRestrictedCode_1():
"""It does nothing when restricted code is already set by prepRestrictedCode."""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line seems to be too long.

@dhavlik dhavlik merged commit 4d7907b into master Sep 15, 2017
@dhavlik dhavlik deleted the increase-code-coverage-of-eval-module branch September 15, 2017 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants