Add another test for LeObject.get() method

leobject/leobject.py

@@ -101,11 +101,6 @@ class _LeObject(object):
 
         letype,leclass = cls._prepare_targets(typename, classname)
 
-        #Fetching LeType
-        if typename is None:
-            if 'type_id' not in field_list:
-                field_list.append('type_id')
-
         #Checking field_list
         if field_list is None or len(field_list) == 0:
             #default field_list
@@ -114,7 +109,14 @@ class _LeObject(object):
             elif not (leclass is None):
                 field_list = leclass._fieldtypes.keys()
             else:
-                field_list = EditorialModel.classtype.common_fields.keys()
+                field_list = list(EditorialModel.classtypes.common_fields.keys())
+
+        #Fetching LeType
+        if letype is None:
+            if 'type_id' not in field_list:
+                field_list.append('type_id')
+
+
         field_list = cls._prepare_field_list(field_list, letype, leclass)
         
         #preparing filters

leobject/test/test_leobject.py

@@ -317,3 +317,65 @@ class LeObjectMockDatasourceTestCase(TestCase):
             LeObject.get(filters, field_list, Numero)
             dsmock.assert_called_with(Publication, Numero, fl_ds, filters_ds, rfilters_ds)
             dsmock.reset_mock()
+
+    @patch('leobject.datasources.dummy.DummyDatasource.get')
+    def test_get_incomplete_targer(self, dsmock):
+        """ Testing LeObject.get() method with partial target specifier """
+        from dyncode import Publication, Numero, LeObject
+
+        args = [
+            (
+                ['lodel_id'],
+                [],
+                None,
+                None,
+
+                ['lodel_id', 'type_id'],
+                [],
+                [],
+                None,
+                None
+            ),
+            (
+                [],
+                [],
+                None,
+                None,
+
+                list(EditorialModel.classtypes.common_fields.keys()),
+                [],
+                [],
+                None,
+                None
+            ),
+            (
+                ['lodel_id'],
+                [],
+                None,
+                Publication,
+
+                ['lodel_id', 'type_id'],
+                [],
+                [],
+                None,
+                Publication
+            ),
+            (
+                [],
+                [],
+                Numero,
+                None,
+
+                Numero._fields,
+                [],
+                [],
+                Numero,
+                Publication
+            ),
+        ]
+
+        for field_list, filters, letype, leclass, fl_ds, filters_ds, rfilters_ds, letype_ds, leclass_ds in args:
+            LeObject.get(filters, field_list, letype, leclass)
+            dsmock.assert_called_with(leclass_ds, letype_ds, fl_ds, filters_ds, rfilters_ds)
+            dsmock.reset_mock()
+