Merge branch 'newlodel' of git.labocleo.org:lodel2 into newlodel

em_test.py

@@ -45,12 +45,14 @@ em_object.new_field(    'date_create',
                         group = base_group,
                         data_handler = 'datetime',
                         now_on_create = True,
+                        internal = True,
 )
 em_object.new_field(    'date_update',
                         display_name = 'Last update',
                         group = base_group,
                         data_handler = 'datetime',
                         now_on_update = True,
+                        internal = True,
 )
 
 ########################

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

plugins/mongodb_datasource/migration_handler.py

@@ -38,6 +38,10 @@ class MongoDbMigrationHandler(object):
         if len(conn_args.keys()) == 0:
             raise MigrationHandlerError("No connection arguments were given")
 
+        if 'host' not in conn_args.keys() or 'port' not in conn_args.keys() or 'db_name' not in conn_args.keys() \
+            or 'username' not in conn_args.keys() or 'password' not in conn_args.keys():
+            raise MigrationHandlerError("Missing connection arguments")
+
         #self.connection_name = conn_args['name']
         self.database = connect(host=conn_args['host'], port=conn_args['port'], db_name=conn_args['db_name'],
                                 username=conn_args['username'], password=conn_args['password'])

tests/leapi/test_leobject.py

@@ -36,4 +36,28 @@ class LeFilteredQueryTestCase(unittest.TestCase):
         with self.assertRaises(LeApiErrors):
             dyncode.Person(lastname = "foo", firstname = "bar")
 
-        
+    
+    def test_initilized(self):
+        """ Testing initialized method """
+        inst = dyncode.Person(
+            lodel_id = 1, lastname="foo")
+        self.assertFalse(inst.initialized)
+
+    def test_uid_fieldname(self):
+        self.assertEqual(dyncode.Person.uid_fieldname(), ["lodel_id"])
+
+    def test_fieldnames_accessor(self):
+        """ Testing fieldnames() accessor method """
+        fnames = dyncode.Person.fieldnames(False)
+        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'})
+