darrenleeweber edited a comment on issue #6764: [WIP][AIRFLOW-6206] Use "aws_" 
prefix for aws_batch_operator.py module [AIP-21]
URL: https://github.com/apache/airflow/pull/6764#issuecomment-564224731
 
 
   The latest commit will be squashed when it's working.  The last piece is to 
get the core-to-contrib tests all passing, but I'm not understanding the 
failure for:
   ```
   FAILED 
tests/test_core_to_contrib.py::TestMovingCoreToContrib::test_is_class_deprecated_7_airflow_providers_amazon_aws_operators_batch_AwsBatchOperator
   - AssertionError: DeprecationWarning not triggered
   ```
   The deprecation warning is coded for the module, but the class name is also 
changing from `AWSBatchOperator` to `AwsBatchOperator` and maybe that it 
throwing off the parameterized test somehow?  The level of abstraction in the 
parameterized test is bending my mind a bit too far to immediately grok the 
problem.
   
   Also, need to update doc links.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to