mirror of
https://github.com/apache/superset.git
synced 2026-04-19 08:04:53 +00:00
refactor: test connection raises only command exceptions (#12307)
* refactor: test connection raises only command exceptions * fix tests * fix tests * fix tests * lint fix
This commit is contained in:
committed by
GitHub
parent
fecfc34cd3
commit
c685c9ea8f
@@ -389,7 +389,9 @@ class TestDatabaseApi(SupersetTestCase):
|
||||
self.login(username="admin")
|
||||
response = self.client.post(uri, json=database_data)
|
||||
response_data = json.loads(response.data.decode("utf-8"))
|
||||
expected_response = {"message": "Could not connect to database."}
|
||||
expected_response = {
|
||||
"message": "Connection failed, please check your connection settings"
|
||||
}
|
||||
self.assertEqual(response.status_code, 422)
|
||||
self.assertEqual(response_data, expected_response)
|
||||
|
||||
@@ -431,7 +433,9 @@ class TestDatabaseApi(SupersetTestCase):
|
||||
self.login(username="admin")
|
||||
rv = self.client.put(uri, json=database_data)
|
||||
response = json.loads(rv.data.decode("utf-8"))
|
||||
expected_response = {"message": "Could not connect to database."}
|
||||
expected_response = {
|
||||
"message": "Connection failed, please check your connection settings"
|
||||
}
|
||||
self.assertEqual(rv.status_code, 422)
|
||||
self.assertEqual(response, expected_response)
|
||||
# Cleanup
|
||||
@@ -787,11 +791,10 @@ class TestDatabaseApi(SupersetTestCase):
|
||||
}
|
||||
url = "api/v1/database/test_connection"
|
||||
rv = self.post_assert_metric(url, data, "test_connection")
|
||||
self.assertEqual(rv.status_code, 400)
|
||||
self.assertEqual(rv.status_code, 422)
|
||||
self.assertEqual(rv.headers["Content-Type"], "application/json; charset=utf-8")
|
||||
response = json.loads(rv.data.decode("utf-8"))
|
||||
expected_response = {
|
||||
"driver_name": "broken",
|
||||
"message": "Could not load database driver: broken",
|
||||
}
|
||||
self.assertEqual(response, expected_response)
|
||||
@@ -803,11 +806,10 @@ class TestDatabaseApi(SupersetTestCase):
|
||||
"server_cert": None,
|
||||
}
|
||||
rv = self.post_assert_metric(url, data, "test_connection")
|
||||
self.assertEqual(rv.status_code, 400)
|
||||
self.assertEqual(rv.status_code, 422)
|
||||
self.assertEqual(rv.headers["Content-Type"], "application/json; charset=utf-8")
|
||||
response = json.loads(rv.data.decode("utf-8"))
|
||||
expected_response = {
|
||||
"driver_name": "mssql+pymssql",
|
||||
"message": "Could not load database driver: mssql+pymssql",
|
||||
}
|
||||
self.assertEqual(response, expected_response)
|
||||
|
||||
Reference in New Issue
Block a user