You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/09/21 12:50:55 UTC

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #26554: Correctly set json_provider_class on Flask app so it uses our encoder

ephraimbuddy commented on code in PR #26554:
URL: https://github.com/apache/airflow/pull/26554#discussion_r976465956


##########
tests/www/test_app.py:
##########
@@ -240,3 +240,13 @@ def test_flask_cli_should_display_routes(self, capsys):
 
         output = capsys.readouterr()
         assert "/login/" in output.out
+
+
+def test_app_can_json_serialize_k8s_pod():
+    # This is mostly testing that we have correctly configured the JSON provider to use. Testing the k8s pos

Review Comment:
   ```suggestion
       # This is mostly testing that we have correctly configured the JSON provider to use. Testing the k8s pod
   ```
   Seems like a typo?



-- 
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