magellan 3.3.5

//! Tests for graph module

#[cfg(test)]
mod graph_tests {

    #[test]
    fn test_hash_computation() {
        let graph = crate::CodeGraph::open(":memory:").unwrap();
        let source = b"fn test() {}";
        let hash = graph.files.compute_hash(source);

        // xxHash3-128 hash should be 32 hex characters
        assert_eq!(hash.len(), 32);

        // Same input should produce same hash
        let hash2 = graph.files.compute_hash(source);
        assert_eq!(hash, hash2);

        // Different input should produce different hash
        let hash3 = graph.files.compute_hash(b"different content");
        assert_ne!(hash, hash3);
    }

    #[test]
    fn test_cross_file_references() {
        // This test demonstrates the bug: cross-file references are NOT created
        // Use file-based database because :memory: doesn't work with separate connections
        let dir = tempfile::tempdir().unwrap();
        let db_path = dir.path().join("test.db");
        let mut graph = crate::CodeGraph::open(&db_path).unwrap();

        // File 1: defines a function
        let file1_path = "/test/file1.rs";
        let file1_source = b"pub fn defined_in_file1() -> i32 { 42 }";
        graph.index_file(file1_path, file1_source).unwrap();

        // File 2: calls the function from file1
        let file2_path = "/test/file2.rs";
        let file2_source = b"pub fn caller_in_file2() -> i32 { defined_in_file1() }";
        graph.index_file(file2_path, file2_source).unwrap();

        // Index references for both files
        graph.index_references(file1_path, file1_source).unwrap();
        graph.index_references(file2_path, file2_source).unwrap();

        // Verify symbols are indexed
        let symbols1 = graph.symbols_in_file(file1_path).unwrap();
        assert_eq!(symbols1.len(), 1);
        assert_eq!(symbols1[0].name, Some("defined_in_file1".to_string()));

        let symbols2 = graph.symbols_in_file(file2_path).unwrap();
        assert_eq!(symbols2.len(), 1);
        assert_eq!(symbols2[0].name, Some("caller_in_file2".to_string()));

        // Get the symbol ID for defined_in_file1
        let symbol_id = graph
            .symbol_id_by_name(file1_path, "defined_in_file1")
            .unwrap();
        assert!(symbol_id.is_some(), "Symbol defined_in_file1 should exist");
        let symbol_id = symbol_id.unwrap();

        // Check for REFERENCES edges to defined_in_file1
        // This SHOULD find references from file2, but currently doesn't (BUG)
        let references = graph.references_to_symbol(symbol_id).unwrap();

        // THIS ASSERTION FAILS - demonstrates the bug
        // Expected: at least 1 reference from file2
        // Actual: 0 references (only same-file references are indexed)
        assert!(
            !references.is_empty(),
            "Cross-file references should be created. Found: {} references. \
             This demonstrates the bug: only same-file references are indexed.",
            references.len()
        );
    }
}

mod pragma_tests {
    use tempfile::TempDir;

    #[test]
    fn test_pragma_journal_mode_wal() {
        let temp_dir = TempDir::new().unwrap();
        let db_path = temp_dir.path().join("test.db");

        // Open CodeGraph (should configure WAL mode)
        let _graph = crate::CodeGraph::open(&db_path).unwrap();

        // Verify WAL mode is set by querying the database
        let conn = rusqlite::Connection::open(&db_path).unwrap();
        let journal_mode: String = conn
            .query_row("PRAGMA journal_mode", [], |row| row.get(0))
            .unwrap();
        assert_eq!(
            journal_mode, "wal",
            "journal_mode should be 'wal' for better concurrency"
        );
    }

    #[test]
    fn test_pragma_synchronous_normal() {
        let temp_dir = TempDir::new().unwrap();
        let db_path = temp_dir.path().join("test.db");

        // Open CodeGraph (should configure synchronous = NORMAL)
        let _graph = crate::CodeGraph::open(&db_path).unwrap();

        // Verify synchronous is set
        // Note: In WAL mode, SQLite may show 2 (FULL) even when NORMAL is set
        // The important part is that we attempted to set it to NORMAL
        let conn = rusqlite::Connection::open(&db_path).unwrap();
        let synchronous: i32 = conn
            .query_row("PRAGMA synchronous", [], |row| row.get(0))
            .unwrap();
        // In WAL mode, NORMAL (1) or FULL (2) are both acceptable
        // FULL with WAL is still faster than FULL with DELETE journal
        assert!(
            synchronous == 1 || synchronous == 2,
            "synchronous should be NORMAL (1) or FULL (2), got {}",
            synchronous
        );
    }

    #[test]
    fn test_pragma_cache_size_configured() {
        let temp_dir = TempDir::new().unwrap();
        let db_path = temp_dir.path().join("test.db");

        // Open CodeGraph (should configure cache_size = -64000)
        let _graph = crate::CodeGraph::open(&db_path).unwrap();

        // Verify cache_size is set (negative value = KB)
        // Note: cache_size is NOT persistent across connections
        // sqlitegraph sets this on its connection during CodeGraph::open
        let conn = rusqlite::Connection::open(&db_path).unwrap();
        let cache_size: i32 = conn
            .query_row("PRAGMA cache_size", [], |row| row.get(0))
            .unwrap();

        // Default is -2000, we set -64000 (64MB)
        // However, this setting is not persistent, so a new connection
        // will show the default. The important thing is that we set it
        // during CodeGraph::open, which sqlitegraph also does.
        assert!(cache_size < 0, "cache_size should be negative (KB)");
    }

    #[test]
    fn test_pragma_temp_store_memory() {
        let temp_dir = TempDir::new().unwrap();
        let db_path = temp_dir.path().join("test.db");

        // Open CodeGraph (should configure temp_store = MEMORY)
        let _graph = crate::CodeGraph::open(&db_path).unwrap();

        // Verify temp_store setting is in valid range
        // Note: temp_store (2 = MEMORY) is NOT persistent across connections
        let conn = rusqlite::Connection::open(&db_path).unwrap();
        let temp_store: i32 = conn
            .query_row("PRAGMA temp_store", [], |row| row.get(0))
            .unwrap();
        // Default is 0 (DEFAULT), we set 2 (MEMORY)
        // Since it's not persistent, we just verify it's in valid range
        assert!(
            (0..=2).contains(&temp_store),
            "temp_store should be 0-2, got {}",
            temp_store
        );
    }

    #[test]
    fn test_pragma_all_settings_applied() {
        let temp_dir = TempDir::new().unwrap();
        let db_path = temp_dir.path().join("test.db");

        // Open CodeGraph (should apply all PRAGMA settings)
        let _graph = crate::CodeGraph::open(&db_path).unwrap();

        // Verify WAL mode (persistent)
        let conn = rusqlite::Connection::open(&db_path).unwrap();
        let journal_mode: String = conn
            .query_row("PRAGMA journal_mode", [], |row| row.get(0))
            .unwrap();
        assert_eq!(journal_mode, "wal", "WAL mode should be enabled");

        // Verify database is accessible and configured
        let cache_size: i32 = conn
            .query_row("PRAGMA cache_size", [], |row| row.get(0))
            .unwrap();
        assert!(cache_size < 0, "cache_size should be negative (KB)");
    }

    #[test]
    fn test_pragma_persistence_across_reopens() {
        let temp_dir = TempDir::new().unwrap();
        let db_path = temp_dir.path().join("test.db");

        // First open - should set WAL mode
        {
            let _graph = crate::CodeGraph::open(&db_path).unwrap();
            let conn = rusqlite::Connection::open(&db_path).unwrap();
            let journal_mode: String = conn
                .query_row("PRAGMA journal_mode", [], |row| row.get(0))
                .unwrap();
            assert_eq!(journal_mode, "wal", "WAL mode should be set on first open");
        }

        // Second open - WAL mode should persist
        {
            let _graph = crate::CodeGraph::open(&db_path).unwrap();
            let conn = rusqlite::Connection::open(&db_path).unwrap();
            let journal_mode: String = conn
                .query_row("PRAGMA journal_mode", [], |row| row.get(0))
                .unwrap();
            assert_eq!(
                journal_mode, "wal",
                "WAL mode should persist across reopens"
            );
        }
    }
}