Oops, add test to make sure python requires don't work by default

dukpy/evaljs.py

@@ -37,7 +37,7 @@ def __init__(self, search_paths, enable_python=False):
         super(RequirableContext, self).__init__()
         self.search_paths = search_paths
         self.evaljs("Duktape.modSearch = dukpy.modSearch;", modSearch=self.require)
-        self.enable_python = True
+        self.enable_python = enable_python
 
     def require(self, id_, require, exports, module):
         # does the module ID begin with 'python/'
@@ -57,7 +57,7 @@ def require(self, id_, require, exports, module):
                 # assume this was probably No such file
                 continue
 
-        raise Exception("Unable to load {}".format(id_))
+        raise ImportError("Unable to load {}".format(id_))
 
     def require_python(self, pyid, require, exports, module):
         try:

tests/tests_evaljs.py

@@ -233,6 +233,19 @@ def test_python_require(self):
         c = dukpy.RequirableContext([], enable_python=True)
         assert c.evaljs("require('python/testpy').call()") == "Hello from Python!"
 
+    def test_python_require_doesnt_work_by_default(self):
+        import sys, imp
+        testpy = imp.new_module('testpy')
+        testpy.call = lambda: 'Hello from Python!'
+        testpy.__all__ = ['call']
+        sys.modules['testpy'] = testpy
+        c = dukpy.RequirableContext([])
+        try:
+            c.evaljs("require('python/testpy').call()") == "Hello from Python!"
+            assert False
+        except ImportError:
+            pass
+
 
 class TestRegressions(object):
     def test_pyargs_segfault(self):