Updated module reload test.

Avoid to load a module twice.

Fixes bug #734.

Reviewer: Marcelo Lira <marcelo.lira@openbossa.org>
          Luciano Wolf <luciano.wolf@openbossa.org>

2 files changed, 7 insertions, 21 deletions

diff --git a/tests/samplebinding/module_reload_test.py b/tests/samplebinding/module_reload_test.py
index 17551bb86..48d6d9572 100644
--- a/tests/samplebinding/module_reload_test.py
+++ b/tests/samplebinding/module_reload_test.py
@@ -10,34 +10,18 @@ dst = os.path.join(workdir, 'test_module.py')
 shutil.copyfile(src, dst)
 sys.path.append(workdir)
 
-def increment_module_value():
-    modfile = open(dst, 'a')
-    modfile.write('MyOtherObjectType.value += 1' + os.linesep)
-    modfile.flush()
-    modfile.close()
-    try:
-        os.remove(dst + 'c')
-    except:
-        os.remove(dst + 'o')
-
 class TestModuleReloading(unittest.TestCase):
 
     def testModuleReloading(self):
         '''Test module reloading with on-the-fly modifications.'''
 
         import test_module
-        self.assertEqual(test_module.MyOtherObjectType.value, 10)
-
-        increment_module_value()
-        reload(sys.modules['test_module'])
-        self.assertEqual(test_module.MyOtherObjectType.value, 11)
-
-        reload(sys.modules['test_module'])
-        self.assertEqual(test_module.MyOtherObjectType.value, 11)
 
-        increment_module_value()
-        reload(sys.modules['test_module'])
-        self.assertEqual(test_module.MyOtherObjectType.value, 12)
+	for i in range(3):
+            oldObject = test_module.obj
+            self.assertTrue(oldObject is test_module.obj)
+            reload(test_module)
+            self.assertFalse(oldObject is test_module.obj)
 
 if __name__ == "__main__":
     unittest.main()
diff --git a/tests/samplebinding/test_module_template.py b/tests/samplebinding/test_module_template.py
index b2e917de4..b6cfb8389 100644
--- a/tests/samplebinding/test_module_template.py
+++ b/tests/samplebinding/test_module_template.py
@@ -8,3 +8,5 @@ class MyObjectType(ObjectType):
 class MyOtherObjectType(OtherObjectType):
     value = 10
 
+
+obj = MyObjectType()