potiuk commented on a change in pull request #4938: [AIRFLOW-4117]
Multi-staging Image - Travis CI tests [Step 3/3]
URL: https://github.com/apache/airflow/pull/4938#discussion_r300241319
##########
File path: tests/contrib/hooks/test_jdbc_hook.py
##########
@@ -47,7 +47,7 @@ def test_jdbc_conn_connection(self, jdbc_mock):
jdbc_conn = jdbc_hook.get_conn()
self.assertTrue(jdbc_mock.called)
self.assertIsInstance(jdbc_conn, Mock)
- self.assertEqual(jdbc_conn.name, jdbc_mock.return_value.name)
+ self.assertEqual(jdbc_conn.name, jdbc_mock.return_value.name) #
pylint: disable=no-member
Review comment:
I reverted migration changes but test_jdbc_hook was beacause now we are
using newer version of pylint (previously we fixed it a 2.3.1. Now pylint from
setup.py is used and not fixed. test_jdbc_hook.py was not on the
pylint_todo.txt list :(
----------------------------------------------------------------
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
