More tests on LeOject

lodel/leapi/query.py

@@ -23,8 +23,10 @@ class LeQuery(object):
         from .leobject import LeObject
         if self._hook_prefix is None:
             raise NotImplementedError("Abstract class")
-        if not issubclass(target_class, LeObject):
-            raise TypeError("target class has to be a child class of LeObject")
+        if not inspect.isclass(target_class) or \
+            not issubclass(target_class, LeObject):
+            raise TypeError("target class has to be a child class of LeObject \
+but %s given"% target_class)
         self._target_class = target_class
         self._ro_datasource = target_class._ro_datasource
         self._rw_datasource = target_class._rw_datasource

tests/leapi/test_leobject.py

@@ -52,3 +52,12 @@ class LeFilteredQueryTestCase(unittest.TestCase):
         self.assertEqual(set(fnames),
             {'lastname', 'linked_texts', 'firstname', 'alias'})
 
+    def test_insert(self):
+        dyncode.Person.insert({'lastname': 'foo', 'firstname': 'bar'})
+    
+    @unittest.skip("wait")
+    def test_bad_insert(self):
+        """ Insert with bad arguments """
+        dyncode.Person.insert({})
+        dyncode.Person.insert({'lodel_id': 1,'lastname': 'foo', 'firstname': 'bar'})
+