mirror of https://github.com/apache/superset.git
fix: use upload v2 for slack (#28783)
This commit is contained in:
parent
ce568c34f9
commit
a608bdb8d4
|
@ -184,7 +184,7 @@ Error: %(text)s
|
|||
# files_upload returns SlackResponse as we run it in sync mode.
|
||||
if files:
|
||||
for file in files:
|
||||
client.files_upload(
|
||||
client.files_upload_v2(
|
||||
channels=channel,
|
||||
file=file,
|
||||
initial_comment=body,
|
||||
|
|
|
@ -1098,7 +1098,7 @@ def test_email_dashboard_report_schedule_force_screenshot(
|
|||
@pytest.mark.usefixtures(
|
||||
"load_birth_names_dashboard_with_slices", "create_report_slack_chart"
|
||||
)
|
||||
@patch("superset.utils.slack.WebClient.files_upload")
|
||||
@patch("superset.utils.slack.WebClient.files_upload_v2")
|
||||
@patch("superset.utils.screenshots.ChartScreenshot.get_screenshot")
|
||||
def test_slack_chart_report_schedule(
|
||||
screenshot_mock,
|
||||
|
@ -1186,7 +1186,7 @@ def test_slack_chart_report_schedule_with_errors(
|
|||
@pytest.mark.usefixtures(
|
||||
"load_birth_names_dashboard_with_slices", "create_report_slack_chart_with_csv"
|
||||
)
|
||||
@patch("superset.utils.slack.WebClient.files_upload")
|
||||
@patch("superset.utils.slack.WebClient.files_upload_v2")
|
||||
@patch("superset.utils.csv.urllib.request.urlopen")
|
||||
@patch("superset.utils.csv.urllib.request.OpenerDirector.open")
|
||||
@patch("superset.utils.csv.get_chart_csv_data")
|
||||
|
@ -1353,7 +1353,7 @@ def test_report_schedule_success_grace(create_alert_slack_chart_success):
|
|||
|
||||
|
||||
@pytest.mark.usefixtures("create_alert_slack_chart_grace")
|
||||
@patch("superset.utils.slack.WebClient.files_upload")
|
||||
@patch("superset.utils.slack.WebClient.files_upload_v2")
|
||||
@patch("superset.utils.screenshots.ChartScreenshot.get_screenshot")
|
||||
def test_report_schedule_success_grace_end(
|
||||
screenshot_mock, file_upload_mock, create_alert_slack_chart_grace
|
||||
|
|
Loading…
Reference in New Issue