Commitfe7fa6c

committed

Merge pull request#348 from mlauter/fix-347

2 parentsb770323 +ce4ab58 commitfe7fa6cCopy full SHA for fe7fa6c

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -342,7 +342,7 @@ def safe_eval(expr, namespace):`
`342`	`342`	`try:`
`343`	`343`	`obj=eval(expr,namespace)`
`344`	`344`	`returnobj`
`345`		`-except (NameError,AttributeError)ase:`
	`345`	`+except (NameError,AttributeError,SyntaxError)ase:`
`346`	`346`	`# If debugging safe_eval, raise this!`
`347`	`347`	`# raise e`
`348`	`348`	`returnSafeEvalFailed`

Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,12 @@ def test_cw(self):`
`34`	`34`	`self.assertEqual(self.repl.cw(), 'datetime')`
`35`	`35`	`"""`
`36`	`36`
	`37`	`+classTestSafeEval(unittest.TestCase):`
	`38`	`+deftest_catches_syntax_error(self):`
	`39`	`+try:`
	`40`	`+autocomplete.safe_eval('1re',{})`
	`41`	`+except:`
	`42`	`+self.fail('safe_eval raises an error')`
`37`	`43`
`38`	`44`	`# make some fake files? Dependency inject? mock?`
`39`	`45`	`classTestFilenameCompletion(unittest.TestCase):`

Comments

(0)