Don't use assert, throw a better exception for module name collisions

1 files changed, 3 insertions, 2 deletions

diff --git a/src/ModuleManager.py b/src/ModuleManager.py
index 24b5fb8c..d0c72782 100644
--- a/src/ModuleManager.py
+++ b/src/ModuleManager.py
@@ -99,8 +99,9 @@ class ModuleManager(object):
         module_object._context = context
         module_object._import_name = name
 
-        assert not module_object._name in self.modules, (
-            "module name '%s' attempted to be used twice.")
+        if name in self.modules:
+            raise ModuleNameCollisionException("Module name '%s' "
+                "attempted to be used twice")
         return module_object
 
     def load_module(self, name):