[FIX] avoid exiting test_symbol_is_model if CachedModel is not there

Author: mmahrouss

server/src/core/python_odoo_builder.rs

@@ -398,7 +398,7 @@ impl PythonOdooBuilder {
         let base_model_syms = session.sync_odoo.get_symbol(session.sync_odoo.config.odoo_path.as_ref().unwrap(), &base_model_tree, u32::MAX);
         let model_syms = session.sync_odoo.get_symbol(session.sync_odoo.config.odoo_path.as_ref().unwrap(), &model_tree, u32::MAX);
         let transient_syms = session.sync_odoo.get_symbol(session.sync_odoo.config.odoo_path.as_ref().unwrap(), &transient_tree, u32::MAX);
-        if base_model_syms.is_empty() || model_syms.is_empty() || transient_syms.is_empty() {
+        if base_model_syms.is_empty(){
             //one of them is not already loaded, but that's not really an issue, as now odoo step has been merged
             //with arch eval step, some files will be odooed before loading the orm fully. In this case we should
             //ignore this error. Moreover if a base is set on the class, it means that the base has been loaded, so
@@ -409,16 +409,16 @@ impl PythonOdooBuilder {
             // });
             return false;
         }
-        if Rc::ptr_eq(symbol, &base_model_syms[0]) ||
-            Rc::ptr_eq(symbol, &model_syms[0]) ||
-            Rc::ptr_eq(symbol, &transient_syms[0])
+        if base_model_syms.first().map_or(false, |first| Rc::ptr_eq(symbol, first))
+            || model_syms.first().map_or(false, |first| Rc::ptr_eq(symbol, first))
+            || transient_syms.first().map_or(false, |first| Rc::ptr_eq(symbol, first))
         {
             return false;
         }
         if compare_semver(session.sync_odoo.full_version.as_str(), "19.1") >= Ordering::Equal{
             let cached_model_tree = (vec![Sy!("odoo"), Sy!("orm"), Sy!("models_cached")], vec![Sy!("CachedModel")]);
             let cached_model = session.sync_odoo.get_symbol(session.sync_odoo.config.odoo_path.as_ref().unwrap(), &cached_model_tree, u32::MAX);
-            if cached_model.is_empty() || Rc::ptr_eq(symbol, &cached_model[0]){
+            if cached_model.first().map_or(false, |first| Rc::ptr_eq(symbol, first)){
                 return false;
             }