[IMP] add tests for request.env

Author: cammarosano

server/tests/data/addons/module_1/__init__.py

@@ -1,2 +1,3 @@
 from . import models
-from . import data
+from . import data
+from . import controllers

server/tests/data/addons/module_1/controllers/__init__.py

@@ -0,0 +1,2 @@
+# -*- coding: utf-8 -*-
+from . import test_http

server/tests/data/addons/module_1/controllers/test_http.py

@@ -0,0 +1,36 @@
+# -*- coding: utf-8 -*-
+from odoo import http
+from odoo.http import request
+
+
+class TestHttpController(http.Controller):
+    
+    @http.route('/test/request', type='http', auth='public')
+    def test_request_type(self):
+        """Test that request has correct type."""
+        # Hovering over 'request' should show Request class
+        req = request
+        
+        # Hovering over 'request.env' should show Environment | None
+        env = request.env
+        
+        # Accessing models via request.env
+        if request.env:
+            partner = request.env['res.partner']
+            users = request.env['res.users']
+        
+        return "OK"
+    
+    @http.route('/test/request/search', type='http', auth='user')
+    def test_request_env_usage(self):
+        """Test using request.env in typical scenarios."""
+        # Direct model access
+        partners = request.env['res.partner'].search([])
+        
+        # With sudo
+        admin_partners = request.env['res.partner'].sudo().search([])
+        
+        # Accessing user
+        current_user = request.env.user
+        
+        return "OK"

server/tests/test_hooks.rs

@@ -0,0 +1,129 @@
+use std::env;
+use std::path::PathBuf;
+use std::cell::RefCell;
+use std::rc::Rc;
+
+use odoo_ls_server::core::odoo::SyncOdoo;
+use odoo_ls_server::core::symbols::symbol::Symbol;
+use odoo_ls_server::core::file_mgr::FileInfo;
+use odoo_ls_server::utils::PathSanitizer;
+use odoo_ls_server::threads::SessionInfo;
+
+mod setup;
+mod test_utils;
+
+/// Test that odoo.http.request and request.env hooks work correctly.
+/// Tests hover, definition navigation, and model access for the request object.
+#[test]
+fn test_request_hooks() {
+    // Skip if no Odoo installation or version too old
+    if env::var("COMMUNITY_PATH").is_err() {
+        panic!("COMMUNITY_PATH not set");
+    }
+    
+    let (mut odoo, config) = setup::setup::setup_server(true);
+    let mut session = setup::setup::create_init_session(&mut odoo, config);
+    
+    // Check Odoo version (hooks require 15.3+)
+    if session.sync_odoo.version_major < 15 || (session.sync_odoo.version_major == 15 && session.sync_odoo.version_minor < 3) {
+        println!("Skipping test_request_hooks: Odoo version < 15.3");
+        return;
+    }
+    
+    let test_file = PathBuf::from(env!("CARGO_MANIFEST_DIR"))
+        .join("tests/data/addons/module_1/controllers/test_http.py")
+        .sanitize();
+    
+    let Some(file_symbol) = SyncOdoo::get_symbol_of_opened_file(&mut session, &PathBuf::from(&test_file)) else {
+        panic!("Failed to get file symbol for {}", test_file);
+    };
+    
+    let file_mgr = session.sync_odoo.get_file_mgr();
+    let file_info = file_mgr.borrow().get_file_info(&test_file).unwrap();
+    
+    // Run all test sections
+    test_request_type_hover(&mut session, &file_symbol, &file_info);
+    test_request_env_definition(&mut session, &file_symbol, &file_info);
+}
+
+/// Test that hovering over 'request' shows Request class
+/// and that hovering over 'request.env' shows Environment type
+fn test_request_type_hover(
+    session: &mut SessionInfo,
+    file_symbol: &Rc<RefCell<Symbol>>,
+    file_info: &Rc<RefCell<FileInfo>>
+) {
+    // Test 1: Hover over 'request' variable (line 11: req = request)
+    // Should show Request class
+    let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 11, 14)
+        .expect("Should get hover text for request");
+    
+    assert!(
+        hover_text.contains("Request"),
+        "Hover over 'request' should show Request class. Got: {}",
+        hover_text
+    );
+    
+    // Test 2: Hover over 'request.env' (line 14: env = request.env)
+    // Should show Environment | None
+    let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 14, 21)
+        .expect("Should get hover text for request.env");
+    
+    assert!(
+        hover_text.contains("Environment"),
+        "Hover over 'request.env' should show Environment type. Got: {}",
+        hover_text
+    );
+    assert!(
+        hover_text.contains("None"),
+        "Hover over 'request.env' should show None. Got: {}",
+        hover_text
+    );
+}
+
+/// Test that request.env provides correct definitions
+fn test_request_env_definition(
+    session: &mut SessionInfo,
+    file_symbol: &Rc<RefCell<Symbol>>,
+    file_info: &Rc<RefCell<FileInfo>>
+) {
+    // Test 1: Go-to-definition on 'request' import (line 3: from odoo.http import request)
+    // Should navigate to request variable in odoo.http
+    let definitions = test_utils::get_definition_locs(session, file_symbol, file_info, 2, 22);
+    
+    assert!(
+        !definitions.is_empty(),
+        "Should find definition for 'request' import"
+    );
+    
+    // Verify it points to odoo/http.py or similar
+    let target_uri = &definitions[0].target_uri.to_string();
+    assert!(
+        target_uri.contains("odoo/http.py"),
+        "Definition should point to http module. Got: {:?}",
+        target_uri
+    );
+    
+    // Test 2: Go-to-definition on 'request.env' (line 15: env = request.env)
+    // Should navigate to Environment class or env attribute
+    let definitions = test_utils::get_definition_locs(session, file_symbol, file_info, 14, 21);
+    
+    assert!(
+        !definitions.is_empty(),
+        "Should find definition for 'request.env'"
+    );
+    
+    // Test 3: Hover on request.env.user (line 34: current_user = request.env.user)
+    // Should show proper type for user
+    let hover_text = test_utils::get_hover_markdown(session, file_symbol, file_info, 33, 35);
+    
+    if let Some(text) = hover_text {
+        // env.user should have a type (typically res.users or User)
+        assert!(
+            !text.is_empty(),
+            "request.env.user should have hover information"
+        );
+    }
+}
+
+