Fix linting

Author: wilhelmhb

dash/dash.py

@@ -98,6 +98,27 @@ class _NoUpdate(object):
 """
 
 
+def _handle_callback_args(*args):
+    """Split args into outputs, inputs and states"""
+    args = [
+        arg
+        # for backward compatibility, one arg can be a list
+        for arg_or_list in args
+        # flatten args that are lists
+        for arg in (
+            arg_or_list if isinstance(arg_or_list, (list, tuple))
+            else [arg_or_list])
+    ]
+    return [
+        # split according to type Output, Input, State
+        [arg for arg in args if isinstance(arg, class_)]
+        for class_ in [Output, Input, State]
+    ] + [
+        # keep list of args in order, for matching order
+        # in the callback's parameters
+        [arg for arg in args if not isinstance(arg, Output)]]
+
+
 # pylint: disable=too-many-instance-attributes
 # pylint: disable=too-many-arguments, too-many-locals
 class Dash(object):
@@ -861,7 +882,7 @@ def clientside_callback(self, clientside_function, *args):
         )
         ```
         """
-        output, inputs, state, callback_args = self._handle_callback_args(*args)
+        output, inputs, state, callback_args = _handle_callback_args(*args)
         self._insert_callback(output, inputs, state, callback_args)
 
         # If JS source is explicitly given, create a namespace and function
@@ -893,28 +914,8 @@ def clientside_callback(self, clientside_function, *args):
             "function_name": function_name,
         }
 
-    def _handle_callback_args(self, *args):
-        """Split args into outputs, inputs and states"""
-        args = [
-            arg
-            # for backward compatibility, one arg can be a list
-            for arg_or_list in args
-            # flatten args that are lists
-            for arg in (
-                arg_or_list if isinstance(arg_or_list, (list, tuple))
-                else [arg_or_list])
-        ]
-        return [
-            # split according to type Output, Input, State
-            [arg for arg in args if isinstance(arg, class_)]
-            for class_ in [Output, Input, State]
-        ] + [
-            # keep list of args in order, for matching order
-            # in the callback's parameters
-            [arg for arg in args if not isinstance(arg, Output)]]
-
     def callback(self, *args):
-        output, inputs, state, callback_args = self._handle_callback_args(*args)
+        output, inputs, state, callback_args = _handle_callback_args(*args)
         callback_id = self._insert_callback(output, inputs, state, callback_args)
 
         def wrap_func(func):

tests/integration/test_integration.py

@@ -647,9 +647,10 @@ def update_output(value):
 def test_inin019_callback_dep_types():
     app = Dash(__name__)
     app.layout = html.Div(
-        [html.Div("child", id="in1"), html.Div("state", id="state1"), html.Div(id="out1"),
-         html.Div("child", id="in2"), html.Div("state", id="state2"), html.Div(id="out2"),
-         html.Div("child", id="in3"), html.Div("state", id="state3"), html.Div(id="out3"),
+        [
+            html.Div("child", id="in1"), html.Div("state", id="state1"), html.Div(id="out1"),
+            html.Div("child", id="in2"), html.Div("state", id="state2"), html.Div(id="out2"),
+            html.Div("child", id="in3"), html.Div("state", id="state3"), html.Div(id="out3"),
         ]
     )