Sort Python safe built-ins (#5781)

Co-authored-by: Jiajie Zhong <[email protected]>
Co-authored-by: Jiajie Zhong <[email protected]>

Author: susodapop

redash/query_runner/python.py

@@ -53,31 +53,31 @@ class Python(BaseQueryRunner):
     should_annotate_query = False
 
     safe_builtins = (
-        "sorted",
-        "reversed",
-        "map",
-        "any",
+        "abs",
         "all",
-        "slice",
+        "any",
+        "bool",
+        "complex",
+        "dict",
+        "divmod",
+        "enumerate",
         "filter",
+        "float",
+        "int",
         "len",
-        "next",
-        "enumerate",
-        "sum",
-        "abs",
-        "min",
+        "list",
+        "map",
         "max",
+        "min",
+        "next",
+        "reversed",
         "round",
-        "divmod",
+        "set",
+        "slice",
+        "sorted",
         "str",
-        "int",
-        "float",
-        "complex",
+        "sum",
         "tuple",
-        "set",
-        "list",
-        "dict",
-        "bool",
     )
 
     @classmethod

tests/query_runner/test_python.py

@@ -0,0 +1,9 @@
+from unittest import TestCase
+
+from redash.query_runner.python import Python
+
+
+class TestPython(TestCase):
+    def test_sorted_safe_builtins(self):
+        src = list(Python.safe_builtins)
+        assert src == sorted(src), 'Python safe_builtins package not sorted.'