[GitHub] [airflow] dstandish commented on a change in pull request #22000: Don't validate that Params are JSON when NOTSET

[GitBox <gi...@apache.org>]

dstandish commented on a change in pull request #22000:
URL: https://github.com/apache/airflow/pull/22000#discussion_r819880585



##########
File path: tests/models/test_param.py
##########
@@ -272,7 +273,24 @@ def return_num(num):
         ti = dr.get_task_instances()[0]
         assert ti.xcom_pull() == 'test'
 
-    def test_param_non_json_serializable(self):
+    @pytest.mark.parametrize(
+        'default, should_warn',
+        [
+            pytest.param(
+                {0, 1, 2},
+                True,
+                id='default-non-JSON-serializable',
+            ),
+            pytest.param(None, False, id='default-None'),  # Param init should not warn
+            pytest.param({"b": 1}, False, id='default-JSON-serializable'),  # Param init should not warn
+        ],
+    )
+    def test_param_json_warning(self, default, should_warn):
+        warning_msg = 'The use of non-json-serializable params is deprecated'

Review comment:
       yup meant to do that thanks




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org