|
| 1 | +use std::env; |
| 2 | +use std::path::PathBuf; |
| 3 | +use std::cell::RefCell; |
| 4 | +use std::rc::Rc; |
| 5 | + |
| 6 | +use odoo_ls_server::core::odoo::SyncOdoo; |
| 7 | +use odoo_ls_server::core::symbols::symbol::Symbol; |
| 8 | +use odoo_ls_server::core::file_mgr::FileInfo; |
| 9 | +use odoo_ls_server::utils::PathSanitizer; |
| 10 | +use odoo_ls_server::threads::SessionInfo; |
| 11 | + |
| 12 | +mod setup; |
| 13 | +mod test_utils; |
| 14 | + |
| 15 | +/// Test that odoo.http.request and request.env hooks work correctly. |
| 16 | +/// Tests hover and definition. |
| 17 | +#[test] |
| 18 | +fn test_controller() { |
| 19 | + let (mut odoo, config) = setup::setup::setup_server(true); |
| 20 | + let mut session = setup::setup::create_init_session(&mut odoo, config); |
| 21 | + |
| 22 | + // This test only applies to Odoo 15.3+. Skip otherwise. |
| 23 | + if session.sync_odoo.version_major < 15 || (session.sync_odoo.version_major == 15 && session.sync_odoo.version_minor < 3) { |
| 24 | + return; |
| 25 | + } |
| 26 | + |
| 27 | + let test_file = PathBuf::from(env!("CARGO_MANIFEST_DIR")) |
| 28 | + .join("tests/data/addons/module_1/controllers/main.py") |
| 29 | + .sanitize(); |
| 30 | + |
| 31 | + let Some(file_symbol) = SyncOdoo::get_symbol_of_opened_file(&mut session, &PathBuf::from(&test_file)) else { |
| 32 | + panic!("Failed to get file symbol for {}", test_file); |
| 33 | + }; |
| 34 | + |
| 35 | + let file_mgr = session.sync_odoo.get_file_mgr(); |
| 36 | + let file_info = file_mgr.borrow().get_file_info(&test_file).unwrap(); |
| 37 | + |
| 38 | + test_request_type_hover(&mut session, &file_symbol, &file_info); |
| 39 | + test_request_env_definition(&mut session, &file_symbol, &file_info); |
| 40 | + test_request_env_subscript(&mut session, &file_symbol, &file_info); |
| 41 | +} |
| 42 | + |
| 43 | +/// Test that hovering over 'request' shows Request class |
| 44 | +/// and that hovering over 'request.env' shows Environment type |
| 45 | +fn test_request_type_hover( |
| 46 | + session: &mut SessionInfo, |
| 47 | + file_symbol: &Rc<RefCell<Symbol>>, |
| 48 | + file_info: &Rc<RefCell<FileInfo>> |
| 49 | +) { |
| 50 | + // Test 1: Hover over 'request' variable (line 11: req = request) |
| 51 | + // Should show Request class |
| 52 | + let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 11, 14) |
| 53 | + .expect("Should get hover text for request"); |
| 54 | + |
| 55 | + assert!( |
| 56 | + hover_text.contains("Request"), |
| 57 | + "Hover over 'request' should show Request class. Got: {}", |
| 58 | + hover_text |
| 59 | + ); |
| 60 | + |
| 61 | + // Test 2: Hover over 'request.env' (line 14: env = request.env) |
| 62 | + // Should show Environment | None |
| 63 | + let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 14, 21) |
| 64 | + .expect("Should get hover text for request.env"); |
| 65 | + |
| 66 | + assert!( |
| 67 | + hover_text.contains("Environment"), |
| 68 | + "Hover over 'request.env' should show Environment type. Got: {}", |
| 69 | + hover_text |
| 70 | + ); |
| 71 | + assert!( |
| 72 | + hover_text.contains("None"), |
| 73 | + "Hover over 'request.env' should show None. Got: {}", |
| 74 | + hover_text |
| 75 | + ); |
| 76 | +} |
| 77 | + |
| 78 | +/// Test that request.env provides correct definitions |
| 79 | +fn test_request_env_definition( |
| 80 | + session: &mut SessionInfo, |
| 81 | + file_symbol: &Rc<RefCell<Symbol>>, |
| 82 | + file_info: &Rc<RefCell<FileInfo>> |
| 83 | +) { |
| 84 | + // Test 1: Go-to-definition on 'request' import (line 3: from odoo.http import request) |
| 85 | + // Should navigate to request variable in odoo.http |
| 86 | + let definitions = test_utils::get_definition_locs(session, file_symbol, file_info, 2, 22); |
| 87 | + |
| 88 | + assert!( |
| 89 | + !definitions.is_empty(), |
| 90 | + "Should find definition for 'request' import" |
| 91 | + ); |
| 92 | + |
| 93 | + // Verify it points to odoo/http.py or similar |
| 94 | + let target_uri = &definitions[0].target_uri.to_string(); |
| 95 | + assert!( |
| 96 | + target_uri.contains("odoo/http.py"), |
| 97 | + "Definition should point to http module. Got: {:?}", |
| 98 | + target_uri |
| 99 | + ); |
| 100 | + |
| 101 | + // Test 2: Go-to-definition on 'request.env' (line 15: env = request.env) |
| 102 | + // Should navigate to Environment class or env attribute |
| 103 | + let definitions = test_utils::get_definition_locs(session, file_symbol, file_info, 14, 21); |
| 104 | + |
| 105 | + assert!( |
| 106 | + !definitions.is_empty(), |
| 107 | + "Should find definition for 'request.env'" |
| 108 | + ); |
| 109 | + |
| 110 | + // Test 3: Hover on request.env.user (line 34: current_user = request.env.user) |
| 111 | + // Should show proper type for user |
| 112 | + let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 33, 35) |
| 113 | + .expect("Should get hover text for 'user'"); |
| 114 | + |
| 115 | + // env.user should have a type (typically res.users or User) |
| 116 | + assert!( |
| 117 | + !hover_text.is_empty(), |
| 118 | + "request.env.user should have hover information" |
| 119 | + ); |
| 120 | +} |
| 121 | + |
| 122 | +/// Test that request.env["model_name"] resolves to Model instance |
| 123 | +fn test_request_env_subscript( |
| 124 | + session: &mut SessionInfo, |
| 125 | + file_symbol: &Rc<RefCell<Symbol>>, |
| 126 | + file_info: &Rc<RefCell<FileInfo>> |
| 127 | +) { |
| 128 | + let partner_class = test_utils::PARTNER_CLASS_NAME(&session.sync_odoo.full_version); |
| 129 | + |
| 130 | + // Test 1: Hover over 'partner' variable (line 19: partner = request.env['res.partner']) |
| 131 | + // Should show Partner/ResPartner class |
| 132 | + let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 18, 12) |
| 133 | + .expect("Should get hover text for 'partner' variable"); |
| 134 | + |
| 135 | + assert!( |
| 136 | + hover_text.contains(partner_class), |
| 137 | + "Hover over 'partner' should show {} class. Got: {}", |
| 138 | + partner_class, |
| 139 | + hover_text |
| 140 | + ); |
| 141 | + |
| 142 | + // Test 2: Hover over .search on request.env['res.partner'].search (line 29) |
| 143 | + // Should display markdown content for the search method |
| 144 | + let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 27, 46) |
| 145 | + .expect("Should get hover text for .search method"); |
| 146 | + |
| 147 | + assert!( |
| 148 | + hover_text.contains("def search"), |
| 149 | + "Hover over .search should have markdown content and mention 'search'. Got: {}", |
| 150 | + hover_text |
| 151 | + ); |
| 152 | +} |
0 commit comments