[IMP] tests for OLS040*

fda-odoo · fda-odoo · commit 4cabb300407e · 2025-11-24T11:46:17.000+01:00
diff --git a/server/tests/data/addons/manifest_module_04001/__manifest__.py b/server/tests/data/addons/manifest_module_04001/__manifest__.py
@@ -0,0 +1,8 @@
+test = "something" # OLS04001
+{
+    "name": "Manifest Module 04001",
+    "version": "1.0",
+    "depends": [test],
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04002/__manifest__.py b/server/tests/data/addons/manifest_module_04002/__manifest__.py
@@ -0,0 +1,8 @@
+{
+    "name": "Manifest Module 04002",
+    "version": "1.0",
+    "depends": [],
+    "data": [],
+    "version": "1.5", # OLS04002
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04003_4/__manifest__.py b/server/tests/data/addons/manifest_module_04003_4/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": 42, #OLS04003
+    "version": "1.0",
+    "depends": "test", # OLS04004
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04005/__manifest__.py b/server/tests/data/addons/manifest_module_04005/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04005",
+    "version": "1.0",
+    "depends": [42], # OLS04005
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04006/__manifest__.py b/server/tests/data/addons/manifest_module_04006/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04006",
+    "version": "1.0",
+    "depends": ["manifest_module_04006"], # OLS04006
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04007/__manifest__.py b/server/tests/data/addons/manifest_module_04007/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04007",
+    "version": "1.0",
+    "depends": [],
+    "data": None, # OLS04007
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04008/__manifest__.py b/server/tests/data/addons/manifest_module_04008/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04008",
+    "version": "1.0",
+    "depends": [],
+    "data": [None], # OLS04008
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04009/__manifest__.py b/server/tests/data/addons/manifest_module_04009/__manifest__.py
@@ -0,0 +1,8 @@
+{
+    "name": "Manifest Module 04009",
+    "version": "1.0",
+    "depends": [],
+    "data": [],
+    test: 5, # OLS04009
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04010/__manifest__.py b/server/tests/data/addons/manifest_module_04010/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04010",
+    "version": "1.0",
+    "depends": ["flebelebele"], # OLS04010
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04011/__manifest__.py b/server/tests/data/addons/manifest_module_04011/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04011",
+    "version": "1.0",
+    "depends": ["flebelebele"],
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04012/__manifest__.py b/server/tests/data/addons/manifest_module_04012/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04012",
+    "version": "1.0",
+    "depends": ["manifest_module_04012b"], # OLS04012
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/data/addons/manifest_module_04012b/__manifest__.py b/server/tests/data/addons/manifest_module_04012b/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Manifest Module 04012b",
+    "version": "1.0",
+    "depends": ["manifest_module_04012"], # OLS04012
+    "data": [],
+    "installable": True,
+}
diff --git a/server/tests/diagnostics/mod.rs b/server/tests/diagnostics/mod.rs
@@ -15,4 +15,5 @@ pub mod ols02002;
 pub mod ols03000_1_to_23;
 pub mod ols03301;
 pub mod ols03302;
+pub mod ols04000_1_to_12;
 pub mod ols05000s;
diff --git a/server/tests/diagnostics/ols04000_1_to_12.rs b/server/tests/diagnostics/ols04000_1_to_12.rs
@@ -0,0 +1,31 @@
+use std::env;
+
+use odoo_ls_server::utils::PathSanitizer;
+
+use crate::{setup::setup::*, test_utils::{verify_diagnostics_against_doc}};
+
+
+#[test]
+fn test_ols04001_to_12() {
+    // Setup server and session with test addons
+    let (mut odoo, config) = setup_server(true);
+    let mut session = create_init_session(&mut odoo, config);
+    let paths = vec![
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04001/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04002/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04003_4/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04005/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04006/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04007/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04008/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04009/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04010/__manifest__.py").sanitize(),
+        env::current_dir().unwrap().join("tests/data/addons/manifest_module_04012/__manifest__.py").sanitize(),
+    ];
+    let diagnostics_map = get_diagnostics_for_paths(&mut session, &paths);
+    for path in paths.iter() {
+        let doc_diags = get_diagnostics_test_comments(&mut session, &path);
+        let diagnostics = diagnostics_map.get(path).cloned().unwrap_or_default();
+        verify_diagnostics_against_doc(diagnostics, doc_diags);
+    }
+}
diff --git a/server/tests/setup/setup.rs b/server/tests/setup/setup.rs
@@ -1,4 +1,5 @@
 use core::str;
+use std::collections::HashMap;
 use std::{env, fs};
 
 use std::path::PathBuf;
@@ -99,6 +100,25 @@ pub fn get_diagnostics_for_path(session: &mut SessionInfo, path: &str) -> Vec<Di
     return res;
 }
 
+pub fn get_diagnostics_for_paths(session: &mut SessionInfo, paths: &Vec<String>) -> HashMap<String, Vec<Diagnostic>> {
+    let mut res = HashMap::new();
+    while let Some(msg) = session._consume_message() {
+        match msg {
+            Message::Notification(n) => {
+                if n.method == PublishDiagnostics::METHOD {
+                    let params: PublishDiagnosticsParams = serde_json::from_value(n.params).expect("Unable to parse PublishDiagnosticsParams");
+                    let params_path = FileMgr::uri2pathname(params.uri.as_str());
+                    if paths.contains(&params_path) {
+                        res.entry(params_path).or_insert_with(Vec::new).extend(params.diagnostics);
+                    }
+                }
+            },
+            _ => {}
+        }
+    }
+    return res;
+}
+
 pub fn get_diagnostics_test_comments(session: &mut SessionInfo, path: &str) -> Vec<(u32, Vec<String>)> {
     let file_mgr = session.sync_odoo.get_file_mgr();
     let file_mgr = file_mgr.borrow();
