potiuk commented on PR #26188:
URL: https://github.com/apache/airflow/pull/26188#issuecomment-1239371520

   Yes. In this case there are two solutions (the first is way worse):
   
   * `test_dag_renderer" could be moved to "tests/www" becasue it mostly tests 
"dag_edges" which is  airflow/www/views.py 
   
   or (probably way better solution)
   
   * `dag_edges` (and corresponding methods) should be moved to "airlfow/utils" 
because they are used by more than just views (they are used by both Views and 
Dot Renderer
   
   I am happy to add PR to do the latter - in this case, this failure simply 
uncovered an implicit dependency that we missed that should be removed.
    


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to