Fix

Author: forman

chartlets.py/chartlets/callback.py

@@ -57,7 +57,7 @@ def from_decorator(
                     f" must be of type Input, State, or Output,"
                     f" but got {arg.__class__.__name__!r}"
                 )
-            if not isinstance(arg, Output):
+            if isinstance(arg, Output):
                 outputs.append(arg)
             else:
                 inputs.append(arg)

chartlets.py/tests/callback_test.py

@@ -158,16 +158,17 @@ def test_decorator_target(self):
     def test_decorator_args(self):
         with pytest.raises(
             TypeError,
-            match="arguments for decorator 'test' must be of"
-            " type Input, but got 'int'",
+            match=(
+                "arguments for decorator 'test' must be of type"
+                " Input, State, or Output, but got 'int'"
+            ),
         ):
             Callback.from_decorator("test", (13,), my_callback)
 
         with pytest.raises(
             TypeError,
             match=(
-                "arguments for decorator 'test' must be of type Input,"
-                " State, Output, AppInput, or AppOutput, but got 'int'"
+                "arguments for decorator 'test' must be of type State, but got 'int'"
             ),
         ):
             Callback.from_decorator("test", (13,), my_callback, states_only=True)