fix(insight-variables): Fixed insight variables erroring

[Gilbert09]

Problem

• Insights with dashboard variables overrides were erroring due to how we were accessing the variable - we were expecting it to be a dict when it was actually a HogQLVariable
• Introduced by some query awaited stuff and changing how we parse the variables
• Reported in slack here

Changes

• Assume the variable is a HogQLVariable and ensure we return HogQLVariable types
• Added a test for this

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Added a test

[greptile-apps]

PR Summary

Fixed an issue where insights with dashboard variable overrides were erroring due to a type mismatch between expected dictionary format and actual HogQLVariable objects.

• Modified /posthog/hogql_queries/apply_dashboard_filters.py to properly handle HogQLVariable types in dashboard variable overrides
• Added proper type annotation for variables_overrides parameter in apply_dashboard_variables functions
• Changed variable access pattern from dictionary-style to property access for code_name attribute
• Added test case in /posthog/api/test/test_query.py to verify the fix works correctly with variable overrides

_{2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[tomasfarias]